[
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
