Bug 1079758 - Add support for systemctl-reload action
Summary: Add support for systemctl-reload action
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: radvd
Version: rawhide
Hardware: Unspecified
OS: Unspecified
medium
low
Target Milestone: ---
Assignee: Pavel Šimerda (pavlix)
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2014-03-23 20:13 UTC by Niels de Vos
Modified: 2014-03-24 16:10 UTC (History)
2 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2014-03-24 16:10:25 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)
Proposed patch got the radvd.service file (695 bytes, patch)
2014-03-23 20:22 UTC, Niels de Vos
no flags Details | Diff

Description Niels de Vos 2014-03-23 20:13:19 UTC
Description of problem:
It seems that 'systemctl reload radvd' does not work.


Version-Release number of selected component (if applicable):
radvd-1.9.2-4.fc20.armv7hl


How reproducible:
100%


Steps to Reproduce:
1. # systemctl reload radvd

Actual results:
Failed to issue method call: Job type reload is not applicable for unit radvd.service.

Expected results:
No error, radvd should dynamically apply the new configuration.


Additional info:
The manual page from radvd.conf mentions that sending a SIGHUB reloads the
configuration.

From systemd.service:

       ExecReload=
	   Commands to execute to trigger a configuration reload in the
	   service. This argument takes multiple command lines, following the
	   same scheme as described for ExecStart= above. Use of this setting
	   is optional. Specifier and environment variable substitution is
	   supported here following the same scheme as for ExecStart=.

	   One additional special environment variables is set: if known
	   $MAINPID is set to the main process of the daemon, and may be used
	   for command lines like the following:

               /bin/kill -HUP $MAINPID

Comment 1 Niels de Vos 2014-03-23 20:22:23 UTC
Created attachment 877905 [details]
Proposed patch got the radvd.service file

Tested OK in my side.

Comment 2 Pavel Šimerda (pavlix) 2014-03-24 16:10:25 UTC
Fixed in rawhide.


Note You need to log in before you can comment on or make changes to this bug.