Skip to content

Migrate from deprecated client.Apply Patch to typed Apply#378

Merged
peanball merged 1 commit into
ironcore-dev:mainfrom
sap-contributions:fix/client-apply-migration
Jul 23, 2026
Merged

Migrate from deprecated client.Apply Patch to typed Apply#378
peanball merged 1 commit into
ironcore-dev:mainfrom
sap-contributions:fix/client-apply-migration

Conversation

@peanball

Copy link
Copy Markdown
Member

controller-runtime v0.23 deprecates the client.Apply Patch pattern in favor
of Client.Apply() with a typed apply configuration.

This fix is prior to the dependabot bump, which caused the 0.23 bump to fail (deprecated functionality called).

… typed Apply

controller-runtime v0.23 deprecates the client.Apply Patch pattern in favor
of Client.Apply() with a typed apply configuration. The only Apply call site
in this repo targets corev1.Service (a built-in Kubernetes type), so the
upstream k8s.io/client-go/applyconfigurations/core/v1 constructors are used
directly — no in-repo applyconfiguration generation is needed.

Signed-off-by: Alexander Lais <alexander.lais@sap.com>
@peanball
peanball requested review from a team as code owners July 23, 2026 09:31
@github-actions github-actions Bot added the bug Something isn't working label Jul 23, 2026
@peanball
peanball merged commit 6e5c53f into ironcore-dev:main Jul 23, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants