reload_cfg doesn't work correctly

Bug #1809080 reported by cheng li
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
neutron
Won't Fix
Undecided
Unassigned

Bug Description

class ProcessManager[1] is used to managed processes in neutron agents. The reload_cfg function is used to reload configuration by sending HUB signal to the target process. This may work for a linux system administrator, where she/he updates the config file and then sends HUP signal to reload.
But in openstack, this is a bit different. I have read the code about dnsmasq[2] and haproxy. The parameters and config files are generated by a callback function. This function generates config files and return the command(with parameters). Sending HUP signal doesn't change parameters, neither regenerate config files.

I would suggest another way to reload/restart process which makes sure the callback function is called.

[1] https://github.com/openstack/neutron/blob/8c59cfe98eb15f024b6c24312c0bf51af02a5dc3/neutron/agent/linux/external_process.py#L50
[2] https://github.com/openstack/neutron/blob/8c59cfe98eb15f024b6c24312c0bf51af02a5dc3/neutron/agent/linux/dhcp.py#L457-L458

cheng li (chengli3)
Changed in neutron:
assignee: nobody → cheng li (chengli3)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to neutron (master)

Fix proposed to branch: master
Review: https://review.openstack.org/626829

Changed in neutron:
status: New → In Progress
Revision history for this message
Slawek Kaplonski (slaweq) wrote : auto-abandon-script

This bug has had a related patch abandoned and has been automatically un-assigned due to inactivity. Please re-assign yourself if you are continuing work or adjust the state as appropriate if it is no longer valid.

Changed in neutron:
assignee: cheng li (chengli3) → nobody
status: In Progress → New
tags: added: timeout-abandon
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on neutron (master)

Change abandoned by Slawek Kaplonski (<email address hidden>) on branch: master
Review: https://review.openstack.org/626829
Reason: This review is > 4 weeks without comment, and failed Jenkins the last time it was checked. We are abandoning this for now. Feel free to reactivate the review by pressing the restore button and leaving a 'recheck' comment to get fresh test results.

Changed in neutron:
status: New → Won't Fix
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.