From 970eaf003328cab69aa4ccaa78cf908f5a87fff0 Mon Sep 17 00:00:00 2001 From: baude Date: Mon, 30 Apr 2018 15:22:50 -0500 Subject: podman should assign a host port to -p when omitted If the user does not provide a host port when adding -p to create/run, podman should inject an available random port. podman run -p 80 .... podman should assign a random port to the host and expose the container port 80 to it Signed-off-by: baude Closes: #703 Approved by: rhatdan --- test/e2e/run_networking_test.go | 14 ++++++++++++++ 1 file changed, 14 insertions(+) (limited to 'test') diff --git a/test/e2e/run_networking_test.go b/test/e2e/run_networking_test.go index b56b11634..9a2858ca1 100644 --- a/test/e2e/run_networking_test.go +++ b/test/e2e/run_networking_test.go @@ -77,4 +77,18 @@ var _ = Describe("Podman rmi", func() { Expect(results.ExitCode()).To(Equal(0)) Expect(results.OutputToString()).To(ContainSubstring(": 80,")) }) + + It("podman run network expose duplicate host port results in error", func() { + session := podmanTest.Podman([]string{"run", "-dt", "-p", "80", ALPINE, "/bin/sh"}) + session.WaitWithDefaultTimeout() + Expect(session.ExitCode()).To(Equal(0)) + + inspect := podmanTest.Podman([]string{"inspect", "-l"}) + inspect.WaitWithDefaultTimeout() + Expect(inspect.ExitCode()).To(Equal(0)) + + containerConfig := inspect.InspectContainerToJSON() + Expect(containerConfig[0].NetworkSettings.Ports[0].HostPort).ToNot(Equal("80")) + }) + }) -- cgit v1.2.3-54-g00ecf