Enable autonodezing by default

Azure · Sep 11, 2023 · 99fff68 · 99fff68
1 parent e02f5e0
commit 99fff68
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 3 deletions.
diff --git a/pkg/api/defaults.go b/pkg/api/defaults.go
@@ -101,7 +101,7 @@ func DefaultOperatorFlags() OperatorFlags {
 		"aro.routefix.enabled":                     flagTrue,
 		"aro.storageaccounts.enabled":              flagTrue,
 		"aro.workaround.enabled":                   flagTrue,
-		"aro.autosizednodes.enabled":               flagFalse,
+		"aro.autosizednodes.enabled":               flagTrue,
 		"rh.srep.muo.enabled":                      flagTrue,
 		"rh.srep.muo.managed":                      flagTrue,
 		"aro.guardrails.enabled":                   flagFalse,

diff --git a/pkg/monitor/cluster/clusterflagsandbanner_test.go b/pkg/monitor/cluster/clusterflagsandbanner_test.go
@@ -5,6 +5,7 @@ package cluster
 
 import (
 	"context"
+	"fmt"
 	"reflect"
 	"testing"
 
@@ -54,12 +55,15 @@ func generateDefaultFlags() arov1alpha1.OperatorFlags {
 func generateNonStandardFlags(nonDefualtFlagNames []string) arov1alpha1.OperatorFlags {
 	nsf := make(arov1alpha1.OperatorFlags)
 	for k, v := range api.DefaultOperatorFlags() {
+		fmt.Println(k, v)
 		nsf[k] = v
 	}
 	for _, n := range nonDefualtFlagNames {
 		if nsf[n] == "true" {
+			fmt.Println(n, "false")
 			nsf[n] = "false"
 		} else {
+			fmt.Println(n, "false")
 			nsf[n] = "true"
 		}
 	}
@@ -126,7 +130,7 @@ func TestEmitOperatorFlagsAndSupportBanner(t *testing.T) {
 				"aro.imageconfig.enabled":    "false",
 				"aro.dnsmasq.enabled":        "false",
 				"aro.genevalogging.enabled":  "false",
-				"aro.autosizednodes.enabled": "true",
+				"aro.autosizednodes.enabled": "false",
 			},
 			expectBannerMetricsValue: 0,
 			expectBannerMetricsDims:  nil,
@@ -169,7 +173,7 @@ func TestEmitOperatorFlagsAndSupportBanner(t *testing.T) {
 				"aro.imageconfig.enabled":    "false",
 				"aro.dnsmasq.enabled":        "false",
 				"aro.genevalogging.enabled":  "false",
-				"aro.autosizednodes.enabled": "true",
+				"aro.autosizednodes.enabled": "false",
 			},
 			expectBannerMetricsValue: 1,
 			expectBannerMetricsDims:  map[string]string{"msg": "contact support"},