Browse code

Improve message when pushing a non-existent image

I was confused earlier when I did:

```
docker push localhost.localdomain:1234/foo
```

Because docker told me:

```
No such id: localhost.localdomain:1234/foo
```

I actually had buried in my mind the solution to this, but the error
message
confused me because I had recently had some fun trying to get the
registry
working and therefore thought it was telling me that I didn't have an
account
on the registry.

This pull request makes it unambiguous that the error is that the
specified
image is unknown.

/cc @cpuguy83

Docker-DCO-1.1-Signed-off-by: Peter Waller <p@pwaller.net> (github: pwaller)

Peter Waller authored on 2014/12/12 02:57:23
Showing 1 changed files
... ...
@@ -88,7 +88,7 @@ func (graph *Graph) Exists(id string) bool {
88 88
 func (graph *Graph) Get(name string) (*image.Image, error) {
89 89
 	id, err := graph.idIndex.Get(name)
90 90
 	if err != nil {
91
-		return nil, err
91
+		return nil, fmt.Errorf("could not find image: %v", err)
92 92
 	}
93 93
 	img, err := image.LoadImage(graph.ImageRoot(id))
94 94
 	if err != nil {