-
Notifications
You must be signed in to change notification settings - Fork 257
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
chore(build): upgrade Go to 1.20.10 #6063
Conversation
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.
Approving. Can you also pull in #6062 since it fixes the same http/2 issue in golang.org/x/net
? Thanks!
@jdstrand I've no idea how to satisfy checklist item |
@jeffreyssmith2nd or @bnpfeife - do you have details for this? |
That will happen as part of the next full release, so not something that needs to be worried about in this PR. |
Ah I'm sorry, misread this. The |
Here's a PR with the docs: #6064 |
Fyi just for clarity, we do want to cut a new chronograf release rather than just stage this PR's changes. However you get there is fine with me. :) |
0c78332
to
17378c7
Compare
17378c7
to
9c40dd4
Compare
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.
I've not done chronograf updates before, but looking at prior commits and seeing that chronograf-20231013
is in quay.io, approving.
Closes https://github.com/influxdata/edge/issues/509
etc/Dockerfile_build
have been pushed to DockerHub, and the changes have been added to.circleci/config.yml