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 controller-gen version to 0.14.0 #391

Closed
wants to merge 1 commit into from

Conversation

thunderboltsid
Copy link
Contributor

@thunderboltsid thunderboltsid commented Mar 13, 2024

Go1.22 had a change that broke controller-gen version 0.8.0 through 0.13.0.

Go1.22 also drops support for GO111MODULE=off go get -u. This PR replaces those directives with go install ones.

yannickstruyf3
yannickstruyf3 previously approved these changes Mar 13, 2024
@nutanix-cn-prow-bot
Copy link

@thunderboltsid: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/e2e-k8s-conformance ca2c9a3 link false /test e2e-k8s-conformance

Full PR test history. Your PR dashboard.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

Go1.22 broke controller-gen version 0.8.0 through 0.13.0.
Go1.22 drops support for GO111MODULE=off go get -u in legacy gopath
mode.
Copy link

✅ None of your dependencies violate policy!

@thunderboltsid
Copy link
Contributor Author

Closing this PR and moving the commits to #390 as E2Es won't pass without Kubernetes 1.29 support on that PR and unit tests on that PR won't pass without changes in this PR.

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.

None yet

2 participants