Make --early-readiness the default (#28)

Co-authored-by: Moritz Clasmeier <mclasmeier@redhat.com>

Author: mclasmeier

cmd/main.go

@@ -43,7 +43,7 @@ Red Hat Advanced Cluster Security (ACS) on any Kubernetes/OpenShift cluster.`,
 
 func init() {
 	rootCmd.PersistentFlags().BoolVarP(&verbose, "verbose", "v", false, "Enable verbose output (show CRs and Helm values)")
-	rootCmd.PersistentFlags().BoolVar(&earlyReadiness, "early-readiness", false, "Only wait for essential workloads (central/sensor), not all workloads")
+	rootCmd.PersistentFlags().BoolVar(&earlyReadiness, "early-readiness", true, "Only wait for essential workloads (central/sensor) to be ready")
 	rootCmd.AddCommand(newDeployCmd())
 	rootCmd.AddCommand(newTeardownCmd())
 	rootCmd.AddCommand(newVersionCmd())

tests/e2e/e2e_test.go

@@ -23,9 +23,10 @@ const (
 )
 
 var (
-	commonDeployArgs              = []string{"--port-forwarding", "--exposure=none", "--resources=small", "--early-readiness"}
-	commonDeployArgsNoPortForward = []string{"--exposure=loadbalancer", "--resources=small", "--early-readiness"}
-	roxieBinary                   string
+	commonDeployArgs              = []string{"--port-forwarding", "--exposure=none", "--resources=small"}
+	commonDeployArgsNoPortForward = []string{"--exposure=loadbalancer", "--resources=small"}
+
+	roxieBinary string
 )
 
 func TestMain(m *testing.M) {
@@ -293,7 +294,7 @@ func TestDeployBothComponentsTogether(t *testing.T) {
 
 	t.Log("=== Deploying both components ===")
 	// We also test --pause-reconciliation flag here.
-	args := append([]string{roxieBinary, "deploy", "both", "--early-readiness", "--pause-reconciliation", "--envrc", envrcPath}, commonDeployArgsNoPortForward...)
+	args := append([]string{roxieBinary, "deploy", "both", "--pause-reconciliation", "--envrc", envrcPath}, commonDeployArgsNoPortForward...)
 	runCommand(t, deployTimeout*2, nil, args...)
 
 	t.Log("Verifying namespace: acs-central")