> Please review this change to allow XX configuration for specifying a 
> different /tmp directory for the JVM to use.  In some container environments, 
> /tmp and /proc/pid/root/tmp might not be usable and an alternate would be 
> used.  This requires a release note and CSR.   Usage is:
> 
> java -XX:AltTempDir=/diags <app>
> jps -J-XX:AltTempDir=/diags
> jcmd -J-XX:AltTempDir=/diags <pid> <cmds>
> 
> Tested with a couple of tests and locally, and ran tier1-4.
> 
> ---------
> - [x] I confirm that I make this contribution in accordance with the [OpenJDK 
> Interim AI Policy](https://openjdk.org/legal/ai).

Coleen Phillimore has updated the pull request incrementally with one 
additional commit since the last revision:

  With @kevinjwalls rewrite for getLocalVmId that ignores temp directories.  
Rewrite tests to actually delete temp files. Fix VirtualMachineImpl to get 
"unable to access target filesystem" failure and not bypass the same filesystem 
check.

-------------

Changes:
  - all: https://git.openjdk.org/jdk/pull/31407/files
  - new: https://git.openjdk.org/jdk/pull/31407/files/a9ebec36..82bab15a

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=31407&range=18
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=31407&range=17-18

  Stats: 118 lines in 4 files changed: 56 ins; 39 del; 23 mod
  Patch: https://git.openjdk.org/jdk/pull/31407.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/31407/head:pull/31407

PR: https://git.openjdk.org/jdk/pull/31407

Reply via email to