wait_until_true is not rootwrap daemon friendly

Bug #1721796 reported by Jakub Libosvar
14
This bug affects 2 people
Affects Status Importance Assigned to Milestone
neutron
Invalid
Wishlist
Jakub Libosvar

Bug Description

As wait_until_true uses an eventlet timeout, it can happen that timeout exception is raised while waiting for output from rootwrap daemon. This leaves data in rwd socket and next attempt to run command via rootwrap daemon returns the previous data. There is a bug reported on oslo.rootwrap - https://bugs.launchpad.net/neutron/+bug/1654287

With a switch of fullstack tests to use rootwrap, it happens a lot that timeouts are raised while running commands in rwd (eg. http://logs.openstack.org/67/488567/2/check/gate-neutron-dsvm-fullstack-ubuntu-xenial/eb8f9a3/testr_results.html.gz)

This bug is to track down an eventlet free solution - we can generalize approach here https://review.openstack.org/#/c/421325/3/neutron/cmd/netns_cleanup.py and since we'll get rid of eventlet dependency, we can move wait_until_true to neutron-lib.

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/510161

Changed in neutron:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on neutron (master)

Change abandoned by Jakub Libosvar (<email address hidden>) on branch: master
Review: https://review.openstack.org/510161
Reason: Abandoning in favor of https://review.openstack.org/#/c/514547/

Revision history for this message
Jakub Libosvar (libosvar) wrote :

Bug https://bugs.launchpad.net/neutron/+bug/1654287 was fixed and released. We'll be monitoring this one but closing for now.

Changed in neutron:
status: In Progress → Invalid
Revision history for this message
Ihar Hrachyshka (ihar-hrachyshka) wrote :

I suggest we close the bug since oslo.rootwrap fixes landed and we haven't seen the failure since then. We can reopen it later if needed.

Changed in neutron:
importance: Medium → Wishlist
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

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