Skip to content

Commit

Permalink
get runtime for podman-remote push earlier
Browse files Browse the repository at this point in the history
to prevent client side panics, we should get the runtime earlier in the
process of push.

Fixes: #4013

Signed-off-by: baude <[email protected]>
  • Loading branch information
baude committed Sep 13, 2019
1 parent 2077218 commit 5539a2a
Showing 1 changed file with 6 additions and 6 deletions.
12 changes: 6 additions & 6 deletions cmd/podman/push.go
Original file line number Diff line number Diff line change
Expand Up @@ -86,6 +86,12 @@ func pushCmd(c *cliconfig.PushValues) error {
destName = args[1]
}

runtime, err := adapter.GetRuntime(getContext(), &c.PodmanCommand)
if err != nil {
return errors.Wrapf(err, "could not create runtime")
}
defer runtime.DeferredShutdown(false)

// --compress and --format can only be used for the "dir" transport
splitArg := strings.SplitN(destName, ":", 2)
if c.Flag("compress").Changed || c.Flag("format").Changed {
Expand All @@ -106,12 +112,6 @@ func pushCmd(c *cliconfig.PushValues) error {
registryCreds = creds
}

runtime, err := adapter.GetRuntime(getContext(), &c.PodmanCommand)
if err != nil {
return errors.Wrapf(err, "could not create runtime")
}
defer runtime.DeferredShutdown(false)

var writer io.Writer
if !c.Quiet {
writer = os.Stderr
Expand Down

0 comments on commit 5539a2a

Please sign in to comment.