chore: Refactor const usage

grafana · Jan 10, 2025 · e45f0eb · e45f0eb
1 parent fbc0b9e
commit e45f0eb
Show file tree

Hide file tree

Showing 5 changed files with 11 additions and 27 deletions.
diff --git a/controllers/controller_shared.go b/controllers/controller_shared.go
@@ -26,9 +26,12 @@ import (
 	"sigs.k8s.io/controller-runtime/pkg/predicate"
 )
 
-const grafanaFinalizer = "operator.grafana.com/finalizer"
-
 const (
+	// Synchronization size and timeout values
+	syncBatchSize    = 100
+	initialSyncDelay = 10 * time.Second
+	RequeueDelay     = 10 * time.Second
+
 	// condition types
 	conditionNoMatchingInstance = "NoMatchingInstance"
 	conditionNoMatchingFolder   = "NoMatchingFolder"
@@ -37,6 +40,9 @@ const (
 	// condition reasons
 	conditionApplySuccessful = "ApplySuccessful"
 	conditionApplyFailed     = "ApplyFailed"
+
+	// Finalizer
+	grafanaFinalizer = "operator.grafana.com/finalizer"
 )
 
 //+kubebuilder:rbac:groups=coordination.k8s.io,resources=leases,verbs=get;list;watch;create;update;patch;delete

diff --git a/controllers/dashboard_controller.go b/controllers/dashboard_controller.go
@@ -57,8 +57,6 @@ import (
 )
 
 const (
-	initialSyncDelay               = "10s"
-	syncBatchSize                  = 100
 	conditionDashboardSynchronized = "DashboardSynchronized"
 )
 
@@ -756,17 +754,12 @@ func (r *GrafanaDashboardReconciler) SetupWithManager(mgr ctrl.Manager, ctx cont
 		Complete(r)
 
 	if err == nil {
-		d, err := time.ParseDuration(initialSyncDelay)
-		if err != nil {
-			return err
-		}
-
 		go func() {
 			for {
 				select {
 				case <-ctx.Done():
 					return
-				case <-time.After(d):
+				case <-time.After(initialSyncDelay):
 					result, err := r.Reconcile(ctx, ctrl.Request{})
 					if err != nil {
 						r.Log.Error(err, "error synchronizing dashboards")

diff --git a/controllers/datasource_controller.go b/controllers/datasource_controller.go
@@ -390,17 +390,12 @@ func (r *GrafanaDatasourceReconciler) SetupWithManager(mgr ctrl.Manager, ctx con
 		Complete(r)
 
 	if err == nil {
-		d, err := time.ParseDuration(initialSyncDelay)
-		if err != nil {
-			return err
-		}
-
 		go func() {
 			for {
 				select {
 				case <-ctx.Done():
 					return
-				case <-time.After(d):
+				case <-time.After(initialSyncDelay):
 					result, err := r.Reconcile(ctx, ctrl.Request{})
 					if err != nil {
 						r.Log.Error(err, "error synchronizing datasources")

diff --git a/controllers/grafana_controller.go b/controllers/grafana_controller.go
@@ -24,7 +24,6 @@ import (
 	"os"
 	"reflect"
 	"strings"
-	"time"
 
 	"github.com/go-logr/logr"
 	"github.com/grafana/grafana-operator/v5/controllers/config"
@@ -45,10 +44,6 @@ import (
 	client2 "github.com/grafana/grafana-operator/v5/controllers/client"
 )
 
-const (
-	RequeueDelay = 10 * time.Second
-)
-
 // GrafanaReconciler reconciles a Grafana object
 type GrafanaReconciler struct {
 	client.Client

diff --git a/controllers/grafanafolder_controller.go b/controllers/grafanafolder_controller.go
@@ -256,17 +256,12 @@ func (r *GrafanaFolderReconciler) SetupWithManager(mgr ctrl.Manager, ctx context
 		Complete(r)
 
 	if err == nil {
-		d, err := time.ParseDuration(initialSyncDelay)
-		if err != nil {
-			return err
-		}
-
 		go func() {
 			for {
 				select {
 				case <-ctx.Done():
 					return
-				case <-time.After(d):
+				case <-time.After(initialSyncDelay):
 					result, err := r.Reconcile(ctx, ctrl.Request{})
 					if err != nil {
 						r.Log.Error(err, "error synchronizing folders")