RFR: 8292541: [Metrics] Reported memory limit may exceed physical machine memory [v3]
Jonathan Dowland
jdowland at openjdk.org
Thu Aug 25 16:15:54 UTC 2022
On Thu, 25 Aug 2022 16:09:01 GMT, Severin Gehwolf <sgehwolf at openjdk.org> wrote:
>> 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`.
I've pushed a small excerpt of the changes from #9880 in 1bf3aa341fd93079289f11e620938e6bed40b382 to this PR in order to add the log lines that the tests depend on.
-------------
PR: https://git.openjdk.org/jdk/pull/10017
More information about the core-libs-dev
mailing list