On Thu, 25 Aug 2022 15:32:08 GMT, Jonathan Dowland <jdowl...@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