-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
Download docker-credential-gcr from release artifacts #2540
Download docker-credential-gcr from release artifacts #2540
Conversation
438db2d
to
916163b
Compare
Thank you @ferrastas for flagging this and submitting the PR here! I wasn't aware of this when I recently bumped the tag there. My only concern with the PR is that Kaniko builds images for multiple architectures: and currently I believe this PR does not parameterize the arch (assumes linux_amd64). If you could parameterize the arch s.t. the Dockerfile works for all arch in our |
Thanks @aaron-prindle, 😓 I totally missed that... |
As described in GoogleContainerTools#2539, using `go install` with a pinned hash version it installs `1.5.1` in all cases. This seems to be an issue on the upstream project https://github.com/GoogleCloudPlatform/docker-credential-gcr and how go proxy deals with versions. As a temporary solution, we download the compiled version from the release artifacts. Closes GoogleContainerTools#2539 Signed-off-by: Ferran Vidal <ferran.vidal.p@gmail.com>
916163b
to
ea63550
Compare
@aaron-prindle seems that one of the integration tests is a bit flaky, I cannot retry the failing job, could you do it, please? |
Definitely @ferrastas, kicking off the job again now. |
@aaron-prindle do you think that you have some time to review and merge this? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! Thanks for your help here @ferrastas!!
…rTools#2540) As described in GoogleContainerTools#2539, using `go install` with a pinned hash version it installs `1.5.1` in all cases. This seems to be an issue on the upstream project https://github.com/GoogleCloudPlatform/docker-credential-gcr and how go proxy deals with versions. As a temporary solution, we download the compiled version from the release artifacts. Closes GoogleContainerTools#2539 Signed-off-by: Ferran Vidal <ferran.vidal.p@gmail.com>
Fixes #2539
Description
As described in #2539, using
go install
with a pinned hash version it installs1.5.1
in all cases.This seems to be an issue on the upstream project https://github.com/GoogleCloudPlatform/docker-credential-gcr and how go proxy deals with versions.
As a temporary solution, we download the compiled version from the release artifacts.
Submitter Checklist
These are the criteria that every PR should meet, please check them off as you
review them:
See the contribution guide for more details.
Reviewer Notes