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

fix: upgrade controller runtime to v0.20.1 and conforming to new TypedEventHandler interface #46

Merged
merged 2 commits into from
Feb 15, 2025

Conversation

scoquelin
Copy link
Contributor

@scoquelin scoquelin commented Feb 2, 2025

This PR is upgrading controller-runtime to v0.20.1 and adapting node_controller.go to conform to the new TypedEventHandler interface

@scoquelin scoquelin changed the title Upgrade controller runtime to v0.20.1 and conforming to new TypedEventHandler interface chore: upgrade controller runtime to v0.20.1 and conforming to new TypedEventHandler interface Feb 2, 2025
@scoquelin scoquelin changed the title chore: upgrade controller runtime to v0.20.1 and conforming to new TypedEventHandler interface fix: upgrade controller runtime to v0.20.1 and conforming to new TypedEventHandler interface Feb 2, 2025
@scoquelin scoquelin force-pushed the upgrade-controller-runtime branch from 301e7f8 to 2795b6d Compare February 2, 2025 22:24
@scoquelin scoquelin requested a review from chomatdam February 2, 2025 22:25
@scoquelin scoquelin self-assigned this Feb 2, 2025
@scoquelin scoquelin force-pushed the upgrade-controller-runtime branch from 2795b6d to 8316782 Compare February 13, 2025 20:41
@scoquelin scoquelin force-pushed the upgrade-controller-runtime branch from 8316782 to af963df Compare February 13, 2025 20:45
@chomatdam chomatdam marked this pull request as ready for review February 14, 2025 23:21
@chomatdam chomatdam merged commit 9d3f9c1 into main Feb 15, 2025
3 checks passed
@chomatdam chomatdam deleted the upgrade-controller-runtime branch February 15, 2025 03:43
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.

2 participants