kudu-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From t...@apache.org
Subject [3/4] incubator-kudu git commit: build: disable check for leaked tmp dirs
Date Fri, 22 Jan 2016 04:25:56 GMT
build: disable check for leaked tmp dirs

The Python tests appear to be leaking tmp directories occasionally due to some
race in cleanup (KUDU-1301). In order to fix this source of precommit
flakiness, we'll just disable this for now.

Change-Id: I9ea325da5866adb494e5419ce54a50b964454f7f
Reviewed-on: http://gerrit.cloudera.org:8080/1817
Reviewed-by: Todd Lipcon <todd@apache.org>
Tested-by: Todd Lipcon <todd@apache.org>


Project: http://git-wip-us.apache.org/repos/asf/incubator-kudu/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-kudu/commit/89380b53
Tree: http://git-wip-us.apache.org/repos/asf/incubator-kudu/tree/89380b53
Diff: http://git-wip-us.apache.org/repos/asf/incubator-kudu/diff/89380b53

Branch: refs/heads/master
Commit: 89380b53d48b085dfe394b9966360d1ef1a4b038
Parents: 23c6c66
Author: Todd Lipcon <todd@apache.org>
Authored: Tue Jan 19 15:53:33 2016 -0800
Committer: Todd Lipcon <todd@apache.org>
Committed: Fri Jan 22 04:21:52 2016 +0000

----------------------------------------------------------------------
 build-support/jenkins/build-and-test.sh | 24 ++++++++++++++----------
 1 file changed, 14 insertions(+), 10 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-kudu/blob/89380b53/build-support/jenkins/build-and-test.sh
----------------------------------------------------------------------
diff --git a/build-support/jenkins/build-and-test.sh b/build-support/jenkins/build-and-test.sh
index a3f7480..6401a33 100755
--- a/build-support/jenkins/build-and-test.sh
+++ b/build-support/jenkins/build-and-test.sh
@@ -305,16 +305,20 @@ if [ $EXIT_STATUS != 0 ]; then
 fi
 
 # If all tests passed, ensure that they cleaned up their test output.
-if [ $EXIT_STATUS == 0 ]; then
-  TEST_TMPDIR_CONTENTS=$(ls $TEST_TMPDIR)
-  if [ -n "$TEST_TMPDIR_CONTENTS" ]; then
-    echo "All tests passed, yet some left behind their test output:"
-    for SUBDIR in $TEST_TMPDIR_CONTENTS; do
-      echo $SUBDIR
-    done
-    EXIT_STATUS=1
-  fi
-fi
+#
+# TODO: Python is currently leaking a tmp directory sometimes (KUDU-1301).
+# Temporarily disabled until that's fixed.
+#
+# if [ $EXIT_STATUS == 0 ]; then
+#   TEST_TMPDIR_CONTENTS=$(ls $TEST_TMPDIR)
+#   if [ -n "$TEST_TMPDIR_CONTENTS" ]; then
+#     echo "All tests passed, yet some left behind their test output:"
+#     for SUBDIR in $TEST_TMPDIR_CONTENTS; do
+#       echo $SUBDIR
+#     done
+#     EXIT_STATUS=1
+#   fi
+# fi
 
 if [ "$DO_COVERAGE" == "1" ]; then
   echo Generating coverage report...


Mime
View raw message