IMPALA-5424: Ignore errors when removing minidumps folder

On developer machines it can happen that /tmp/minidumps does not exists
when test_minidump_relative_path gets executed. In this case errors from
rmtree should be ignored.

Change-Id: Ifab76a30898805d2df5e7452079a536d8747ac50
Reviewed-on: http://gerrit.cloudera.org:8080/7062
Reviewed-by: Alex Behm <[email protected]>
Tested-by: Impala Public Jenkins


Project: http://git-wip-us.apache.org/repos/asf/incubator-impala/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-impala/commit/5518cbcb
Tree: http://git-wip-us.apache.org/repos/asf/incubator-impala/tree/5518cbcb
Diff: http://git-wip-us.apache.org/repos/asf/incubator-impala/diff/5518cbcb

Branch: refs/heads/master
Commit: 5518cbcb78a708047dc7056afc1f41e7f1dd8f7d
Parents: 1b9cb04
Author: Lars Volker <[email protected]>
Authored: Fri Jun 2 11:03:37 2017 -0700
Committer: Impala Public Jenkins <[email protected]>
Committed: Fri Jun 2 23:23:52 2017 +0000

----------------------------------------------------------------------
 tests/custom_cluster/test_breakpad.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-impala/blob/5518cbcb/tests/custom_cluster/test_breakpad.py
----------------------------------------------------------------------
diff --git a/tests/custom_cluster/test_breakpad.py 
b/tests/custom_cluster/test_breakpad.py
index 5479a77..120e9b9 100644
--- a/tests/custom_cluster/test_breakpad.py
+++ b/tests/custom_cluster/test_breakpad.py
@@ -191,7 +191,7 @@ class TestBreakpad(CustomClusterTestSuite):
     written to 'log_dir'.
     """
     minidump_base_dir = os.path.join(os.environ.get('LOG_DIR', '/tmp'), 
'minidumps')
-    shutil.rmtree(minidump_base_dir)
+    shutil.rmtree(minidump_base_dir, ignore_errors=True)
     # Omitting minidump_path as a parameter to the cluster will choose the 
default
     # configuration, which is a FLAGS_log_dir/minidumps.
     self.start_cluster_with_args()

Reply via email to