Merge pull request #10657 from schrej/fix/capd-dont-expose-host-ports

k8s-ci-robot · web-flow · commit 8e8650ca6dc5 · 2024-05-23T06:14:57.000-07:00
🐛 capd: don't add host ports to exposed ports of containers
diff --git a/test/infrastructure/container/docker.go b/test/infrastructure/container/docker.go
@@ -727,7 +727,6 @@ func configurePortMappings(portMappings []PortMapping, config *dockercontainer.C
 		}
 		hostConfig.PortBindings[port] = append(hostConfig.PortBindings[port], mapping)
 		exposedPorts[port] = struct{}{}
-		exposedPorts[nat.Port(fmt.Sprintf("%d/tcp", pm.HostPort))] = struct{}{}
 	}
 
 	config.ExposedPorts = exposedPorts

Original file line number	Diff line number	Diff line change
`@@ -727,7 +727,6 @@ func configurePortMappings(portMappings []PortMapping, config *dockercontainer.C`
`727`	`727`	`}`
`728`	`728`	`hostConfig.PortBindings[port] = append(hostConfig.PortBindings[port], mapping)`
`729`	`729`	`exposedPorts[port] = struct{}{}`
`730`		`- exposedPorts[nat.Port(fmt.Sprintf("%d/tcp", pm.HostPort))] = struct{}{}`
`731`	`730`	`}`
`732`	`731`
`733`	`732`	`config.ExposedPorts = exposedPorts`