yahoo-eng-team team mailing list archive

[Miro Tomaska <2017037@xxxxxxxxxxxxxxxxxx>]

Public bug reported:

neutron.tests.functional.agent.linux.test_keepalived.KeepalivedManagerTestCase.test_keepalived_spawn 
can fail sometimes at clean up with "No such process" error see [1]

It can happen that KeepalivedManager.disable is not able to SIGTERM process in time and then we try to execute SIGKILL on that process but the process just finished causing SIGKILL to not find this process.
We dont want this condition to fail the test so catch and handle this condition in _stop_keepalived_manager

[1] https://32177b17610869e1b83a-af1192da167b817506a58f20e6af9357.ssl.cf2.rackcdn.com/periodic/opendev.org/openstack/neutron/master/neutron-functional-with-uwsgi-fips/dc3710e/testr_results.html
[2] https://opendev.org/openstack/neutron/src/branch/master/neutron/agent/linux/keepalived.py#L473-L486

** Affects: neutron
     Importance: Undecided
     Assignee: Miro Tomaska (mtomaska)
         Status: New

** Changed in: neutron
     Assignee: (unassigned) => Miro Tomaska (mtomaska)

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to neutron.
https://bugs.launchpad.net/bugs/2017037

Title:
  Improve keepalived test pid termination

Status in neutron:
  New

Bug description:
  neutron.tests.functional.agent.linux.test_keepalived.KeepalivedManagerTestCase.test_keepalived_spawn 
  can fail sometimes at clean up with "No such process" error see [1]

  It can happen that KeepalivedManager.disable is not able to SIGTERM process in time and then we try to execute SIGKILL on that process but the process just finished causing SIGKILL to not find this process.
  We dont want this condition to fail the test so catch and handle this condition in _stop_keepalived_manager

  [1] https://32177b17610869e1b83a-af1192da167b817506a58f20e6af9357.ssl.cf2.rackcdn.com/periodic/opendev.org/openstack/neutron/master/neutron-functional-with-uwsgi-fips/dc3710e/testr_results.html
  [2] https://opendev.org/openstack/neutron/src/branch/master/neutron/agent/linux/keepalived.py#L473-L486

To manage notifications about this bug go to:
https://bugs.launchpad.net/neutron/+bug/2017037/+subscriptions