only allow in-place if explicitly set (for testing)

jkyros · maxcao13 · commit d3452865c693 · 2025-01-07T08:05:26.000-08:00
diff --git a/vertical-pod-autoscaler/pkg/updater/logic/updater.go b/vertical-pod-autoscaler/pkg/updater/logic/updater.go
@@ -381,7 +381,7 @@ func (u *updater) AttemptInPlaceScalingIfPossible(ctx context.Context, vpaSize i
 		return
 	}
 
-	if vpa_api_util.GetUpdateMode(vpa) == vpa_types.UpdateModeInPlaceOrRecreate || vpa_api_util.GetUpdateMode(vpa) == vpa_types.UpdateModeAuto {
+	if vpa_api_util.GetUpdateMode(vpa) == vpa_types.UpdateModeInPlaceOrRecreate {
 
 		// separate counters/stats for in-place updates
 		withInPlaceUpdatable := false

Original file line number	Diff line number	Diff line change
`@@ -381,7 +381,7 @@ func (u *updater) AttemptInPlaceScalingIfPossible(ctx context.Context, vpaSize i`
`381`	`381`	`return`
`382`	`382`	`}`
`383`	`383`
`384`		`- if vpa_api_util.GetUpdateMode(vpa) == vpa_types.UpdateModeInPlaceOrRecreate \|\| vpa_api_util.GetUpdateMode(vpa) == vpa_types.UpdateModeAuto {`
	`384`	`+ if vpa_api_util.GetUpdateMode(vpa) == vpa_types.UpdateModeInPlaceOrRecreate {`
`385`	`385`
`386`	`386`	`// separate counters/stats for in-place updates`
`387`	`387`	`withInPlaceUpdatable := false`