Create MAX_CONCURRENT_RECONCILES env var

percona · Apr 20, 2023 · d64e70d · d64e70d
1 parent 08bc3d8
commit d64e70d
Show file tree

Hide file tree

Showing 7 changed files with 1,216 additions and 1 deletion.
diff --git a/config/crd/bases/pxc.percona.com_perconaxtradbclusters.yaml b/config/crd/bases/pxc.percona.com_perconaxtradbclusters.yaml
diff --git a/deploy/bundle.yaml b/deploy/bundle.yaml
diff --git a/deploy/crd.yaml b/deploy/crd.yaml
diff --git a/deploy/cw-bundle.yaml b/deploy/cw-bundle.yaml
diff --git a/deploy/cw-operator.yaml b/deploy/cw-operator.yaml
@@ -42,6 +42,8 @@ spec:
           value: percona-xtradb-cluster-operator
         - name: DISABLE_TELEMETRY
           value: "false"
+        - name: MAX_CONCURRENT_RECONCILES
+          value: "20"
         image: perconalab/percona-xtradb-cluster-operator:main
         imagePullPolicy: Always
         resources:

diff --git a/deploy/operator.yaml b/deploy/operator.yaml
@@ -45,6 +45,8 @@ spec:
           value: percona-xtradb-cluster-operator
         - name: DISABLE_TELEMETRY
           value: "false"
+        - name: MAX_CONCURRENT_RECONCILES
+          value: "20"
         image: perconalab/percona-xtradb-cluster-operator:main
         imagePullPolicy: Always
         livenessProbe:

diff --git a/pkg/controller/pxc/controller.go b/pkg/controller/pxc/controller.go
@@ -4,7 +4,9 @@ import (
 	"context"
 	"encoding/base64"
 	"encoding/json"
+	"os"
 	"reflect"
+	"strconv"
 	"strings"
 	"sync"
 	"sync/atomic"
@@ -81,10 +83,31 @@ func newReconciler(mgr manager.Manager) (reconcile.Reconciler, error) {
 	}, nil
 }
 
+const (
+	MaxConcurrentReconcilesEnvVar = "MAX_CONCURRENT_RECONCILES"
+)
+
+func getMaxConcurrentReconciles() (int, error) {
+	s, found := os.LookupEnv(MaxConcurrentReconcilesEnvVar)
+	if !found {
+		return 0, errors.Errorf("%s must be set", MaxConcurrentReconcilesEnvVar)
+	}
+
+	value, err := strconv.Atoi(s)
+	if err != nil {
+		return 0, errors.Wrapf(err, "%s must be int", MaxConcurrentReconcilesEnvVar)
+	}
+	return value, nil
+}
+
 // add adds a new Controller to mgr with r as the reconcile.Reconciler
 func add(mgr manager.Manager, r reconcile.Reconciler) error {
+	mcr, err := getMaxConcurrentReconciles()
+	if err != nil {
+		return err
+	}
 	// Create a new controller
-	c, err := controller.New("perconaxtradbcluster-controller", mgr, controller.Options{MaxConcurrentReconciles: 20, Reconciler: r})
+	c, err := controller.New("perconaxtradbcluster-controller", mgr, controller.Options{MaxConcurrentReconciles: mcr, Reconciler: r})
 	if err != nil {
 		return err
 	}