diff options
author | Patrick Steinhardt <psteinhardt@gitlab.com> | 2022-03-09 13:46:15 +0300 |
---|---|---|
committer | Patrick Steinhardt <psteinhardt@gitlab.com> | 2022-03-09 13:46:15 +0300 |
commit | ebb18ecc514b369fb0958dab95ba803a20c67cd4 (patch) | |
tree | 0777320f9c87ef60fdbe7b7640c44bbb5db4c28b | |
parent | 1ee9d9c2d3981592085407f5f3b05502fd2df105 (diff) | |
parent | ac24add628cfdd58b5e0fedb5fb31ab7a696b8d9 (diff) |
Merge branch 'jc-fix-cgroups-memory-race' into 'master'
cgroups: Remove paths field
See merge request gitlab-org/gitaly!4398
-rw-r--r-- | internal/cgroups/v1_linux.go | 4 | ||||
-rw-r--r-- | internal/cgroups/v1_linux_test.go | 2 |
2 files changed, 0 insertions, 6 deletions
diff --git a/internal/cgroups/v1_linux.go b/internal/cgroups/v1_linux.go index 56a710644..66a4de23a 100644 --- a/internal/cgroups/v1_linux.go +++ b/internal/cgroups/v1_linux.go @@ -18,7 +18,6 @@ import ( type CGroupV1Manager struct { cfg cgroupscfg.Config hierarchy func() ([]cgroups.Subsystem, error) - paths map[string]interface{} memoryFailedTotal, cpuUsage *prometheus.GaugeVec procs *prometheus.GaugeVec } @@ -29,7 +28,6 @@ func newV1Manager(cfg cgroupscfg.Config) *CGroupV1Manager { hierarchy: func() ([]cgroups.Subsystem, error) { return defaultSubsystems(cfg.Mountpoint) }, - paths: make(map[string]interface{}), memoryFailedTotal: prometheus.NewGaugeVec( prometheus.GaugeOpts{ Name: "gitaly_cgroup_memory_failed_total", @@ -102,8 +100,6 @@ func (cg *CGroupV1Manager) AddCommand(cmd *command.Command) error { return fmt.Errorf("failed adding process to cgroup: %w", err) } - cg.paths[cgroupPath] = struct{}{} - return nil } diff --git a/internal/cgroups/v1_linux_test.go b/internal/cgroups/v1_linux_test.go index e0236cefb..34defbf8e 100644 --- a/internal/cgroups/v1_linux_test.go +++ b/internal/cgroups/v1_linux_test.go @@ -67,7 +67,6 @@ func TestAddCommand(t *testing.T) { v1Manager1 := &CGroupV1Manager{ cfg: config, hierarchy: mock.hierarchy, - paths: make(map[string]interface{}), } require.NoError(t, v1Manager1.Setup()) ctx := testhelper.Context(t) @@ -80,7 +79,6 @@ func TestAddCommand(t *testing.T) { v1Manager2 := &CGroupV1Manager{ cfg: config, hierarchy: mock.hierarchy, - paths: make(map[string]interface{}), } require.NoError(t, v1Manager2.AddCommand(cmd2)) |