mirror of
https://github.com/opencontainers/runc.git
synced 2026-07-12 09:46:25 +08:00
Kir Kolyshkin
3f65946756
A cgroup manager's Set method sets cgroup resources, but historically it was accepting configs.Cgroups. Refactor it to accept resources only. This is an improvement from the API point of view, as the method can not change cgroup configuration (such as path to the cgroup etc), it can only set (modify) its resources/limits. This also lays the foundation for complicated resource updates, as now Set has two sets of resources -- the one that was previously specified during cgroup manager creation (or the previous Set), and the one passed in the argument, so it could deduce the difference between these. This is a long term goal though. Signed-off-by: Kir Kolyshkin <kolyshkin@gmail.com>
42 lines
1.0 KiB
Go
42 lines
1.0 KiB
Go
// +build linux
|
|
|
|
package fs
|
|
|
|
import (
|
|
"strconv"
|
|
"testing"
|
|
|
|
"github.com/opencontainers/runc/libcontainer/cgroups/fscommon"
|
|
)
|
|
|
|
const (
|
|
classidBefore = 0x100002
|
|
classidAfter = 0x100001
|
|
)
|
|
|
|
func TestNetClsSetClassid(t *testing.T) {
|
|
helper := NewCgroupTestUtil("net_cls", t)
|
|
defer helper.cleanup()
|
|
|
|
helper.writeFileContents(map[string]string{
|
|
"net_cls.classid": strconv.FormatUint(classidBefore, 10),
|
|
})
|
|
|
|
helper.CgroupData.config.Resources.NetClsClassid = classidAfter
|
|
netcls := &NetClsGroup{}
|
|
if err := netcls.Set(helper.CgroupPath, helper.CgroupData.config.Resources); err != nil {
|
|
t.Fatal(err)
|
|
}
|
|
|
|
// As we are in mock environment, we can't get correct value of classid from
|
|
// net_cls.classid.
|
|
// So. we just judge if we successfully write classid into file
|
|
value, err := fscommon.GetCgroupParamUint(helper.CgroupPath, "net_cls.classid")
|
|
if err != nil {
|
|
t.Fatalf("Failed to parse net_cls.classid - %s", err)
|
|
}
|
|
if value != classidAfter {
|
|
t.Fatal("Got the wrong value, set net_cls.classid failed.")
|
|
}
|
|
}
|