(kudu) branch branch-1.18.x updated: [build-support-test] remove TEST_TMPDIR with its content

[achennaka]

This is an automated email from the ASF dual-hosted git repository.

achennaka pushed a commit to branch branch-1.18.x
in repository https://gitbox.apache.org/repos/asf/kudu.git


The following commit(s) were added to refs/heads/branch-1.18.x by this push:
     new d8fc6d75f [build-support-test] remove TEST_TMPDIR with its content
d8fc6d75f is described below

commit d8fc6d75f34a89633ba96c20fe1bfed4d10b0f2f
Author: Alexey Serbin <ale...@apache.org>
AuthorDate: Mon Dec 2 12:07:35 2024 -0800

    [build-support-test] remove TEST_TMPDIR with its content
    
    In case of concurrent build/test tasks, it makes sense to create
    per-task TEST_TMPDIR directory because of the logic in the
    jenkins/build-and-test.sh script that introspects its content.
    When doing so, it make sense to remove the directory along with its
    content, not just the content under the directory.  Otherwise,
    an empty TEST_TMPDIR directory would be left behind.  It's not an
    issue when TEST_TMPDIR is the same for every build/task, but with
    custom, per-build TEST_TMPDIR, there would be too many empty directories
    left behind.
    
    With this patch, the script is amended to address that: as a part
    of clean-up, it now removes the TEST_TMPDIR directory along with its
    content, not just the content of the directory.
    
    Change-Id: I1d79f9482575fecba2e81c2d5fe85cbe25afd6b5
    Reviewed-on: http://gerrit.cloudera.org:8080/22150
    Reviewed-by: Abhishek Chennaka <achenn...@cloudera.com>
    Tested-by: Alexey Serbin <ale...@apache.org>
    (cherry picked from commit 8027299167f6f4f0fe586b6c70829054aae31dab)
    Reviewed-on: http://gerrit.cloudera.org:8080/22152
---
 build-support/jenkins/build-and-test.sh | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/build-support/jenkins/build-and-test.sh 
b/build-support/jenkins/build-and-test.sh
index d1f175349..eb923d467 100755
--- a/build-support/jenkins/build-and-test.sh
+++ b/build-support/jenkins/build-and-test.sh
@@ -482,7 +482,7 @@ fi
 
 # our tests leave lots of data lying around, clean up before we run
 if [ -d "$TEST_TMPDIR" ]; then
-  rm -Rf $TEST_TMPDIR/*
+  rm -Rf $TEST_TMPDIR
 fi
 
 # actually do the build