github/containerd
2
0
Fork 0
Code Issues Pull Requests Packages Projects Releases 2 Wiki Activity

cri: fix integration test on cgroupsv2 system

Browse Source 
TestUpdateContainerResources_MemoryLimit still uses old method to
check memlimit in cgroup.

Signed-off-by: Shengjing Zhu <zhsj@debian.org>
This commit is contained in:
Shengjing Zhu 2022-02-27 23:16:03 +08:00
parent 52471721fd
commit 775facd573
1 changed files with 4 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
integration/container_update_resources_test.go
View File

@ -260,11 +260,8 @@ func TestUpdateContainerResources_MemoryLimit(t *testing.T) {
require.NoError(t, err) require.NoError(t, err)
t.Log("Check memory limit in cgroup") t.Log("Check memory limit in cgroup")
cgroup, err := cgroups.Load(cgroups.V1, cgroups.PidPath(int(task.Pid()))) memLimit := getCgroupMemoryLimitForTask(t, task)
require.NoError(t, err) assert.Equal(t, uint64(400*1024*1024), memLimit)
stat, err := cgroup.Stat(cgroups.IgnoreNotExist)
require.NoError(t, err)
assert.Equal(t, uint64(400*1024*1024), stat.Memory.Usage.Limit)
t.Log("Update container memory limit after started") t.Log("Update container memory limit after started")
err = runtimeService.UpdateContainerResources(cn, &runtime.LinuxContainerResources{ err = runtimeService.UpdateContainerResources(cn, &runtime.LinuxContainerResources{
@ -278,7 +275,6 @@ func TestUpdateContainerResources_MemoryLimit(t *testing.T) {
checkMemoryLimit(t, spec, 800*1024*1024) checkMemoryLimit(t, spec, 800*1024*1024)
t.Log("Check memory limit in cgroup") t.Log("Check memory limit in cgroup")
stat, err = cgroup.Stat(cgroups.IgnoreNotExist) memLimit = getCgroupMemoryLimitForTask(t, task)
require.NoError(t, err) assert.Equal(t, uint64(800*1024*1024), memLimit)
assert.Equal(t, uint64(800*1024*1024), stat.Memory.Usage.Limit)
} }