On Fri, 30 Aug 2024 11:05:24 GMT, Matthias Baesken <mbaes...@openjdk.org> wrote:
>> 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 10 additional >> commits since the last revision: >> >> - Add root check for SystemdMemoryAwarenessTest.java >> - Merge branch 'master' into jdk-8333446-systemd-slice-tests >> - Merge branch 'master' into jdk-8333446-systemd-slice-tests >> - Merge branch 'master' into jdk-8333446-systemd-slice-tests >> - Add Whitebox check for host cpu >> - Merge branch 'master' into jdk-8333446-systemd-slice-tests >> - Merge branch 'master' into jdk-8333446-systemd-slice-tests >> - Merge branch 'master' into jdk-8333446-systemd-slice-tests >> - Fix comments >> - 8333446: Add tests for hierarchical container support > > Looking through the coding it looks more or less okay to me; but if you > really need to run it under user 'root' I think we will not have so much use > for this in our test environments because we use other test users. > Not saying that this is a very bad thing, maybe it is just the way it is, > that 'root' is needed ? @MBaesken Any more thoughts on this? ------------- PR Comment: https://git.openjdk.org/jdk/pull/19530#issuecomment-2333790472
