summaryrefslogtreecommitdiff
path: root/cmd
diff options
context:
space:
mode:
authorOpenShift Merge Robot <openshift-merge-robot@users.noreply.github.com>2019-04-12 12:40:51 -0700
committerGitHub <noreply@github.com>2019-04-12 12:40:51 -0700
commit15723671b3edefd3cac150dc86323aa1f701ef8d (patch)
tree38a9abc521ad867f75bba810a0fae94a55ab28b8 /cmd
parent89ffcf9df9d83bc8e0ea7c8004591fb0825117cf (diff)
parentfcea6fda82cc3809f8990c0c80a52f6c4a03c5cd (diff)
downloadpodman-15723671b3edefd3cac150dc86323aa1f701ef8d.tar.gz
podman-15723671b3edefd3cac150dc86323aa1f701ef8d.tar.bz2
podman-15723671b3edefd3cac150dc86323aa1f701ef8d.zip
Merge pull request #2911 from giuseppe/fix-pull-errors
pull: fix a couple of issues
Diffstat (limited to 'cmd')
-rw-r--r--cmd/podman/pull.go9
1 files changed, 7 insertions, 2 deletions
diff --git a/cmd/podman/pull.go b/cmd/podman/pull.go
index 491d3a8c2..7cc7b65b3 100644
--- a/cmd/podman/pull.go
+++ b/cmd/podman/pull.go
@@ -61,7 +61,12 @@ func init() {
// pullCmd gets the data from the command line and calls pullImage
// to copy an image from a registry to a local machine
-func pullCmd(c *cliconfig.PullValues) error {
+func pullCmd(c *cliconfig.PullValues) (retError error) {
+ defer func() {
+ if retError != nil && exitCode == 0 {
+ exitCode = 1
+ }
+ }()
if c.Bool("trace") {
span, _ := opentracing.StartSpanFromContext(Ctx, "pullCmd")
defer span.Finish()
@@ -163,7 +168,7 @@ func pullCmd(c *cliconfig.PullValues) error {
for _, name := range names {
newImage, err := runtime.New(getContext(), name, c.String("signature-policy"), authfile, writer, &dockerRegistryOptions, image.SigningOptions{}, true, nil)
if err != nil {
- println(errors.Wrapf(err, "error pulling image %q", name))
+ logrus.Errorf("error pulling image %q", name)
foundImage = false
continue
}