Skip to content

Commit

Permalink
Revert "[memorylimiter] Fix cgroups parsing (open-telemetry#6825)" (o…
Browse files Browse the repository at this point in the history
…pen-telemetry#7142)

This reverts commit ede9e0a.
  • Loading branch information
bogdandrutu authored Feb 6, 2023
1 parent d587e1d commit f7af599
Show file tree
Hide file tree
Showing 3 changed files with 1 addition and 21 deletions.
16 changes: 0 additions & 16 deletions .chloggen/fix-memory-limiter.yaml

This file was deleted.

4 changes: 1 addition & 3 deletions internal/cgroups/cgroups.go
Original file line number Diff line number Diff line change
Expand Up @@ -104,9 +104,7 @@ func NewCGroups(procPathMountInfo, procPathCGroup string) (CGroups, error) {
if err != nil {
return err
}
if strings.HasPrefix(cgroupPath, "/sys") {
cgroups[opt] = NewCGroup(cgroupPath)
}
cgroups[opt] = NewCGroup(cgroupPath)
}

return nil
Expand Down
2 changes: 0 additions & 2 deletions internal/cgroups/testdata/proc/cgroups/mountinfo
Original file line number Diff line number Diff line change
Expand Up @@ -6,5 +6,3 @@
6 5 0:5 / /sys/fs/cgroup/cpuset rw,nosuid,nodev,noexec,relatime shared:6 - cgroup cgroup rw,cpuset
7 5 0:6 /docker /sys/fs/cgroup/cpu,cpuacct rw,nosuid,nodev,noexec,relatime shared:7 - cgroup cgroup rw,cpu,cpuacct
8 5 0:7 /docker /sys/fs/cgroup/memory rw,nosuid,nodev,noexec,relatime shared:8 - cgroup cgroup rw,memory
9 1 0:8 / /var/lib/docker/overlay2/9054a95f2cf7296867089e1bd37931742a17eb3308a795d51adb2654ee2276df/merged/sys/fs/cgroup ro,nosuid,nodev,noexec,relatime - tmpfs tmpfs rw,mode=755
10 9 0:9 /docker /var/lib/docker/overlay2/9054a95f2cf7296867089e1bd37931742a17eb3308a795d51adb2654ee2276df/merged/sys/fs/cgroup/memory ro,nosuid,nodev,noexec,relatime master:17 - cgroup cgroup rw,memory

0 comments on commit f7af599

Please sign in to comment.