RFR: 8292541: [Metrics] Reported memory limit may exceed physical machine memory [v3]

[Severin Gehwolf]

On Thu, 25 Aug 2022 15:32:08 GMT, Jonathan Dowland <jdowland at openjdk.org> 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