sandbox: cgroup: prefix cgroup related methods

rename to allow group in auto-generated docs.

Signed-off-by: Jose Carlos Venegas Munoz <jose.carlos.venegas.munoz@intel.com>
This commit is contained in:
Jose Carlos Venegas Munoz 2019-08-15 17:31:19 -05:00
parent 529ec25fb7
commit caac68c09f
2 changed files with 12 additions and 12 deletions

View File

@ -133,12 +133,12 @@ func TestUpdateCgroups(t *testing.T) {
} }
// empty path // empty path
err := s.updateCgroups() err := s.cgroupsUpdate()
assert.NoError(err) assert.NoError(err)
// path doesn't exist // path doesn't exist
s.state.CgroupPath = "/abc/123/rgb" s.state.CgroupPath = "/abc/123/rgb"
err = s.updateCgroups() err = s.cgroupsUpdate()
assert.Error(err) assert.Error(err)
if os.Getuid() != 0 { if os.Getuid() != 0 {
@ -152,7 +152,7 @@ func TestUpdateCgroups(t *testing.T) {
s.hypervisor = &mockHypervisor{mockPid: 0} s.hypervisor = &mockHypervisor{mockPid: 0}
// bad pid // bad pid
err = s.updateCgroups() err = s.cgroupsUpdate()
assert.Error(err) assert.Error(err)
// fake workload // fake workload
@ -161,7 +161,7 @@ func TestUpdateCgroups(t *testing.T) {
s.hypervisor = &mockHypervisor{mockPid: cmd.Process.Pid} s.hypervisor = &mockHypervisor{mockPid: cmd.Process.Pid}
// no containers // no containers
err = s.updateCgroups() err = s.cgroupsUpdate()
assert.NoError(err) assert.NoError(err)
s.config = &SandboxConfig{} s.config = &SandboxConfig{}
@ -186,11 +186,11 @@ func TestUpdateCgroups(t *testing.T) {
}, },
} }
err = s.updateCgroups() err = s.cgroupsUpdate()
assert.NoError(err) assert.NoError(err)
// cleanup // cleanup
assert.NoError(cmd.Process.Kill()) assert.NoError(cmd.Process.Kill())
err = s.deleteCgroups() err = s.cgroupsDelete()
assert.NoError(err) assert.NoError(err)
} }

View File

@ -758,7 +758,7 @@ func (s *Sandbox) Delete() error {
} }
} }
if err := s.deleteCgroups(); err != nil { if err := s.cgroupsDelete(); err != nil {
return err return err
} }
@ -1148,7 +1148,7 @@ func (s *Sandbox) CreateContainer(contConfig ContainerConfig) (VCContainer, erro
return nil, err return nil, err
} }
if err = s.updateCgroups(); err != nil { if err = s.cgroupsUpdate(); err != nil {
return nil, err return nil, err
} }
@ -1325,7 +1325,7 @@ func (s *Sandbox) UpdateContainer(containerID string, resources specs.LinuxResou
return err return err
} }
if err := s.updateCgroups(); err != nil { if err := s.cgroupsUpdate(); err != nil {
return err return err
} }
@ -1416,7 +1416,7 @@ func (s *Sandbox) createContainers() error {
} }
} }
if err := s.updateCgroups(); err != nil { if err := s.cgroupsUpdate(); err != nil {
return err return err
} }
if err := s.storeSandbox(); err != nil { if err := s.storeSandbox(); err != nil {
@ -1876,7 +1876,7 @@ func (s *Sandbox) GetHypervisorType() string {
return string(s.config.HypervisorType) return string(s.config.HypervisorType)
} }
func (s *Sandbox) updateCgroups() error { func (s *Sandbox) cgroupsUpdate() error {
if s.state.CgroupPath == "" { if s.state.CgroupPath == "" {
s.Logger().Warn("sandbox's cgroup won't be updated: cgroup path is empty") s.Logger().Warn("sandbox's cgroup won't be updated: cgroup path is empty")
return nil return nil
@ -1908,7 +1908,7 @@ func (s *Sandbox) updateCgroups() error {
return nil return nil
} }
func (s *Sandbox) deleteCgroups() error { func (s *Sandbox) cgroupsDelete() error {
s.Logger().Debug("Deleting sandbox cgroup") s.Logger().Debug("Deleting sandbox cgroup")
path := cgroupNoConstraintsPath(s.state.CgroupPath) path := cgroupNoConstraintsPath(s.state.CgroupPath)