diff options
| author | Charlie Doern <cdoern@redhat.com> | 2022-07-25 10:21:14 -0400 |
|---|---|---|
| committer | Matthew Heon <matthew.heon@pm.me> | 2022-07-26 14:46:12 -0400 |
| commit | c85722eb9f552f2456d33589a1417264369a6366 (patch) | |
| tree | 2e055a356eadd8c992766cdd4168ebb0293c9719 /test | |
| parent | c3e0f8ebef13e107b179fb6cf867ca8902f3761a (diff) | |
| download | podman-c85722eb9f552f2456d33589a1417264369a6366.tar.gz podman-c85722eb9f552f2456d33589a1417264369a6366.tar.bz2 podman-c85722eb9f552f2456d33589a1417264369a6366.zip | |
pod create --share none should not create infra
for podman pod create, when we are not sharing any namespaces there is no point for the infra container.
This is especially true since resources have also been decoupled from the container recently.
handle this on the cmd level so that we can still create infra if set explicitly
resolves #15048
Signed-off-by: Charlie Doern <cdoern@redhat.com>
Diffstat (limited to 'test')
| -rw-r--r-- | test/e2e/pod_infra_container_test.go | 16 |
1 files changed, 16 insertions, 0 deletions
diff --git a/test/e2e/pod_infra_container_test.go b/test/e2e/pod_infra_container_test.go index a2e090524..b53630156 100644 --- a/test/e2e/pod_infra_container_test.go +++ b/test/e2e/pod_infra_container_test.go @@ -435,4 +435,20 @@ var _ = Describe("Podman pod create", func() { Expect(session).Should(Exit(0)) Expect(session.OutputToString()).To(ContainSubstring(hostname)) }) + + tests := []string{"", "none"} + for _, test := range tests { + test := test + It("podman pod create --share="+test+" should not create an infra ctr", func() { + session := podmanTest.Podman([]string{"pod", "create", "--share", test}) + session.WaitWithDefaultTimeout() + Expect(session).Should(Exit(0)) + + session = podmanTest.Podman([]string{"pod", "inspect", "--format", "{{.NumContainers}}", session.OutputToString()}) + session.WaitWithDefaultTimeout() + Expect(session).Should(Exit(0)) + Expect(session.OutputToString()).Should((Equal("0"))) + }) + } + }) |