]> review.fuel-infra Code Review - openstack-build/neutron-build.git/commit
make floating IP specification test robust to races
authorKevin Benton <blak111@gmail.com>
Thu, 7 Jan 2016 23:40:27 +0000 (15:40 -0800)
committerKevin Benton <blak111@gmail.com>
Mon, 11 Jan 2016 07:18:15 +0000 (23:18 -0800)
commit9390dde943915da9fe0e21b8d4822a3f31a347cf
tree55da1be3c015d5e926264c1bd5618bbcc0bd4f86
parent8e5424865386dfce5c01864a63474711caad760b
make floating IP specification test robust to races

The get_unused_ip function returns an address that isn't in use on
the network at the time it's called. However, by the time the caller
uses it, it may have been allocated to another concurrent test if
the network is shared or external.

The test_create_floatingip_with_specified_ip_address test creates
a floating IP with an unused IP on the external network, which has
lots of activity on it from other tests so it would occasionally
have its address taken by another test.

This patch just catches the in use error and tries again with a
new IP.

Change-Id: I5a2f78a6da1e400dea07949f35964abb767edbb2
Closes-Bug: #1532033
neutron/tests/api/admin/test_floating_ips_admin_actions.py