commit c8aa34b05bac2167f92c3cbf7247177dfa844835
author Vasyl Saienko <vsaienko@mirantis.com> Mon Aug 01 14:18:37 2016 +0300
committer Vasyl Saienko <vsaienko@mirantis.com> Tue Aug 02 08:59:40 2016 +0000
tree 8ecccf96619d1b1bd1a07437ac3a631b8ac6f0d1
parent 1493c7f0ba49bfccb9ff8516b10a65d949d7462e

Add a TODO to remove Ironic related workaround

This patch adds a TODO to do not forget to remove temporary workaround
that ignores 'DOWN' Neutron port status for baremetal instance.

Related-Bug: #1599836

Change-Id: I71232e61e977999e8beb105824ad04d9120ad453

tempest/scenario/manager.py

diff --git a/tempest/scenario/manager.py b/tempest/scenario/manager.py
index b151375..a0d6043 100644
--- a/tempest/scenario/manager.py
+++ b/tempest/scenario/manager.py
@@ -832,6 +832,7 @@
         # NOTE(vsaienko) With Ironic, instances live on separate hardware
         # servers. Neutron does not bind ports for Ironic instances, as a
         # result the port remains in the DOWN state.
+        # TODO(vsaienko) remove once bug: #1599836 is resolved.
         if CONF.service_available.ironic:
             p_status.append('DOWN')
         port_map = [(p["id"], fxip["ip_address"])