Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Regression: Service image no longer pinned by hash in API v1.26 #34324

Open
matthanley opened this issue Jul 31, 2017 · 5 comments
Open

Regression: Service image no longer pinned by hash in API v1.26 #34324

matthanley opened this issue Jul 31, 2017 · 5 comments

Comments

@matthanley
Copy link

Description

Creating a service in 17.06 fails to pin image by tag, causing subsequent calls to /v1.26/services/<service>/update failing to update to most recent image when given image:tag

Steps to reproduce the issue:

  • Service created via docker-py library (using API v1.26)

Describe the results you received:

  • docker service inspect shows image as provided (in image:tag format)

Describe the results you expected:

  • docker service inspect show service manifest linked to SHA hash reference of the image

Additional information you deem important (e.g. issue happens only occasionally):

Seems to be related to #32388 but has affected earlier API versions.

Output of docker version:

Docker version 17.06.0-ce, build 02c1d87
@matthanley
Copy link
Author

matthanley commented Jul 31, 2017

From additional testing, when updating a service manually via command line I'm getting the warning:

image <image>:staging could not be accessed on a registry to record
its digest. Each node will access <image>:staging independently,
possibly leading to different nodes running different
versions of the image.

<image> in this instance is hosted on a private registry, but I can docker pull <image> successfully.


EDIT

docker service update --image <image> --with-registry-auth <service> works without the above error, but docker service inspect still doesn't have the service image pinned.

"Spec": {
            ...
            "TaskTemplate": {
                "ContainerSpec": {
                    "Image": "***:staging",

@danieljuhl
Copy link

danieljuhl commented Aug 1, 2017

Probably related to / duplicate of #33929

@matthanley
Copy link
Author

matthanley commented Aug 1, 2017

Absolutely, the description in #33929 matches what I'm seeing.

(I don't think this should be a dup though, as this issue is specifically relating to the regression breaking backwards compatibility with an existing API version)

@vdemeester
Copy link
Member

@danieljuhl
Copy link

The other issue is also a regression breaking backwards comp., just on the CLI and not through the API. But I suspect that the root for both issues is the same, which just affects all possible ways to trigger the update.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants