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

Igor Ignatyev iignatyev at openjdk.java.net
Fri Oct 15 19:05:08 UTC 2021


On Fri, 15 Oct 2021 18:45:18 GMT, Igor Ignatyev <iignatyev at openjdk.org> wrote:

>> 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

Thank you Jon!

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

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


More information about the jtreg-dev mailing list