Skip to content

Adapt VPA resources to explicitly specify update mode 'InPlaceOrRecreate' - #100

Merged
gardener-prow[bot] merged 1 commit into
gardener:masterfrom
DobromirNPeev:enable-in-place-pod-resource-updates
Jul 24, 2026
Merged

Adapt VPA resources to explicitly specify update mode 'InPlaceOrRecreate'#100
gardener-prow[bot] merged 1 commit into
gardener:masterfrom
DobromirNPeev:enable-in-place-pod-resource-updates

Conversation

@DobromirNPeev

Copy link
Copy Markdown
Contributor

What this PR does / why we need it:
Gardener's VPAInPlaceUpdates feature gate has been promoted and unconditionally enabled. This unconditionally enables the Gardener Resource
Manager's vpa-in-place-updates admission webhook, which was previously mutating the value from Recreate to InPlaceOrRecreate at admission
time. This change makes the extension explicitly set the correct value, aligning the code with the actual runtime behavior and removing the implicit
dependency on the webhook mutation.

Which issue(s) this PR fixes:
Part of gardener/gardener#12955

Special notes for your reviewer:
N/A

Release note:

The update mode of the VerticalPodAutoscaler resources deployed by the kupid is now explicitly set to `InPlaceOrRecreate`, reflecting the actual runtime behavior after the unconditional enablement of the `VPAInPlaceUpdates` feature gate.

@DobromirNPeev
DobromirNPeev requested a review from a team as a code owner July 22, 2026 11:07
@gardener-prow gardener-prow Bot added do-not-merge/needs-kind Indicates a PR lacks a `kind/foo` label and requires one. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. cla: yes Indicates the PR's author has signed the cla-assistant.io CLA. labels Jul 22, 2026
@Velin-Todorov

Copy link
Copy Markdown

I reviewed this and it appears okay. Consider it approved.

@gardener-prow gardener-prow Bot added the lgtm Indicates that a PR is ready to be merged. label Jul 24, 2026
@gardener-prow

gardener-prow Bot commented Jul 24, 2026

Copy link
Copy Markdown

LGTM label has been added.

DetailsGit tree hash: 1234133633b4c55af7763848f5ab34a3b31fe7ba

@gardener-prow gardener-prow Bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Jul 24, 2026
@shreyas-s-rao

Copy link
Copy Markdown
Member

/approve

@gardener-prow

gardener-prow Bot commented Jul 24, 2026

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: shreyas-s-rao

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@shreyas-s-rao

Copy link
Copy Markdown
Member

/kind enhancement

@gardener-prow gardener-prow Bot added kind/enhancement Enhancement, improvement, extension and removed do-not-merge/needs-kind Indicates a PR lacks a `kind/foo` label and requires one. labels Jul 24, 2026
@gardener-prow
gardener-prow Bot merged commit e28afd8 into gardener:master Jul 24, 2026
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. cla: yes Indicates the PR's author has signed the cla-assistant.io CLA. kind/enhancement Enhancement, improvement, extension lgtm Indicates that a PR is ready to be merged. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants