diff options
author | Daniel J Walsh <dwalsh@redhat.com> | 2020-04-23 09:25:15 -0400 |
---|---|---|
committer | Daniel J Walsh <dwalsh@redhat.com> | 2020-04-23 09:27:58 -0400 |
commit | f44f830e9f3ac40b1aa28d4bd64eeebf3aca6237 (patch) | |
tree | 2639c3c528e714ab3607153083d7570cea894175 | |
parent | 6f28c92865c4ac262bce790ecda2a07d1127760f (diff) | |
download | podman-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.go | 1 | ||||
-rw-r--r-- | cmd/podman/main.go | 3 | ||||
-rw-r--r-- | libpod/runtime.go | 3 |
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 |