Refactor validate_ports function
Since 1.28 is ancient and all the stable branches have support
for `get_node_vifs` let's remove the unnecessary check.
Change-Id: I5b619321be876e04f4b29dce083ca0cc80c4ed5b
diff --git a/ironic_tempest_plugin/tests/scenario/test_baremetal_basic_ops.py b/ironic_tempest_plugin/tests/scenario/test_baremetal_basic_ops.py
index 4fa73ee..7912788 100644
--- a/ironic_tempest_plugin/tests/scenario/test_baremetal_basic_ops.py
+++ b/ironic_tempest_plugin/tests/scenario/test_baremetal_basic_ops.py
@@ -133,16 +133,7 @@
def validate_ports(self):
node_uuid = self.node['uuid']
- vifs = []
- # TODO(vsaienko) switch to get_node_vifs() when all stable releases
- # supports Ironic API 1.28
- if self._is_version_supported('1.28'):
- vifs = self.get_node_vifs(node_uuid)
- else:
- for port in self.get_ports(self.node['uuid']):
- vif = port['extra'].get('vif_port_id')
- if vif:
- vifs.append({'id': vif})
+ vifs = self.get_node_vifs(node_uuid)
ir_ports = self.get_ports(node_uuid)
ir_ports_addresses = [x['address'] for x in ir_ports]