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

manifest list: request specific permissions #1024

Merged
merged 1 commit into from
Apr 30, 2018

Conversation

clnperez
Copy link
Contributor

When creating manifest lists, don't use "*" as the permission when
creating the token handler. This causes problems with gitlab's repos.

Fixes #1010

Signed-off-by: Christy Norman [email protected]

- What I did

- How I did it

- How to verify it

- Description for the changelog

- A picture of a cute animal (not mandatory but encouraged)

When creating manifest lists, don't use "*" as the permission when
creating the token handler. This causes problems with gitlab's repos.

Fixes docker#1010

Signed-off-by: Christy Norman <[email protected]>
@clnperez clnperez changed the title specify specific permissions manifest list: request specific permissions Apr 24, 2018
@clnperez
Copy link
Contributor Author

I hope this is the super simple solution here. I tried another way to do it, but you need push and pull perm for the push, so, it seems like this is easiest/best.

@vdemeester is there a way to download the build artifacts for @jesusofsuburbia to test? I can't find it where I think it should be.

@vdemeester
Copy link
Collaborator

@clnperez yeah, it's a bit tricky to get but it's possible to get them from the articfacts in the circleci build (somehow the tab doesn't show when not logged but is accessible) : https://circleci.com/gh/docker/cli/11565#artifacts/containers/0

@clnperez
Copy link
Contributor Author

@jesusofsuburbia can you try that one out? you can just chmod+x and run it instead of the docker installed on your system. If you expand the build section in the link from the previous comment, you can find binaries for all the platforms docker supports.

@mschrupp
Copy link

@clnperez will do so today or tomorrow, thanks so far!

@thaJeztah
Copy link
Member

@jlhawn @dmcgowan PTAL

@mschrupp
Copy link

@clnperez it works! I tested it multiple times...

docker:18.04.0-ce fails, but overwriting /usr/local/bin/docker with the binary you provided works

thank you so much

@clnperez
Copy link
Contributor Author

thank you for reporting :)

Copy link
Member

@thaJeztah thaJeztah left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, but would like @dmcgowan or @jlhawn to have a quick peek

Copy link
Contributor

@dmcgowan dmcgowan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@thaJeztah
Copy link
Member

Thanks for reviewing 👍 merging

@thaJeztah thaJeztah merged commit f42c66b into docker:master Apr 30, 2018
@GordonTheTurtle GordonTheTurtle added this to the 18.05.0 milestone Apr 30, 2018
@thaJeztah thaJeztah modified the milestones: 18.05.0, 18.06.0 May 2, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Docker manifest fails for registries/users where JWT action "*" is restricted (e.g. gitlab)
7 participants