b230fb8ac4
The slice of strings more precisely captures the hierarchic nature of
the cgroup paths we use to represent pods and their groupings.
It also ensures we're reducing the chances of passing an incorrect path
format to a cgroup driver that requires a different path naming, since
now explicit conversions are always needed.
The new constructor NewCgroupName starts from an existing CgroupName,
which enforces a hierarchy where a root is always needed. It also
performs checking on the component names to ensure invalid characters
("/" and "_") are not in use.
A RootCgroupName for the top of the cgroup hierarchy tree is introduced.
This refactor results in a net reduction of around 30 lines of code,
mainly with the demise of ConvertCgroupNameToSystemd which had fairly
complicated logic in it and was doing just too many things.
There's a small TODO in a helper updateSystemdCgroupInfo that was
introduced to make this commit possible. That logic really belongs in
libcontainer, I'm planning to send a PR there to include it there.
(The API already takes a field with that information, only that field is
only processed in cgroupfs and not systemd driver, we should fix that.)
Tested by running the e2e-node tests on both Ubuntu 16.04 (with cgroupfs
driver) and CentOS 7 (with systemd driver.)
98 lines
2.3 KiB
Go
98 lines
2.3 KiB
Go
// +build !linux
|
|
|
|
/*
|
|
Copyright 2016 The Kubernetes Authors.
|
|
|
|
Licensed under the Apache License, Version 2.0 (the "License");
|
|
you may not use this file except in compliance with the License.
|
|
You may obtain a copy of the License at
|
|
|
|
http://www.apache.org/licenses/LICENSE-2.0
|
|
|
|
Unless required by applicable law or agreed to in writing, software
|
|
distributed under the License is distributed on an "AS IS" BASIS,
|
|
WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
|
|
See the License for the specific language governing permissions and
|
|
limitations under the License.
|
|
*/
|
|
|
|
package cm
|
|
|
|
import "fmt"
|
|
|
|
type unsupportedCgroupManager struct{}
|
|
|
|
// Make sure that unsupportedCgroupManager implements the CgroupManager interface
|
|
var _ CgroupManager = &unsupportedCgroupManager{}
|
|
|
|
type CgroupSubsystems struct {
|
|
Mounts []interface{}
|
|
MountPoints map[string]string
|
|
}
|
|
|
|
func NewCgroupManager(_ interface{}) CgroupManager {
|
|
return &unsupportedCgroupManager{}
|
|
}
|
|
|
|
func (m *unsupportedCgroupManager) Name(_ CgroupName) string {
|
|
return ""
|
|
}
|
|
|
|
func (m *unsupportedCgroupManager) Exists(_ CgroupName) bool {
|
|
return false
|
|
}
|
|
|
|
func (m *unsupportedCgroupManager) Destroy(_ *CgroupConfig) error {
|
|
return nil
|
|
}
|
|
|
|
func (m *unsupportedCgroupManager) Update(_ *CgroupConfig) error {
|
|
return nil
|
|
}
|
|
|
|
func (m *unsupportedCgroupManager) Create(_ *CgroupConfig) error {
|
|
return fmt.Errorf("Cgroup Manager is not supported in this build")
|
|
}
|
|
|
|
func (m *unsupportedCgroupManager) GetResourceStats(name CgroupName) (*ResourceStats, error) {
|
|
return nil, fmt.Errorf("Cgroup Manager is not supported in this build")
|
|
}
|
|
|
|
func (m *unsupportedCgroupManager) Pids(_ CgroupName) []int {
|
|
return nil
|
|
}
|
|
|
|
func (m *unsupportedCgroupManager) CgroupName(name string) CgroupName {
|
|
return CgroupName([]string{})
|
|
}
|
|
|
|
func (m *unsupportedCgroupManager) ReduceCPULimits(cgroupName CgroupName) error {
|
|
return nil
|
|
}
|
|
|
|
var RootCgroupName = CgroupName([]string{})
|
|
|
|
func NewCgroupName(base CgroupName, components ...string) CgroupName {
|
|
return CgroupName(append(base, components...))
|
|
}
|
|
|
|
func (cgroupName CgroupName) ToSystemd() string {
|
|
return ""
|
|
}
|
|
|
|
func ParseSystemdToCgroupName(name string) CgroupName {
|
|
return nil
|
|
}
|
|
|
|
func (cgroupName CgroupName) ToCgroupfs() string {
|
|
return ""
|
|
}
|
|
|
|
func ParseCgroupfsToCgroupName(name string) CgroupName {
|
|
return nil
|
|
}
|
|
|
|
func IsSystemdStyleName(name string) bool {
|
|
return false
|
|
}
|