Fix 'Number of new ports: 2' failure mode on test_hotplug_nic
After change ce34d2fcf10 merged became evident that this test fails because
there are 'too' many ports that match the server id. Looking closer this is
due to the fact that we compare the entire port dictionaries instead of just
uuids. This may lead to spurious failures, when some port attributes (like
the STATUS) changes during the test execution.
It should suffice comparing uuids rather than dictionaries.
Change-Id: I5fd25689ad73258b092cef62e5657871eedd8b30
Closes-bug: #1321207
diff --git a/tempest/scenario/test_network_basic_ops.py b/tempest/scenario/test_network_basic_ops.py
index ae326c4..265f9e5 100644
--- a/tempest/scenario/test_network_basic_ops.py
+++ b/tempest/scenario/test_network_basic_ops.py
@@ -243,7 +243,7 @@
def check_ports():
self.new_port_list = [port for port in
self._list_ports(device_id=server['id'])
- if port != old_port]
+ if port['id'] != old_port['id']]
return len(self.new_port_list) == 1
if not test.call_until_true(check_ports, CONF.network.build_timeout,