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

Update to go 1.21 #5510

Closed

Conversation

Neo2308
Copy link

@Neo2308 Neo2308 commented Aug 12, 2023

What this PR does:
Updated build-image/Dockerfile as instructed at https://cortexmetrics.io/docs/contributing/how-to-update-the-build-image/

Which issue(s) this PR fixes:

Checklist

  • Tests updated
  • Documentation added
  • CHANGELOG.md updated - the order of entries should be [CHANGE], [FEATURE], [ENHANCEMENT], [BUGFIX]

@Neo2308 Neo2308 force-pushed the feature/master/update-to-go-1.21 branch from b9a1423 to 96a8c55 Compare August 12, 2023 10:19
@Neo2308
Copy link
Author

Neo2308 commented Aug 12, 2023

Waiting for a maintainer to push the build image to quay (Steps 2-4 on https://cortexmetrics.io/docs/contributing/how-to-update-the-build-image/). Once that is done, I can make the rest of the changes in this PR.

build-image/Dockerfile Outdated Show resolved Hide resolved
Signed-off-by: Friedrich Gonzalez <[email protected]>
Copy link
Contributor

@yeya24 yeya24 left a comment

Choose a reason for hiding this comment

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

Should we update all images we use to 1.21? Now I can still see we are using go 1.20 in CI

@yeya24
Copy link
Contributor

yeya24 commented Oct 24, 2023

Hi @Neo2308, sorry for the late response.

I have built the image based on this branch and pushed the image to quay.

This is the image. quay.io/cortexproject/build-image:feature-master-update-to-go-1.21-050c36e7b.

Please let me know if you are still willing to continue this. Thanks!

@yeya24
Copy link
Contributor

yeya24 commented Oct 24, 2023

Hey @Neo2308, I am sorry I will close this pr due to another pr for the same purpose #5618 got merged.

Thanks so much for your contribution and I apologize for any inconvenience I caused for not helping unblock this change timely.

@yeya24 yeya24 closed this Oct 24, 2023
@Neo2308 Neo2308 deleted the feature/master/update-to-go-1.21 branch October 25, 2023 10:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants