On Wed, 20 Sep 2023 05:32:07 GMT, David Holmes <dhol...@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
