Todd Lipcon has submitted this change and it was merged. Change subject: dist-test: set TEST_TMPDIR inside test directory ......................................................................
dist-test: set TEST_TMPDIR inside test directory Recently I hit a case where I had a test which was crashing with an ASAN warning halfway through, after writing a bunch of data into the test tmpdir. Because we weren't setting TEST_TMPDIR in the dist-test environment, the leftover data ended up orphaned in /tmp/kudutest-1000, which wasn't cleaned up after the dist-test task ended. This just sets the tmpdir to be inside the root working directory of the test, so that we are guaranteed to clean it up after the test finishes. To verify, I submitted a job and looked at the resulting test logs, and saw that it was now writing within the test directory instead of within /tmp/ Change-Id: I0497591860f7e8b1ea18c1ee9b05becb282f6ec8 Reviewed-on: http://gerrit.cloudera.org:8080/5203 Reviewed-by: Adar Dembo <[email protected]> Tested-by: Kudu Jenkins --- M build-support/run_dist_test.py 1 file changed, 5 insertions(+), 0 deletions(-) Approvals: Adar Dembo: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/5203 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0497591860f7e8b1ea18c1ee9b05becb282f6ec8 Gerrit-PatchSet: 5 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Todd Lipcon <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Mike Percy <[email protected]> Gerrit-Reviewer: Todd Lipcon <[email protected]>
