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 runtime 0.17 #756

Merged
merged 4 commits into from
Feb 8, 2024
Merged

Conversation

Zerpet
Copy link
Contributor

@Zerpet Zerpet commented Feb 1, 2024

Summary Of Changes

  • Implement CustomValidator interface because Validator is deprecated

dependabot bot and others added 2 commits February 1, 2024 17:01
Bumps [sigs.k8s.io/controller-runtime](https://github.com/kubernetes-sigs/controller-runtime) from 0.16.3 to 0.17.0.
- [Release notes](https://github.com/kubernetes-sigs/controller-runtime/releases)
- [Changelog](https://github.com/kubernetes-sigs/controller-runtime/blob/main/RELEASE.md)
- [Commits](kubernetes-sigs/controller-runtime@v0.16.3...v0.17.0)

---
updated-dependencies:
- dependency-name: sigs.k8s.io/controller-runtime
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
The interface Validator is deprecated starting in controller-runtime
0.17.0. We have to implement the new interface to avoid future breaking
changes.

Signed-off-by: Aitor Perez Cedres <[email protected]>
@Zerpet Zerpet self-assigned this Feb 1, 2024
@Zerpet
Copy link
Contributor Author

Zerpet commented Feb 1, 2024

System tests are failing locally for me. I suspect it's something related to my local kubernetes deployment with k3d. Let's see if they pass in CI.

Signed-off-by: Aitor Perez Cedres <[email protected]>
Signed-off-by: Aitor Perez Cedres <[email protected]>
@Zerpet Zerpet marked this pull request as ready for review February 6, 2024 16:27
Copy link
Member

@MirahImage MirahImage left a comment

Choose a reason for hiding this comment

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

Love the constant need to keep up with changing APIs. /sarcasm

Appreciate the more explicit testing around meaningful error messages.

@Zerpet Zerpet merged commit e8241cb into main Feb 8, 2024
5 checks passed
@Zerpet Zerpet added this to the v1.13.1 milestone Feb 8, 2024
@Zerpet Zerpet deleted the update-controller-runtime-0.17 branch February 8, 2024 11:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants