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

jkyros · jkyros · commit b75b2b51e153 · 2024-11-14T16:04:37.000-06:00
diff --git a/vertical-pod-autoscaler/pkg/updater/logic/updater.go b/vertical-pod-autoscaler/pkg/updater/logic/updater.go
@@ -380,7 +380,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
`@@ -380,7 +380,7 @@ func (u *updater) AttemptInPlaceScalingIfPossible(ctx context.Context, vpaSize i`
`380`	`380`	`return`
`381`	`381`	`}`
`382`	`382`
`383`		`- if vpa_api_util.GetUpdateMode(vpa) == vpa_types.UpdateModeInPlaceOrRecreate \|\| vpa_api_util.GetUpdateMode(vpa) == vpa_types.UpdateModeAuto {`
	`383`	`+ if vpa_api_util.GetUpdateMode(vpa) == vpa_types.UpdateModeInPlaceOrRecreate {`
`384`	`384`
`385`	`385`	`// separate counters/stats for in-place updates`
`386`	`386`	`withInPlaceUpdatable := false`