]> review.fuel-infra Code Review - openstack-build/neutron-build.git/commit
Fixed functional test that validates graceful ovs agent restart
authorIhar Hrachyshka <ihrachys@redhat.com>
Fri, 4 Sep 2015 20:21:41 +0000 (22:21 +0200)
committerIhar Hrachyshka <ihrachys@redhat.com>
Fri, 4 Sep 2015 20:28:42 +0000 (22:28 +0200)
commited3c317ed9182538747f74713154ad94e9d866db
treec9d44f069191920419087ee024a7569df7b6342c
parentf4e980f7d5608b594e64d5ab78c594d61a664955
Fixed functional test that validates graceful ovs agent restart

The async_ping function returns a callable that returns True when all ping
futures are done. Since those futures are running for 10 secs, there was no
chance that the result of the callable was True.

The test was bailing out without calling bridge reset even a single time,
effectively leaving the feature untested in gate.

Another thing to note is that for some reason the patch fixed oslo rootwrap
errors in the test when executed locally. Since I still don't understand how
it's possible that it fixes the issue for me, I mark the bug as related only,
and will track logstash after it's merged to see whether it applies unknown
magic to gate jobs too.

Change-Id: Iaa977abddf1a0c6af7e964f1a5cd545ffb79585a
Related-Bug: #1490051
neutron/tests/functional/agent/test_l2_ovs_agent.py