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

[v14] Bump controller-tools to 0.14 as 0.9 panics with go 1.22 #43556

Merged
merged 1 commit into from
Jun 26, 2024

Conversation

hugoShaka
Copy link
Contributor

Fixes a controller-gen panic caused by the go 1.22 backport we did today.

The 0.14.0 version comes from branch/v15 which is not panicking (according to the issue comments, 0.12.1 is broken and 0.13.0 works).

Copy link

The PR changelog entry failed validation: Changelog entry not found in the PR body. Please add a "no-changelog" label to the PR, or changelog lines starting with changelog: followed by the changelog entries for the PR.

@hugoShaka hugoShaka added the no-changelog Indicates that a PR does not require a changelog entry label Jun 26, 2024
@hugoShaka hugoShaka enabled auto-merge June 26, 2024 17:58
Copy link
Contributor

@codingllama codingllama left a comment

Choose a reason for hiding this comment

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

Thanks for the fix!

@hugoShaka hugoShaka added this pull request to the merge queue Jun 26, 2024
Merged via the queue into branch/v14 with commit c770bcf Jun 26, 2024
32 of 33 checks passed
@hugoShaka hugoShaka deleted the hugo/fix-controller-tool-panic branch June 26, 2024 18:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport bug no-changelog Indicates that a PR does not require a changelog entry size/sm
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants