RFR: 8370966: Create regression test for the hierarchical memory limit fix in JDK-8370572
Severin Gehwolf
sgehwolf at openjdk.org
Fri Nov 7 17:43:18 UTC 2025
Please review this test-only enhancement. It creates a regression test for the Amazon ECS setup on cgroups v1 where the parent memory limit isn't visible inside the container and, thus, needs to rely on the cg v1 specific `hierarchical_memory_limit` token in `memory.stat`. The proposed test is cg v1 only and needs to be run as root. It's skipped otherwise. It's useful to have when working on refactorings like #27743 so as not to regress.
The other changes are an effort to reduce code duplication in the test code where similar patterns have been used in other container tests.
Testing (all on Linux x86_64):
- [x] CG version 2, run as root. Engine: docker. Test is skipped.
- [x] CG version 1, run as root. Engine: docker. Test passes and fails without the product fix of [JDK-8370572](https://bugs.openjdk.org/browse/JDK-8370572)
- [x] CG version 1, run as root. Engine: podman. Test passes and fails without the product fix of [JDK-8370572](https://bugs.openjdk.org/browse/JDK-8370572)
- [X] CG version 1, run as non-root. Test skipped.
- [x] GHA, though I don't think this is very useful for this change.
Thoughts?
-------------
Commit messages:
- 8370966: Create regression test for hierarchical memory limit fix in JDK-8370572
Changes: https://git.openjdk.org/jdk/pull/28201/files
Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=28201&range=00
Issue: https://bugs.openjdk.org/browse/JDK-8370966
Stats: 191 lines in 8 files changed: 154 ins; 32 del; 5 mod
Patch: https://git.openjdk.org/jdk/pull/28201.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/28201/head:pull/28201
PR: https://git.openjdk.org/jdk/pull/28201
More information about the hotspot-dev
mailing list