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

Using imgpkg as a library causes docker authentication failure #397

Closed
joaopapereira opened this issue Jun 7, 2022 · 1 comment
Closed
Assignees
Labels
bug This issue describes a defect or unexpected behavior carvel accepted This issue should be considered for future work and that the triage process has been completed priority/important-longterm Important over the long term, but may not be staffed and/or may need multiple releases to complete.

Comments

@joaopapereira
Copy link
Member

What steps did you take:
When using imgpkg as a library when trying to push images to the registry, imgpkg always tries to get the credentials from the docker-credential helper as if it was in an IAAS, causing, in some cases, the authentication to fail and get 401.

Users do not know that the following patch needs to be applied https://github.com/vmware-tanzu/carvel-imgpkg/blob/develop/hack/patch-k8s-pkg-credentialprovider.patch on their vendor directory

What happened:
If the users do not apply the patch that imgpkg is also using on the dependency their application will get a 401.

What did you expect:
Remove the need to patch this dependency, but incorporate the needed code as part of imgpkg or use a different credential provider that does not force us to do this patch in the vendor folder.

Anything else you would like to add:
The bare minimum that we can do is to add some information about this in the docs. Not the optimal solution.

Environment:

  • imgpkg version (use imgpkg --version): version higher than 0.19
  • Docker registry used (e.g. Docker HUB): All
  • OS (e.g. from /etc/os-release): All

Vote on this request

This is an invitation to the community to vote on issues, to help us prioritize our backlog. Use the "smiley face" up to the right of this comment to vote.

👍 "I would like to see this addressed as soon as possible"
👎 "There are other more important things to focus on right now"

We are also happy to receive and review Pull Requests if you want to help work on this issue.

@joaopapereira joaopapereira added bug This issue describes a defect or unexpected behavior carvel triage This issue has not yet been reviewed for validity priority/important-longterm Important over the long term, but may not be staffed and/or may need multiple releases to complete. carvel accepted This issue should be considered for future work and that the triage process has been completed and removed carvel triage This issue has not yet been reviewed for validity labels Jun 7, 2022
joaopapereira added a commit to carvel-dev/vendir that referenced this issue Aug 22, 2022
Due to the inclusion of imgpkg as a library we need to ensure that the
change made in carvel-dev/imgpkg#255
stays in effect.
This will behavior can be removed when we complete the issue
carvel-dev/imgpkg#397

Signed-off-by: Joao Pereira <joaod@vmware.com>
joaopapereira added a commit to carvel-dev/vendir that referenced this issue Aug 22, 2022
Due to the inclusion of imgpkg as a library we need to ensure that the
change made in carvel-dev/imgpkg#255
stays in effect.
This will behavior can be removed when we complete the issue
carvel-dev/imgpkg#397

Signed-off-by: Joao Pereira <joaod@vmware.com>
joaopapereira added a commit to carvel-dev/vendir that referenced this issue Aug 22, 2022
Due to the inclusion of imgpkg as a library we need to ensure that the
change made in carvel-dev/imgpkg#255
stays in effect.
This will behavior can be removed when we complete the issue
carvel-dev/imgpkg#397

Signed-off-by: Joao Pereira <joaod@vmware.com>
joaopapereira added a commit to carvel-dev/vendir that referenced this issue Aug 23, 2022
Due to the inclusion of imgpkg as a library we need to ensure that the
change made in carvel-dev/imgpkg#255
stays in effect.
This will behavior can be removed when we complete the issue
carvel-dev/imgpkg#397

Signed-off-by: Joao Pereira <joaod@vmware.com>
@joaopapereira joaopapereira self-assigned this Sep 2, 2022
@joaopapereira
Copy link
Member Author

Fixed by PR #431 and released in version 0.32.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug This issue describes a defect or unexpected behavior carvel accepted This issue should be considered for future work and that the triage process has been completed priority/important-longterm Important over the long term, but may not be staffed and/or may need multiple releases to complete.
Projects
None yet
Development

No branches or pull requests

1 participant