Integrated: 7903030: jtreg should print stdout if JVM gathering properties fails

[Igor Ignatyev]

On Fri, 15 Oct 2021 17:34:31 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

This pull request has now been integrated.

Changeset: 832f368e
Author:    Igor Ignatyev <iignatyev at openjdk.org>
URL:       https://git.openjdk.java.net/jtreg/commit/832f368ee06cdd8e8d7e185a618972385c60d839
Stats:     5 lines in 1 file changed: 3 ins; 0 del; 2 mod

7903030: jtreg should print stdout if JVM gathering properties fails

Reviewed-by: jjg

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

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