neutron

Ensure "keepalived" is correctly disabled (the process does not exist)

Bug #1908057 reported by Rodolfo Alonso on 2020-12-14

This bug affects 1 person

Affects		Status	Importance	Assigned to	Milestone
	neutron	Fix Committed	Medium	Rodolfo Alonso

Bug Description

Ensure "keepalived" is correctly disabled, that means the process does not exist. Currently the process is terminated sending a SIGTERM signal [1]. If that fails, for any reason, the "keepalived" process will continue running. For example in [2][3]: the SIGTERM signal is sent but the process is still running ("psutil.Process(pid)" is able to retrieve the process information).

[1]https://github.com/openstack/neutron/blob/28c79c9747a3031a0d1321199d2d72336e4076ed/neutron/agent/linux/keepalived.py#L475-L480
[2]https://be8bc338a39cf869db17-e8a1a74829c12f710e3c5d0fa3d85b60.ssl.cf1.rackcdn.com/763231/10/check/neutron-functional-with-uwsgi/db6a5a3/controller/logs/dsvm-functional-logs/neutron.tests.functional.agent.l3.extensions.test_gateway_ip_qos_extension.TestRouterGatewayIPQosAgentExtensionDVR.test_dvr_router_lifecycle_ha_with_snat_with_fips.txt
[3]http://paste.openstack.org/show/801003/

Tags:

Slawek Kaplonski (slaweq) on 2020-12-14

Changed in neutron:
importance:	Undecided → Medium
importance:	Medium → Critical
status:	New → Confirmed
importance:	Critical → Medium
tags:	added: l3-ha

Rodolfo Alonso (rodolfo-alonso-hernandez) on 2020-12-15

Changed in neutron:
assignee:	nobody → Rodolfo Alonso (rodolfo-alonso-hernandez)