From 0c81af1848a4b98da3b1049dde4b061598674d5e Mon Sep 17 00:00:00 2001 From: Andrey Kurilin Date: Fri, 21 Nov 2014 20:15:37 +0200 Subject: [PATCH] TEMPEST: refactor error handling while installation Tempest installation doesn't contain enough information for debugging errors, which can be occured during installation. This patch adds: - logging failed command and its output to: `rally.verification.verifiers.tempest.tempest.check_output`; - handling TempestSetupFailure in validation tempest scenarios; - fix in hacking check for logging module. Change-Id: Ida723c6898cb6845459169f387ca36bf280fb644 --- tests/hacking/checks.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tests/hacking/checks.py b/tests/hacking/checks.py index 871482b7..2c63da6c 100644 --- a/tests/hacking/checks.py +++ b/tests/hacking/checks.py @@ -82,10 +82,10 @@ def check_assert_methods_from_mock(logical_line, filename): def check_import_of_logging(logical_line, filename): """Check correctness import of logging module N310.""" - excluded_files = ["./rally/log.py"] + excluded_files = ["./rally/log.py", "./tests/unit/test_log.py"] forbidden_imports = ["from rally.openstack.common import log", - "import rally.openstack.common.log" + "import rally.openstack.common.log", "import logging"] if filename not in excluded_files: