radware lbaas unit test failed: MismatchError: 409 != 204

[Akihiro Motoki]

neutron.tests.unit.services.loadbalancer.drivers.radware.test_plugin_driver.TestLoadBalancerPlugin.test_create_vip failed:
http://logs.openstack.org/63/76363/2/check/gate-neutron-python27/ca7bdc5/

(I might see similar issue, but i could not find it.)

Traceback (most recent call last):
  File "neutron/tests/unit/services/loadbalancer/drivers/radware/test_plugin_driver.py", line 234, in test_create_vip
    self.rest_call_mock.assert_has_calls(calls, any_order=True)
  File "/usr/lib/python2.7/contextlib.py", line 35, in __exit__
    self.gen.throw(type, value, traceback)
  File "neutron/tests/unit/test_db_plugin.py", line 559, in subnet
    self._delete('subnets', subnet['subnet']['id'])
  File "/usr/lib/python2.7/contextlib.py", line 35, in __exit__
    self.gen.throw(type, value, traceback)
  File "neutron/tests/unit/test_db_plugin.py", line 530, in network
    self._delete('networks', network['network']['id'])
  File "neutron/tests/unit/test_db_plugin.py", line 446, in _delete
    self.assertEqual(res.status_int, expected_code)
  File "/home/jenkins/workspace/gate-neutron-python27/.tox/py27/local/lib/python2.7/site-packages/testtools/testcase.py", line 321, in assertEqual
    self.assertThat(observed, matcher, message)
  File "/home/jenkins/workspace/gate-neutron-python27/.tox/py27/local/lib/python2.7/site-packages/testtools/testcase.py", line 406, in assertThat
    raise mismatch_error
MismatchError: 409 != 204

[Akihiro Motoki (amotoki)]

It causes unit test failure, so I set its priority to High.

[Akihiro Motoki (amotoki)]

bug 1245208 may be related.

[OpenStack Infra (hudson-openstack)]

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

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/78554
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=3cbf7bc598d44a6326cdbd94b210385599bab9be
Submitter: Jenkins
Branch: master

commit 3cbf7bc598d44a6326cdbd94b210385599bab9be
Author: Eugene Nikanorov <email address hidden>
Date: Thu Mar 6 12:41:10 2014 +0400

    Fix unittest failure in radware lbaas driver

    Issue greenthread.sleep(0) to switch greenthreads and wait
    for vip to complete deletion

    Change-Id: I46b0702b0df527965ecf94e334ed3327f408a05c
    Closes-Bug: #1288312

[YAMAMOTO Takashi (yamamoto)]

i saw the failure which looks same to me today. probably the fix was not enough?
http://logs.openstack.org/61/78861/1/check/gate-neutron-python26/38315b7/

[OpenStack Infra (hudson-openstack)]

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

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/79888
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=21aa7e3f78094837ca63d3742459c7a8d011c9dd
Submitter: Jenkins
Branch: master

commit 21aa7e3f78094837ca63d3742459c7a8d011c9dd
Author: Eugene Nikanorov <email address hidden>
Date: Wed Mar 12 12:35:45 2014 +0400

    Fix unittest failure in radware lbaas driver

    Wait for vip delete completion before deleting network
    and subnet.
    Also fix indentation in the driver

    Closes-Bug: #1288312
    Change-Id: I88b3252102d6c4a60bbadf2951bbb74fcd19792b

[Mark McClain (markmcclain)]

reopening as this bug still occurs

[OpenStack Infra (hudson-openstack)]

Related fix proposed to branch: master
Review: https://review.openstack.org/80426

[Mark McClain (markmcclain)]

The Radware team will taking over management of this bug.

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/80426
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=3f0af0e4dcf55cfa6ba23a27507b6b27f899e139
Submitter: Jenkins
Branch: master

commit 3f0af0e4dcf55cfa6ba23a27507b6b27f899e139
Author: Aaron Rosen <email address hidden>
Date: Thu Mar 13 17:14:52 2014 -0700

    Skip radware failing test for now

    Change-Id: Ic6c9b1aad20c732864fc4569fb566263df0f6879
    Related-bug: #1288312

[Samuel Bercovici (samuelb)]

The usage of a threadduring tests was fixed and merged.
see:: https://review.openstack.org/#/c/81323/
May we close this bug?

[Eugene Nikanorov (enikanorov)]

Sam, 'Fix Committed' is one of the forms of closing the bug