Bug 1860889: increase QPS and burst settings for clients

Signed-off-by: Simon Pasquier <spasquie@redhat.com>

OWNERS

@@ -1,15 +1,13 @@
 component: "Monitoring"
 
 reviewers:
-- brancz
 - s-urbaniak
 - paulfantom
 - LiliC
 - pgier
 - simonpasquier
 
 approvers:
-- brancz
 - bparees
 - s-urbaniak
 - paulfantom
@@ -18,5 +16,6 @@ approvers:
 - simonpasquier
 
 emeritus_approvers:
+- brancz
 - squat
 - metalmatze

cmd/operator/main.go

@@ -155,6 +155,10 @@ func Main() int {
 		return 1
 	}
 
+	// CMO runs many tasks in parallel and the default values for rate limiting are too low.
+	config.QPS = 20
+	config.Burst = 50
+
 	userWorkloadConfigMapName := "user-workload-monitoring-config"
 	o, err := cmo.New(config, *releaseVersion, *namespace, *namespaceUserWorkload, *namespaceSelector, *configMapName, userWorkloadConfigMapName, *remoteWrite, images.asMap(), telemetryConfig.Matches)
 	if err != nil {

pkg/client/client.go

@@ -716,7 +716,7 @@ func (c *Client) WaitForDeploymentRollout(dep *appsv1.Deployment) error {
 		}
 		if d.Status.UnavailableReplicas != 0 {
 			lastErr = errors.Errorf("got %d unavailable replicas",
-				d.Status.UpdatedReplicas)
+				d.Status.UnavailableReplicas)
 			return false, nil
 		}
 		return true, nil