Comment 4 for bug 1512275

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Fix merged to openstack/neutron (openstack-ci/fuel-8.0/liberty)

Reviewed: https://review.fuel-infra.org/13535
Submitter: Pkgs Jenkins <email address hidden>
Branch: openstack-ci/fuel-8.0/liberty

Commit: d6ff039e97f0afab3829e90e33952888ab49384a
Author: Elena Ezhova <email address hidden>
Date: Fri Nov 20 11:30:45 2015

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.

Conflicts:
 neutron/tests/functional/agent/test_l3_agent.py

Closes-Bug: #1512275
Cherry-picked from https://review.openstack.org/#/c/240267/
Change-Id: I48eb4697a5fad97bcf08cfe1f80921a46d94029d