mirror of https://github.com/containers/podman.git
test: drop pod-create --device-read-bps test
the test is not doing what it believes to do. The containers are not supposed to be joining the infra container cgroup. In addition, the result is validated only on cgroup v1 systems (that are not used in the CI). We may want to add it back, or a variant of it, once the --device-read-bps option applies to the pod parent cgroup. Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
This commit is contained in:
parent
73cdc925f0
commit
8106568ca2
|
@ -899,27 +899,6 @@ ENTRYPOINT ["sleep","99999"]
|
||||||
|
|
||||||
})
|
})
|
||||||
|
|
||||||
It("podman pod create --device-read-bps", func() {
|
|
||||||
SkipIfRootless("Cannot create devices in /dev in rootless mode")
|
|
||||||
SkipIfRootlessCgroupsV1("Setting device-read-bps not supported on cgroupv1 for rootless users")
|
|
||||||
|
|
||||||
podName := "testPod"
|
|
||||||
session := podmanTest.Podman([]string{"pod", "create", "--device-read-bps", "/dev/zero:1mb", "--name", podName})
|
|
||||||
session.WaitWithDefaultTimeout()
|
|
||||||
Expect(session).Should(Exit(0))
|
|
||||||
|
|
||||||
if CGROUPSV2 {
|
|
||||||
session = podmanTest.Podman([]string{"run", "--rm", "--pod", podName, ALPINE, "sh", "-c", "cat /sys/fs/cgroup/$(sed -e 's|0::||' < /proc/self/cgroup)/io.max"})
|
|
||||||
} else {
|
|
||||||
session = podmanTest.Podman([]string{"run", "--rm", "--pod", podName, ALPINE, "cat", "/sys/fs/cgroup/blkio/blkio.throttle.read_bps_device"})
|
|
||||||
}
|
|
||||||
session.WaitWithDefaultTimeout()
|
|
||||||
Expect(session).Should(Exit(0))
|
|
||||||
if !CGROUPSV2 {
|
|
||||||
Expect(session.OutputToString()).To(ContainSubstring("1048576"))
|
|
||||||
}
|
|
||||||
})
|
|
||||||
|
|
||||||
It("podman pod create --volumes-from", func() {
|
It("podman pod create --volumes-from", func() {
|
||||||
volName := "testVol"
|
volName := "testVol"
|
||||||
volCreate := podmanTest.Podman([]string{"volume", "create", volName})
|
volCreate := podmanTest.Podman([]string{"volume", "create", volName})
|
||||||
|
|
Loading…
Reference in New Issue