Skip to content

Commit

Permalink
Fix uninitialized cleanup_failure var
Browse files Browse the repository at this point in the history
After merging changes for cleanup exception handling,
the cleanup_failure var was not correctly initialized.

Signed-off-by: Pablo Chacin <pchacin@suse.com>
  • Loading branch information
Pablo Chacin committed May 17, 2019
1 parent edcdb2f commit 3907504
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 0 deletions.
1 change: 1 addition & 0 deletions ci/infra/testrunner/platforms/terraform.py
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,7 @@ def _cleanup_platform(self):

def cleanup(self):
""" Clean up """
cleanup_failure = False
try:
self._cleanup_platform()
except Exception as ex:
Expand Down
1 change: 1 addition & 0 deletions ci/infra/testrunner/utils/caaspctl.py
Original file line number Diff line number Diff line change
Expand Up @@ -55,6 +55,7 @@ def cleanup(self):
os.path.join(self.conf.workspace, "ssh-agent-sock"),
os.path.join(self.conf.workspace, "test-cluster")]

cleanup_failure = False
for dir in dirs:
try:
self.utils.runshellcommand("rm -rf {}".format(dir))
Expand Down

0 comments on commit 3907504

Please sign in to comment.