diff options
author | Valentin Rothberg <rothberg@redhat.com> | 2021-02-15 11:53:07 +0100 |
---|---|---|
committer | Matthew Heon <mheon@redhat.com> | 2021-02-18 10:47:04 -0500 |
commit | af37b6762f70b7dd31272d403c4c89753a54b281 (patch) | |
tree | cac4875c1aec7f95050cff1d28c52c95a1d68dd5 | |
parent | 8dc2fb2c78a8404341e9e037ee4418d876c26366 (diff) | |
download | podman-af37b6762f70b7dd31272d403c4c89753a54b281.tar.gz podman-af37b6762f70b7dd31272d403c4c89753a54b281.tar.bz2 podman-af37b6762f70b7dd31272d403c4c89753a54b281.zip |
correct startup error message
The error message when failing to create an image engine unconditionally
pointed to the Podman socket which is quite confusing when running
locally.
Move the error message to the point where the first ping to the service
fails.
[NO TESTS NEEDED]
Signed-off-by: Valentin Rothberg <rothberg@redhat.com>
-rw-r--r-- | cmd/podman/root.go | 2 | ||||
-rw-r--r-- | pkg/bindings/connection.go | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/cmd/podman/root.go b/cmd/podman/root.go index 0ee530242..1f613a4c5 100644 --- a/cmd/podman/root.go +++ b/cmd/podman/root.go @@ -158,7 +158,7 @@ func persistentPreRunE(cmd *cobra.Command, args []string) error { // Prep the engines if _, err := registry.NewImageEngine(cmd, args); err != nil { - return errors.Wrapf(err, "Cannot connect to the Podman socket, make sure there is a Podman REST API service running.") + return err } if _, err := registry.NewContainerEngine(cmd, args); err != nil { return err diff --git a/pkg/bindings/connection.go b/pkg/bindings/connection.go index 7b26037eb..6ff072e75 100644 --- a/pkg/bindings/connection.go +++ b/pkg/bindings/connection.go @@ -124,7 +124,7 @@ func NewConnectionWithIdentity(ctx context.Context, uri string, identity string) ctx = context.WithValue(ctx, clientKey, &connection) if err := pingNewConnection(ctx); err != nil { - return nil, err + return nil, errors.Wrap(err, "cannot connect to the Podman socket, please verify that Podman REST API service is running") } return ctx, nil } |