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 and Kubernetes dependencies #1104

Merged
merged 2 commits into from
May 24, 2023

Conversation

hiddeco
Copy link
Member

@hiddeco hiddeco commented May 23, 2023

This deals with various breaking changes in controller-runtime, as
documented in the release notes:
https://github.com/kubernetes-sigs/controller-runtime/releases/tag/v0.15.0

In short:

  • Watches now use a client.Object instead of a source.Kind.
  • handler.MapFunc signature accepts a Go context, which is used to
    log any errors, instead of silently ignoring them and/or panicking.
  • Fake clients used in tests are now configured using
    WithStatusSubresource to enable the correct behavior for status
    updates and patches.
  • Max concurrent reconciles is configured on the manager, instead of
    configuring them per reconciler instance.
  • Various manager configuration options have been moved to new
    structures and/or fields.

In addition to this, all other dependencies which had updates are
updated to their latest (compatible) versions as well.

Part of: fluxcd/flux2#3907

Fix: #1098

- github.com/fluxcd/pkg/apis/meta to v1.1.0
- k8s.io/apimachinery to v0.27.2
- sigs.k8s.io/controller-runtime to v0.15.0

Signed-off-by: Hidde Beydals <hidde@hhh.computer>
@hiddeco hiddeco force-pushed the update-ctrl-runtime branch 3 times, most recently from ce3435d to eb135cf Compare May 23, 2023 22:11
@hiddeco hiddeco requested a review from stefanprodan May 23, 2023 22:11
@hiddeco hiddeco added the dependencies Pull requests that update a dependency label May 23, 2023
Copy link
Member

@stefanprodan stefanprodan left a comment

Choose a reason for hiding this comment

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

LGTM

Thanks @hiddeco 🥇

This deals with various breaking changes in controller-runtime, as
documented in the release notes:
https://github.com/kubernetes-sigs/controller-runtime/releases/tag/v0.15.0

In short:

- `Watches` now use a `client.Object` instead of a `source.Kind`.
- `handler.MapFunc` signature accepts a Go context, which is used to
  log any errors, instead of silently ignoring them and/or panicking.
- Fake clients used in tests are now configured using
  `WithStatusSubresource` to enable the correct behavior for status
  updates and patches.
- Max concurrent reconciles is configured on the manager, instead of
  configuring them per reconciler instance.
- Various manager configuration options have been moved to new
  structures and/or fields.

In addition to this, all other dependencies which had updates are
updated to their latest (compatible) versions as well.

Signed-off-by: Hidde Beydals <hidde@hhh.computer>
@hiddeco hiddeco merged commit 75a30f9 into main May 24, 2023
9 checks passed
@hiddeco hiddeco deleted the update-ctrl-runtime branch May 24, 2023 07:16
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Upgrade Go module for docker for fixing overlay network traffic CVEs
2 participants