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

Download docker-credential-gcr from release artifacts #2540

Merged

Conversation

ferrastas
Copy link
Contributor

Fixes #2539

Description

As described in #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.

Submitter Checklist

These are the criteria that every PR should meet, please check them off as you
review them:

  • Includes unit tests
  • Adds integration tests if needed.

See the contribution guide for more details.

Reviewer Notes

  • The code flow looks good.
  • Unit tests and or integration tests added.

@aaron-prindle
Copy link
Collaborator

aaron-prindle commented May 31, 2023

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:
https://github.com/GoogleContainerTools/kaniko/blob/main/.github/workflows/images.yaml#L36

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 github/workflows /images.yaml and we get the correct docker-credential-gcr based on the arch we should be able to merge this.

@ferrastas
Copy link
Contributor Author

Thanks @aaron-prindle, 😓 I totally missed that...
I will push a different commit.

@aaron-prindle aaron-prindle modified the milestone: v1.11.0 May 31, 2023
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>
@ferrastas
Copy link
Contributor Author

@aaron-prindle seems that one of the integration tests is a bit flaky, I cannot retry the failing job, could you do it, please?

@aaron-prindle
Copy link
Collaborator

Definitely @ferrastas, kicking off the job again now.

@ferrastas
Copy link
Contributor Author

@aaron-prindle do you think that you have some time to review and merge this?

Copy link
Collaborator

@aaron-prindle aaron-prindle left a 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!!

@aaron-prindle aaron-prindle merged commit f4dae00 into GoogleContainerTools:main Jun 2, 2023
kylecarbs pushed a commit to coder/kaniko that referenced this pull request Jul 12, 2023
…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>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Install docker-credential-gcr using pinned version, installs 1.5.1 instead of the desired version
2 participants