summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDaniel J Walsh <dwalsh@redhat.com>2020-04-23 09:25:15 -0400
committerDaniel J Walsh <dwalsh@redhat.com>2020-04-23 09:27:58 -0400
commitf44f830e9f3ac40b1aa28d4bd64eeebf3aca6237 (patch)
tree2639c3c528e714ab3607153083d7570cea894175
parent6f28c92865c4ac262bce790ecda2a07d1127760f (diff)
downloadpodman-f44f830e9f3ac40b1aa28d4bd64eeebf3aca6237.tar.gz
podman-f44f830e9f3ac40b1aa28d4bd64eeebf3aca6237.tar.bz2
podman-f44f830e9f3ac40b1aa28d4bd64eeebf3aca6237.zip
Don't check configuration until user input is applied
Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
-rw-r--r--cmd/podman/cliconfig/config.go1
-rw-r--r--cmd/podman/main.go3
-rw-r--r--libpod/runtime.go3
3 files changed, 5 insertions, 2 deletions
diff --git a/cmd/podman/cliconfig/config.go b/cmd/podman/cliconfig/config.go
index 6d98aaf0e..0ae10e337 100644
--- a/cmd/podman/cliconfig/config.go
+++ b/cmd/podman/cliconfig/config.go
@@ -708,7 +708,6 @@ type UntagValues struct {
func GetDefaultConfig() *config.Config {
var err error
conf, err := config.NewConfig("")
- conf.CheckCgroupsAndAdjustConfig()
if err != nil {
logrus.Errorf("Error loading container config %v\n", err)
os.Exit(1)
diff --git a/cmd/podman/main.go b/cmd/podman/main.go
index 4435b036e..75318db60 100644
--- a/cmd/podman/main.go
+++ b/cmd/podman/main.go
@@ -126,6 +126,9 @@ func before(cmd *cobra.Command, args []string) error {
return err
}
+ defaultContainerConfig.Engine.CgroupManager = MainGlobalOpts.CGroupManager
+ defaultContainerConfig.CheckCgroupsAndAdjustConfig()
+
if err := setupRootless(cmd, args); err != nil {
return err
}
diff --git a/libpod/runtime.go b/libpod/runtime.go
index a6032ad23..d5e2c51d8 100644
--- a/libpod/runtime.go
+++ b/libpod/runtime.go
@@ -131,8 +131,9 @@ func NewRuntime(ctx context.Context, options ...RuntimeOption) (runtime *Runtime
if err != nil {
return nil, err
}
+ runtime, err = newRuntimeFromConfig(ctx, conf, options...)
conf.CheckCgroupsAndAdjustConfig()
- return newRuntimeFromConfig(ctx, conf, options...)
+ return runtime, err
}
// NewRuntimeFromConfig creates a new container runtime using the given