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
build: Bump go to version 1.22.2 #11482
base: main
Are you sure you want to change the base?
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.
/approve
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: dhiller The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Kubernetes is building with go version 1.22[1] KubeVirt should align with this. go v1.22.2 includes a security fix for net/http and some more bug fixes[2] Build KubeVirt with go v1.22.2 [1] https://github.com/kubernetes/kubernetes/blob/b3926d137cd2964cd3a04088ded30845910547b1/go.mod#L9 [2] https://github.com/golang/go/issues?q=milestone%3AGo1.22.2+label%3ACherryPickApproved Signed-off-by: Brian Carey <bcarey@redhat.com>
rule_go v0.46.0 has support for go v1.22[1] [1] https://github.com/bazelbuild/rules_go/releases/tag/v0.46.0 Signed-off-by: Brian Carey <bcarey@redhat.com>
`make generate` fails with a panic with the current version Signed-off-by: Brian Carey <bcarey@redhat.com>
Signed-off-by: Brian Carey <bcarey@redhat.com>
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.
Thank you!
What this PR does
Kubernetes is building with go version 1.22[1]
KubeVirt should align with this.
go v1.22.2 includes a security fix for net/http and some more bug
fixes[2]
Build KubeVirt with go v1.22.2
[1] https://github.com/kubernetes/kubernetes/blob/b3926d137cd2964cd3a04088ded30845910547b1/go.mod#L9
[2] https://github.com/golang/go/issues?q=milestone%3AGo1.22.2+label%3ACherryPickApproved
Fixes #
Why we need it and why it was done in this way
The following tradeoffs were made:
The following alternatives were considered:
Links to places where the discussion took place:
Special notes for your reviewer
Majority of changes come from
make generate
changing the formatting of descriptions - 8bf4abfChecklist
This checklist is not enforcing, but it's a reminder of items that could be relevant to every PR.
Approvers are expected to review this list.
Release note