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

jkyros · jkyros · commit 245a3cd33c0e · 2024-08-30T15:17:31.000-05:00
diff --git a/vertical-pod-autoscaler/pkg/updater/logic/updater.go b/vertical-pod-autoscaler/pkg/updater/logic/updater.go
@@ -371,7 +371,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
`@@ -371,7 +371,7 @@ func (u *updater) AttemptInPlaceScalingIfPossible(ctx context.Context, vpaSize i`
`371`	`371`	`return`
`372`	`372`	`}`
`373`	`373`
`374`		`- if vpa_api_util.GetUpdateMode(vpa) == vpa_types.UpdateModeInPlaceOrRecreate \|\| vpa_api_util.GetUpdateMode(vpa) == vpa_types.UpdateModeAuto {`
	`374`	`+ if vpa_api_util.GetUpdateMode(vpa) == vpa_types.UpdateModeInPlaceOrRecreate {`
`375`	`375`
`376`	`376`	`// separate counters/stats for in-place updates`
`377`	`377`	`withInPlaceUpdatable := false`