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

Upgrade controller-runtime to v0.15.0 and k8s api to v0.30.0 #953

Merged
merged 2 commits into from
May 3, 2024

Conversation

kralicky
Copy link
Contributor

@kralicky kralicky commented May 1, 2024

Summary

Some small refactors were made to the manager setup logic to account for breaking changes in controller-runtime, and to prepare for its upcoming generic builder api.

Related issues

#952

Checklist

  • reference any related issues
  • updated docs
  • updated unit tests
  • updated UPGRADING.md
  • add appropriate tag (improvement / bug / etc)
  • ready for review

@kralicky kralicky added the dependencies Pull requests that update a dependency file label May 1, 2024
@kralicky kralicky requested a review from wasaga May 1, 2024 21:59
@kralicky kralicky requested a review from a team as a code owner May 1, 2024 21:59
@kralicky kralicky force-pushed the kralicky/upgrade-controller-runtime branch from ae8f5cb to 018ea2d Compare May 1, 2024 22:24
Some small refactors were made to the manager setup logic to account for
breaking changes in controller-runtime, and to prepare for its upcoming
generic builder api.
@kralicky kralicky force-pushed the kralicky/upgrade-controller-runtime branch from 018ea2d to 8b64e88 Compare May 1, 2024 22:25
go.mod Outdated Show resolved Hide resolved
@kralicky kralicky force-pushed the kralicky/upgrade-controller-runtime branch from 91cca80 to 4b72477 Compare May 2, 2024 21:31
@kralicky kralicky merged commit f966708 into main May 3, 2024
7 checks passed
@kralicky kralicky deleted the kralicky/upgrade-controller-runtime branch May 3, 2024 20:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants