RFR: 8369441: Two container tests fail after JDK-8292984 [v2]
Severin Gehwolf
sgehwolf at openjdk.org
Thu Oct 9 16:36:35 UTC 2025
> Please review this simple test fix to the some container tests which currently fail. `TestJFREvents.java` fails because it was relying on a log line at the trace level in order to parse the total machine memory from it and use in the test. We should instead not rely on the log line but use the existing Whitebox API to query that info. `TestMemoryAwareness.java` fails because it matches a log line that was changed in JDK-8292984.
>
> Updating both tests fix the problems.
>
> **Testing:**
> - [x] GHA (not really useful for this as the touched tests don't run and it touches nothing else)
> - [x] Ran the affected container tests locally on Linux x86_64 (cg v2). Fail before this fix, pass after.
>
> Thanks,
> Severin
Severin Gehwolf has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains three additional commits since the last revision:
- Fix copyright
- Merge branch 'master' into jdk-8369441-test-fixes
- 8369441: Two container tests fail after JDK-8292984
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/27699/files
- new: https://git.openjdk.org/jdk/pull/27699/files/a828c243..e2659fcf
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=27699&range=01
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=27699&range=00-01
Stats: 6461 lines in 167 files changed: 4868 ins; 860 del; 733 mod
Patch: https://git.openjdk.org/jdk/pull/27699.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/27699/head:pull/27699
PR: https://git.openjdk.org/jdk/pull/27699
More information about the hotspot-dev
mailing list