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

fix: Upgrade Golang builder version to 1.22.7 #379

Merged
merged 1 commit into from
Sep 23, 2024

Conversation

omegion
Copy link
Contributor

@omegion omegion commented Sep 23, 2024

What this PR does / why we need it:

Fixes Docker image builder.

Which issue this PR fixes

  • Docker builder fails due to Golang version, this PR upgrades the Golang version to build the image successfully.

Special notes:

Checklist chart

[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

  • Chart Version bumped
  • Variables are documented in the README.md

@omegion
Copy link
Contributor Author

omegion commented Sep 23, 2024

The docker image builder fails due to the Golang version. This fixes the builder, fyi @jtblin.

@omegion
Copy link
Contributor Author

omegion commented Sep 23, 2024

created an issue for this PR: #380

@jtblin jtblin merged commit 76915c8 into jtblin:master Sep 23, 2024
@jtblin
Copy link
Owner

jtblin commented Sep 23, 2024

Thanks!

@omegion
Copy link
Contributor Author

omegion commented Sep 23, 2024

@jtblin any chance you cut a new version for Dockerhub, please?

@jtblin
Copy link
Owner

jtblin commented Sep 23, 2024

@omegion just tried but circleci is not picking the workflow as it looks like it didn't get the branch when I pushed the tags for some reason 🤷

Going to have to troubleshoot later.

@omegion
Copy link
Contributor Author

omegion commented Sep 25, 2024

@jtblin any luck with the CI pipeline for the release?

@jtblin
Copy link
Owner

jtblin commented Sep 27, 2024

Sorry I haven't add the chance to look into it but jtblin/kube2iam:76915c8 should be the latest version with your fix.

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.

3 participants