On Wed, 11 Dec 2024 15:24:12 GMT, Severin Gehwolf <sgehw...@openjdk.org> wrote:
>> Would you recommand also to include the paths in that warning? Something like >> ```cgroup controller path at '/sys/fs/cgroup' seems to have moved to >> '../../test', detected limits won't be accurate``` >> This way it will have all the necessary information to investigate customer >> cases. > > Seems fine yes. Updated the log message. The log example in cg v1: [0.001s][trace][os,container] OSContainer::init: Initializing Container Support [0.001s][debug][os,container] Detected optional pids controller entry in /proc/cgroups [0.001s][debug][os,container] Detected cgroups hybrid or legacy hierarchy, using cgroups v1 controllers [0.001s][warning][os,container] Cgroup memory controller path at '/sys/fs/cgroup/memory' seems to have moved to '/../../test', detected limits won't be accurate [0.001s][debug ][os,container] OSContainer::init: is_containerized() = true because all controllers are mounted read-only (container case) [0.001s][trace ][os,container] Path to /cpu.cfs_quota_us is /sys/fs/cgroup/cpu,cpuacct/cpu.cfs_quota_us [0.001s][trace ][os,container] CPU Quota is: -1 [0.002s][trace ][os,container] Path to /cpu.cfs_period_us is /sys/fs/cgroup/cpu,cpuacct/cpu.cfs_period_us [0.002s][trace ][os,container] CPU Period is: 100000 [0.002s][trace ][os,container] OSContainer::active_processor_count: 48 [0.002s][trace ][os,container] CgroupSubsystem::active_processor_count (cached): 48 [0.002s][trace ][os,container] total physical memory: 133623721984 [0.002s][trace ][os,container] Path to /memory.limit_in_bytes is /sys/fs/cgroup/memory/memory.limit_in_bytes [0.002s][trace ][os,container] Memory Limit is: 419430400 [0.004s][trace ][os,container] CgroupSubsystem::active_processor_count (cached): 48 [0.027s][trace ][os,container] Path to /cpu.cfs_quota_us is /sys/fs/cgroup/cpu,cpuacct/cpu.cfs_quota_us [0.027s][trace ][os,container] CPU Quota is: -1 [0.027s][trace ][os,container] Path to /cpu.cfs_period_us is /sys/fs/cgroup/cpu,cpuacct/cpu.cfs_period_us [0.027s][trace ][os,container] CPU Period is: 100000 [0.027s][trace ][os,container] OSContainer::active_processor_count: 48 openjdk version "24-internal" 2025-03-18 OpenJDK Runtime Environment (build 24-internal-adhoc.bellsoft.jdk) OpenJDK 64-Bit Server VM (build 24-internal-adhoc.bellsoft.jdk, mixed mode, sharing) ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/21808#discussion_r1881224417
