Bug 1079758

Summary: Add support for systemctl-reload action
Product: [Fedora] Fedora Reporter: Niels de Vos <ndevos>
Component: radvdAssignee: Pavel Šimerda (pavlix) <psimerda>
Status: CLOSED RAWHIDE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: low Docs Contact:
Priority: medium    
Version: rawhideCC: jpopelka, psimerda
Target Milestone: ---Keywords: EasyFix, Patch
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2014-03-24 16:10:25 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Attachments:
Description Flags
Proposed patch got the radvd.service file none

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.