You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cassandra.apache.org by "Michael Semb Wever (Jira)" <ji...@apache.org> on 2020/04/30 14:08:00 UTC

[jira] [Comment Edited] (CASSANDRA-15777) All test targets to set jvm arg "java.io.tmpdir" according to ${tmp.dir}

    [ https://issues.apache.org/jira/browse/CASSANDRA-15777?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17096565#comment-17096565 ] 

Michael Semb Wever edited comment on CASSANDRA-15777 at 4/30/20, 2:07 PM:
--------------------------------------------------------------------------

[~Bereng], if by "having TPMDIR variable just like the other scripts" you mean [this|https://github.com/apache/cassandra-builds/blob/master/build-scripts/cassandra-dtest-pytest.sh#L23], then that's for python processes. Anything java needs the {{-Djava.io.tmpdir=…}} argument (which can be specified as as {{-Dtmp.dir=…}} in our ant build script).


was (Author: michaelsembwever):
[~Bereng], if by "having TPMDIR variable just like the other scripts" you mean [this|https://github.com/apache/cassandra-builds/blob/master/build-scripts/cassandra-dtest-pytest.sh#L23], then that's for python processes. Anything java needs the {{-Djava.io.tmpdir=…}} argument.

> 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-unsubscribe@cassandra.apache.org
For additional commands, e-mail: commits-help@cassandra.apache.org