diff options
| author | Matthew Heon <mheon@redhat.com> | 2020-04-24 09:44:51 -0400 |
|---|---|---|
| committer | Matthew Heon <mheon@redhat.com> | 2020-04-24 09:44:51 -0400 |
| commit | b1552d9124d3d2f927366a86c31a7902eb60ca64 (patch) | |
| tree | e846f513577a5890a1bfbd7ffdf8e25941aa7293 /pkg/specgen | |
| parent | 46463e454e7fed9c25c762912a1dc7c3c7421cce (diff) | |
| download | podman-b1552d9124d3d2f927366a86c31a7902eb60ca64.tar.gz podman-b1552d9124d3d2f927366a86c31a7902eb60ca64.tar.bz2 podman-b1552d9124d3d2f927366a86c31a7902eb60ca64.zip | |
Do not join pod namespaces without an infra ctr
We do not want to join pod namespaces if no infra container is
present. A pod may claim it shares namespaces without an infra
container (I'll take an action item to fix that - it really
should not be allowed), which was tripping up our default
namespace code and forcing us to try and join the namespaces of
the nonexistant infra container.
Signed-off-by: Matthew Heon <mheon@redhat.com>
Diffstat (limited to 'pkg/specgen')
| -rw-r--r-- | pkg/specgen/generate/namespaces.go | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/pkg/specgen/generate/namespaces.go b/pkg/specgen/generate/namespaces.go index 2aaeb9513..2e0e088bf 100644 --- a/pkg/specgen/generate/namespaces.go +++ b/pkg/specgen/generate/namespaces.go @@ -26,7 +26,7 @@ func GetDefaultNamespaceMode(nsType string, cfg *config.Config, pod *libpod.Pod) nsType = strings.ToLower(nsType) // If the pod is not nil - check shared namespaces - if pod != nil { + if pod != nil && pod.HasInfraContainer() { podMode := false switch { case nsType == "pid" && pod.SharesPID(): |