RFR: 8259446: runtime/jni/checked/TestCheckedReleaseArrayElements.java fails with stderr not empty [v2]

Daniel D.Daugherty dcubed at openjdk.java.net
Fri Jan 8 22:28:58 UTC 2021


On Fri, 8 Jan 2021 22:18:13 GMT, David Holmes <dholmes at openjdk.org> wrote:

>> The test checks stderr is empty.
>> The test is run in our tier3 CI testing.
>> Tier3 (and some other tiers) has a configuration that includes the -showversion flag which outputs the version string to stderr.
>> 
>> => ergo the test fails because stderr is not empty!
>> 
>> Fix: exec the secondary VM without including the jtreg test VM arguments.
>> 
>> I think this is really a bug in our test job definitions and we've been very lucky to not encounter problems with using `-showversion`, but as a quick fix this suffices, and we don't expect any other JVM flags to have any impact here.
>> 
>> Tested by running jtreg with/without -showversion being passed to JVM under test.
>> Tier3 test run in progress too.
>> Thanks.
>
> David Holmes has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Add comment as requested by Dan

Thumbs up! Also, this is a trivial change.

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

Marked as reviewed by dcubed (Reviewer).

PR: https://git.openjdk.java.net/jdk/pull/2001


More information about the hotspot-runtime-dev mailing list