reconcile clusteroperator status more frequently (#3006)

sanchezl · web-flow · commit 0dbf79d28343 · 2023-08-23T17:37:20.000-04:00
If status is modified, the operator must set status to proper values. This ensures that accidents cannot permanently reset status and gives a clear indication that the operator is "live". This came up as important when operators were NOT live, during cert rotations and we had no indication of problems.

Signed-off-by: Luis Sanchez &lt;sanchezl@redhat.com&gt;
diff --git a/pkg/lib/operatorstatus/status.go b/pkg/lib/operatorstatus/status.go
@@ -228,7 +228,7 @@ func MonitorClusterStatus(name string, syncCh <-chan error, stopCh <-chan struct
 		// if we've reported success, we can sleep longer, otherwise we want to keep watching for
 		// successful
 		if successfulSyncs > 0 {
-			time.Sleep(5 * time.Minute)
+			time.Sleep(25 * time.Second)
 		}
 
 	}, 5*time.Second, stopCh)

Original file line number	Diff line number	Diff line change
`@@ -228,7 +228,7 @@ func MonitorClusterStatus(name string, syncCh <-chan error, stopCh <-chan struct`
`228`	`228`	`// if we've reported success, we can sleep longer, otherwise we want to keep watching for`
`229`	`229`	`// successful`
`230`	`230`	`if successfulSyncs > 0 {`
`231`		`- time.Sleep(5 * time.Minute)`
	`231`	`+ time.Sleep(25 * time.Second)`
`232`	`232`	`}`
`233`	`233`
`234`	`234`	`}, 5*time.Second, stopCh)`