-
Notifications
You must be signed in to change notification settings - Fork 9
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: Update golang version and dependencies #31
Conversation
This PR is waiting on: kubevirt/kubevirt#10972 |
Signed-off-by: Andrej Krejcir <[email protected]>
Signed-off-by: Andrej Krejcir <[email protected]>
go.mod
Outdated
k8s.io/client-go => k8s.io/client-go v0.26.3 | ||
|
||
// TODO - remove after openshift dependencies stop using the wrong URL | ||
vbom.ml/util => github.com/fvbommel/util v0.0.3 |
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.
Is the broken openshift dep still in use? Can you tell where it originates from?
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 updated the openshift dependencies and now its not used anymore.
kubevirt.io/client-go module was updated to the latest commit in main branch, because it contains needed changes. When a new version of kubevirt.io/client-go is released, we will update go.mod file Signed-off-by: Andrej Krejcir <[email protected]>
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.
/lgtm
replace k8s.io/client-go => k8s.io/client-go v0.23.5 | ||
// Replacing these versions because kubevirt.io/client-go requires it. | ||
replace ( | ||
k8s.io/api => k8s.io/api v0.26.4 |
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.
Bump these to latest dot release v0.26.12
?
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.
Because this exact version is used by kubevirt.io/client-go
:
https://github.com/kubevirt/client-go/blob/304cdf11f9c46df9f15d1ed980d00b4bddb146b0/go.mod#L79-L104
I will try to post a PR to fix the inconvenient dependencies of kubevirt.io/client-go
.
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 APPROVED This pull-request has been approved by: 0xFelix The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
What this PR does / why we need it:
Updated golang to
1.20
, and module dependencies.Release note: