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
@hardikdr hardikdr added the area/iaas Issues related to IronCore IaaS development. label Aug 21, 2026
@hardikdr hardikdr added this to Roadmap Aug 21, 2026
@github-project-automation github-project-automation Bot moved this to Done in Roadmap Aug 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/iaas Issues related to IronCore IaaS development. bug Something isn't working

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants