RFR: 7903030: jtreg should print stdout if JVM gathering properties fails [v2]

Igor Ignatyev iignatyev at openjdk.java.net
Fri Oct 15 18:45:18 UTC 2021


> Hi all,
> 
> could you please review this patch?
> from JBS:
>> currently, if the JVM, which was created to gather (at-requires) properties, fails for any reason, jtreg prints out "failed to get JDK properties for" message together w/ the exit code. and although jtreg always prints out stderr of this JVM, it might not contain enough information to understand the failure and has caused confusion in past.
> 
> testing: run `jtreg -vmoptions:"-Xmx1g -Xms2g"`:
> 
> Error occurred during initialization of VM
> Initial heap size set to a larger value than the maximum heap size
> failed to get JDK properties for /Library/Java/JavaVirtualMachines/jdk-14.jdk/Contents/Home/bin/java -Xmx1g -Xms2g; exit code 1
> Error: failed to get JDK properties for /Library/Java/JavaVirtualMachines/jdk-14.jdk/Contents/Home/bin/java -Xmx1g -Xms2g; exit code 1
> 
> vs old behavior:
> 
> failed to get JDK properties for /Library/Java/JavaVirtualMachines/jdk-14.jdk/Contents/Home/bin/java -Xmx1g -Xms2g; exit code 1
> Error: failed to get JDK properties for /Library/Java/JavaVirtualMachines/jdk-14.jdk/Contents/Home/bin/java -Xmx1g -Xms2g; exit code 1
> 
> 
> Thanks,
> -- Igor

Igor Ignatyev has updated the pull request incrementally with one additional commit since the last revision:

  print out the whole command line as well as current working diretory

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

Changes:
  - all: https://git.openjdk.java.net/jtreg/pull/28/files
  - new: https://git.openjdk.java.net/jtreg/pull/28/files/cc2088c2..2655ca49

Webrevs:
 - full: https://webrevs.openjdk.java.net/?repo=jtreg&pr=28&range=01
 - incr: https://webrevs.openjdk.java.net/?repo=jtreg&pr=28&range=00-01

  Stats: 2 lines in 1 file changed: 0 ins; 0 del; 2 mod
  Patch: https://git.openjdk.java.net/jtreg/pull/28.diff
  Fetch: git fetch https://git.openjdk.java.net/jtreg pull/28/head:pull/28

PR: https://git.openjdk.java.net/jtreg/pull/28


More information about the jtreg-dev mailing list