[ https://issues.apache.org/jira/browse/CASSANDRA-15777?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17096521#comment-17096521 ]
Berenguer Blasi commented on CASSANDRA-15777: --------------------------------------------- I did take a cursory look at the changes and I am only wondering if cassandra-test.sh could benefit from havingĀ {{TPMDIR}} variable just like the other scripts, instead of having it hardcoded? > All test targets to set jvm arg "java.io.tmpdir" according to ${tmp.dir} > ------------------------------------------------------------------------ > > Key: CASSANDRA-15777 > URL: https://issues.apache.org/jira/browse/CASSANDRA-15777 > Project: Cassandra > Issue Type: Bug > Components: Build, Test/unit > Reporter: Michael Semb Wever > Assignee: Michael Semb Wever > Priority: Normal > Fix For: 2.2.x, 3.0.x, 3.11.x, 4.0-alpha > > > Make all test targets declare the temp directory (java.io.tmpdir) as defined > by tmp.dir > This was originally done in CASSANDRA-7712 but has been eroded over time as > the test targets evolved. > The attached patch moves the > {code}<jvmarg value="-Djava.io.tmpdir=${tmp.dir}"/>{code} > into the {{testmacrohelper}} macrodef. > Like CASSANDRA-7712, jenkins agents are filling up their root volumes because > of what's left behind under {{"/tmp"}} > The build scripts have already been configured to define the temp directory > to be within the jenkins job's workspace: > https://github.com/apache/cassandra-builds/commit/33ba1e30ea196180f7d70f8e6ec47fdf6844f3f6#diff-91876f5f158ec50dab9a70cc06c06922 -- This message was sent by Atlassian Jira (v8.3.4#803005) --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org For additional commands, e-mail: commits-h...@cassandra.apache.org