RFR: 8316562: serviceability/sa/jmap-hprof/JMapHProfLargeHeapTest.java times out after JDK-8314829

Leonid Mesnik lmesnik at openjdk.org
Wed Sep 20 05:53:41 UTC 2023


On Wed, 20 Sep 2023 05:32:07 GMT, David Holmes <dholmes at openjdk.org> wrote:

>> Test start failing because of unexpected VM logging.
>> The fix is to search expected string ignoring any unknown output from forked process.
>> Tested with tier1 and by running test with various options used in CI.
>
> test/hotspot/jtreg/serviceability/sa/jmap-hprof/JMapHProfLargeHeapTest.java line 95:
> 
>> 93:                 } else {
>> 94:                     System.out.println("Ignoring: " + pidstring);
>> 95:                 }
> 
> Do we really want this in the test output?

It might be helpful to ensure that we don't mess with matching PID line if test fails.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/15833#discussion_r1331032496


More information about the serviceability-dev mailing list