Merge "Actually log state transition timeouts in debug log"
diff --git a/ironic_tempest_plugin/common/waiters.py b/ironic_tempest_plugin/common/waiters.py
index c5b9bd0..c9d87bd 100644
--- a/ironic_tempest_plugin/common/waiters.py
+++ b/ironic_tempest_plugin/common/waiters.py
@@ -12,6 +12,7 @@
# License for the specific language governing permissions and limitations
# under the License.
+from oslo_log import log
import six
from tempest import config
from tempest.lib.common.utils import test_utils
@@ -19,6 +20,8 @@
from ironic_tempest_plugin.common import utils
+LOG = log.getLogger(__name__)
+
CONF = config.CONF
@@ -73,12 +76,14 @@
return True
elif (abort_on_error_state
and node['provision_state'].endswith(' failed')):
- raise lib_exc.TempestException(
- 'Node %(node)s reached failure state %(state)s while waiting '
- 'for %(attr)s=%(expected)s. Error: %(error)s' %
- {'node': node_id, 'state': node['provision_state'],
- 'attr': attr, 'expected': status,
- 'error': node.get('last_error')})
+ msg = ('Node %(node)s reached failure state %(state)s while '
+ 'waiting for %(attr)s=%(expected)s. '
+ 'Error: %(error)s' %
+ {'node': node_id, 'state': node['provision_state'],
+ 'attr': attr, 'expected': status,
+ 'error': node.get('last_error')})
+ LOG.debug(msg)
+ raise lib_exc.TempestException(msg)
return False
if not test_utils.call_until_true(is_attr_in_status, timeout,
@@ -92,6 +97,7 @@
caller = test_utils.find_test_caller()
if caller:
message = '(%s) %s' % (caller, message)
+ LOG.debug(message)
raise lib_exc.TimeoutException(message)