Skip to content

Commit

Permalink
Fix TestGetContainerStateAfterUpdate on cgroup v2
Browse files Browse the repository at this point in the history
CI was failing on cgroup v2 because mockCgroupManager.GetUnifiedPath()
was returning an error.

Now the function returns the value of mockCgroupManager.unifiedPath,
but the value is currently not used in the tests.

Fix #2286

Signed-off-by: Akihiro Suda <[email protected]>
  • Loading branch information
AkihiroSuda committed Apr 3, 2020
1 parent 0c6659a commit 4540b59
Showing 1 changed file with 6 additions and 5 deletions.
11 changes: 6 additions & 5 deletions libcontainer/container_linux_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -15,10 +15,11 @@ import (
)

type mockCgroupManager struct {
pids []int
allPids []int
stats *cgroups.Stats
paths map[string]string
pids []int
allPids []int
stats *cgroups.Stats
paths map[string]string
unifiedPath string
}

type mockIntelRdtManager struct {
Expand Down Expand Up @@ -55,7 +56,7 @@ func (m *mockCgroupManager) GetPaths() map[string]string {
}

func (m *mockCgroupManager) GetUnifiedPath() (string, error) {
return "", fmt.Errorf("unimplemented")
return m.unifiedPath, nil
}

func (m *mockCgroupManager) Freeze(state configs.FreezerState) error {
Expand Down

0 comments on commit 4540b59

Please sign in to comment.