Catch os.makedirs() exceptions in tempest.py
authorCédric Ollivier <cedric.ollivier@orange.com>
Wed, 26 Apr 2017 07:18:04 +0000 (09:18 +0200)
committerCédric Ollivier <cedric.ollivier@orange.com>
Wed, 26 Apr 2017 07:35:47 +0000 (09:35 +0200)
It can raise an error exception if the leaf directory already exists or
cannot be created [1].

[1] https://docs.python.org/2/library/os.html

JIRA: FUNCTEST-804

Change-Id: Ie410bccf1bb2dfd36df6e918c152ecf94844b5d7
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
functest/opnfv_tests/openstack/tempest/tempest.py

index 8fa413a..e6c6b44 100644 (file)
@@ -208,11 +208,9 @@ class TempestCommon(testcase.TestCase):
     def run(self):
 
         self.start_time = time.time()
-
-        if not os.path.exists(conf_utils.TEMPEST_RESULTS_DIR):
-            os.makedirs(conf_utils.TEMPEST_RESULTS_DIR)
-
         try:
+            if not os.path.exists(conf_utils.TEMPEST_RESULTS_DIR):
+                os.makedirs(conf_utils.TEMPEST_RESULTS_DIR)
             image_and_flavor = conf_utils.create_tempest_resources()
             conf_utils.configure_tempest(
                 self.DEPLOYMENT_DIR,