On Thu, 25 Aug 2022 15:32:08 GMT, Jonathan Dowland <[email protected]> wrote:
>> Jonathan Dowland has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> Address nit
>>
>> Thanks Thomas Stuefe
>
> test/hotspot/jtreg/containers/docker/TestMemoryAwareness.java line 53:
>
>> 51: DockerRunOptions opts = Common.newOpts(imageName);
>> 52: String goodMem = Common.run(opts).firstMatch("total physical
>> memory: (\\d+)", 1);
>> 53: assertNotNull(goodMem, "no match for 'total physical memory' in
>> trace output");
>
> I've just realised this will not match (yet) since the relevant log line is
> added in #9880 .
Yes. There is a feature to use dependent PRs, which could be used here. Just
target `pr/9880` instead of `master`.
-------------
PR: https://git.openjdk.org/jdk/pull/10017
