]> review.fuel-infra Code Review - openstack-build/neutron-build.git/commit
Use SIGUSR1 to notify l3 agent of changing prefix file
authorElena Ezhova <eezhova@mirantis.com>
Fri, 13 Nov 2015 11:29:38 +0000 (14:29 +0300)
committerElena Ezhova <eezhova@mirantis.com>
Fri, 13 Nov 2015 14:53:47 +0000 (17:53 +0300)
commitbb16a2ef6c768d6fd685ab36f9dc94002f76e768
tree30cb46c9f0a9b0dfcea6c313393985e4c44ba4cd
parentdab4acd15dc23f7585f4cfde0138bcfc5c684137
Use SIGUSR1 to notify l3 agent of changing prefix file

It is common for all OpenStack services to use SIGHUP signal to
reload configuration and restart. This functionality, including
defining signal handler, is defined by oslo.service.

Meanwhile, this is currently not so for l3 agent. PrefixDelegation
class that is instantiated in L3NATAgent overrides handler for SIGHUP,
thus removing handler that was set in oslo.service.

The proposed solution is to use another signal, such as SIGUSR1,
instead of SIGHUP to notify l3 agent that a prefix file was
somehow changed.

Added a functional test for restarting L3 agent using SIGHUP.

Change-Id: I48eb4697a5fad97bcf08cfe1f80921a46d94029d
Closes-Bug: #1511401
neutron/agent/linux/pd.py
neutron/cmd/pd_notify.py
neutron/tests/functional/agent/test_l3_agent.py
neutron/tests/functional/test_service.py