Skip to content

feat(vpa): enable InPlaceOrRecreate feature gate for updater and admissionController#1291

Merged
botantler[bot] merged 1 commit intomainfrom
feat/vpa-inplaceorrecreate
Mar 28, 2026
Merged

feat(vpa): enable InPlaceOrRecreate feature gate for updater and admissionController#1291
botantler[bot] merged 1 commit intomainfrom
feat/vpa-inplaceorrecreate

Conversation

@devantler
Copy link
Copy Markdown
Collaborator

Description

Enable VPA InPlaceOrRecreate feature gate for updater and admissionController components in the standalone VPA helm-release. Also fixes the values comment URL from Goldilocks to VPA.

Changes

  • Added extraArgs with feature-gates: InPlaceOrRecreate=true to both updater and admissionController in the VPA helm-release
  • Fixed values comment URL from goldilocks/values.yaml to vpa/values.yaml
  • Added TODO comments to remove feature gates when VPA 1.6.0+ ships (InPlaceOrRecreate becomes GA)

Context

Issues #993 and #990 (remove VPA image overrides from Goldilocks sub-chart) were already resolved when VPA was separated into a standalone helm release. Those issues are now closed.

Issue #989 (remove InPlaceOrRecreate extraArgs when VPA supports it by default) remains open — the feature gates were lost during the VPA restructuring and are now re-enabled. They can be removed once VPA chart reaches appVersion 1.6.0+ where InPlaceOrRecreate is GA.

Closes #993
Closes #990
Refs #989

Copilot AI review requested due to automatic review settings March 28, 2026 12:03
@botantler botantler bot enabled auto-merge March 28, 2026 12:03
@github-project-automation github-project-automation bot moved this from 🫴 Ready to 🚀 In Finalization in 🌊 Project Board Mar 28, 2026
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Enables the VPA InPlaceOrRecreate behavior by re-introducing the InPlaceOrRecreate feature gate for the standalone Vertical Pod Autoscaler HelmRelease (after VPA was separated out from the Goldilocks sub-chart), and fixes the referenced upstream values URL comment.

Changes:

  • Add extraArgs.feature-gates: InPlaceOrRecreate=true for the VPA updater component.
  • Add extraArgs.feature-gates: InPlaceOrRecreate=true for the VPA admissionController component.
  • Fix the upstream values reference comment from Goldilocks to VPA and add TODO notes for later removal once GA.

@botantler botantler bot added this pull request to the merge queue Mar 28, 2026
Merged via the queue into main with commit e45d40c Mar 28, 2026
10 checks passed
@botantler botantler bot deleted the feat/vpa-inplaceorrecreate branch March 28, 2026 12:15
@github-project-automation github-project-automation bot moved this from 🚀 In Finalization to ✅ Done in 🌊 Project Board Mar 28, 2026
@botantler
Copy link
Copy Markdown
Contributor

botantler bot commented Mar 28, 2026

🎉 This PR is included in version 2.24.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@botantler botantler bot added the released label Mar 28, 2026
@devantler devantler restored the feat/vpa-inplaceorrecreate branch March 28, 2026 12:18
@botantler botantler bot deleted the feat/vpa-inplaceorrecreate branch March 28, 2026 12:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Status: ✅ Done

2 participants