Fix `podman pod create --infra=false`

We were accidentally setting incorrect defaults for the network
namespace for rootless `pod create` when infra containers were
not being created. This should resolve that issue.

Signed-off-by: Matthew Heon <matthew.heon@pm.me>

1 files changed, 2 insertions, 10 deletions

diff --git a/pkg/specgen/pod_validate.go b/pkg/specgen/pod_validate.go
index 08f1c0300..640447e71 100644
--- a/pkg/specgen/pod_validate.go
+++ b/pkg/specgen/pod_validate.go
@@ -1,7 +1,6 @@
 package specgen
 
 import (
-	"github.com/containers/libpod/pkg/rootless"
 	"github.com/containers/libpod/pkg/util"
 	"github.com/pkg/errors"
 )
@@ -37,8 +36,8 @@ func (p *PodSpecGenerator) Validate() error {
 		return err
 	}
 	if p.NoInfra {
-		if p.NetNS.NSMode == NoNetwork {
-			return errors.New("NoInfra and a none network cannot be used toegther")
+		if p.NetNS.NSMode != Default && p.NetNS.NSMode != "" {
+			return errors.New("NoInfra and network modes cannot be used toegther")
 		}
 		if p.StaticIP != nil {
 			return exclusivePodOptions("NoInfra", "StaticIP")
@@ -86,13 +85,6 @@ func (p *PodSpecGenerator) Validate() error {
 	}
 
 	// Set Defaults
-	if p.NetNS.Value == "" {
-		if rootless.IsRootless() {
-			p.NetNS.NSMode = Slirp
-		} else {
-			p.NetNS.NSMode = Bridge
-		}
-	}
 	if len(p.InfraImage) < 1 {
 		p.InfraImage = containerConfig.Engine.InfraImage
 	}