Skip to content

Upgrade controller-runtime to latest 0.24.1#670

Merged
MarcosDY merged 7 commits into
spiffe:mainfrom
alecholmez:main
May 14, 2026
Merged

Upgrade controller-runtime to latest 0.24.1#670
MarcosDY merged 7 commits into
spiffe:mainfrom
alecholmez:main

Conversation

@alecholmez
Copy link
Copy Markdown
Contributor

This change bumps controller-runtime to 0.24.1 (latest release as of May 13th, 2026), and updates the manager webhook registration to use the new generics support introduced in controller-runtime 0.23.0.

See breaking changes for more info:
https://github.com/kubernetes-sigs/controller-runtime/releases/tag/v0.23.0

Signed-off-by: Alec Holmes <alec.holmes@greymatter.io>
Copy link
Copy Markdown
Collaborator

@kfox1111 kfox1111 left a comment

Choose a reason for hiding this comment

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

Looks pretty good to me. But I dont know this part of the code base very well.

@marcosy what do you think?

Signed-off-by: Alec Holmes <alec.holmes@greymatter.io>
@alecholmez alecholmez requested a review from kfox1111 May 13, 2026 20:05
Signed-off-by: Alec Holmes <alec.holmes@greymatter.io>
@alecholmez
Copy link
Copy Markdown
Contributor Author

alecholmez commented May 13, 2026

I just addressed the last deprecation warning for the scheme builder. The deprecation warning example seems to imply that the new pattern is registering types in a central location, which I followed here. If everyone prefers that I split that back out I can do so just let me know.

Copy link
Copy Markdown
Collaborator

@MarcosDY MarcosDY left a comment

Choose a reason for hiding this comment

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

Looks great just a question,
why you use a commit on protobuf? can you use the tag instead?

Comment thread go.mod
Comment thread .go-version Outdated
Signed-off-by: Alec Holmes <alec.holmes@greymatter.io>
Signed-off-by: Alec Holmes <alec.holmes@greymatter.io>
Comment thread go.mod Outdated
Signed-off-by: Alec Holmes <alec.holmes@greymatter.io>
Signed-off-by: Alec Holmes <alec.holmes@greymatter.io>
@alecholmez
Copy link
Copy Markdown
Contributor Author

@MarcosDY Should be at 1.26.3 across the board now

@MarcosDY MarcosDY added this pull request to the merge queue May 14, 2026
Merged via the queue into spiffe:main with commit 715d9e2 May 14, 2026
7 checks passed
@alecholmez
Copy link
Copy Markdown
Contributor Author

@kfox1111 @MarcosDY Will you guys be able to cut a new tag with the controller-runtime update?

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.

3 participants