Merge pull request #8231 from baude/fedorarootlesscpulimit

fedora rootless cpu settings
This commit is contained in:
OpenShift Merge Robot
2020-11-03 12:21:08 -05:00
committed by GitHub
2 changed files with 4 additions and 6 deletions

View File

@ -613,12 +613,10 @@ func SkipIfRootlessCgroupsV1(reason string) {
} }
} }
func SkipIfUnprevilegedCPULimits() { func SkipIfUnprivilegedCPULimits() {
info := GetHostDistributionInfo() info := GetHostDistributionInfo()
if isRootless() && if isRootless() && info.Distribution == "fedora" {
info.Distribution == "fedora" && ginkgo.Skip("Rootless Fedora doesn't have permission to set CPU limits")
(info.Version == "31" || info.Version == "32") {
ginkgo.Skip("Rootless Fedora doesn't have permission to set CPU limits before version 33")
} }
} }

View File

@ -1406,7 +1406,7 @@ spec:
It("podman play kube allows setting resource limits", func() { It("podman play kube allows setting resource limits", func() {
SkipIfContainerized("Resource limits require a running systemd") SkipIfContainerized("Resource limits require a running systemd")
SkipIfRootlessCgroupsV1("Limits require root or cgroups v2") SkipIfRootlessCgroupsV1("Limits require root or cgroups v2")
SkipIfUnprevilegedCPULimits() SkipIfUnprivilegedCPULimits()
podmanTest.CgroupManager = "systemd" podmanTest.CgroupManager = "systemd"
var ( var (