fix resources ordering

Signed-off-by: Thibault Mange <[email protected]>

operators/endpointmetrics/controllers/observabilityendpoint/observabilityaddon_controller.go

@@ -264,6 +264,12 @@ func (r *ObservabilityAddonReconciler) Reconcile(ctx context.Context, req ctrl.R
 		}
 
 		deployer := deploying.NewDeployer(r.Client)
+
+		// Ordering resources to ensure they are applied in the correct order
+		slices.SortFunc(toDeploy, func(a, b *unstructured.Unstructured) bool {
+			return (resourcePriority(a) - resourcePriority(b)) < 0
+		})
+
 		for _, res := range toDeploy {
 			if res.GetNamespace() != namespace {
 				globalRes = append(globalRes, res)
@@ -622,3 +628,18 @@ func remove(list []string, s string) []string {
 	}
 	return result
 }
+
+// resourcePriority returns the priority of the resource.
+// This is used to order the resources to be created in the correct order.
+func resourcePriority(resource *unstructured.Unstructured) int {
+	switch resource.GetKind() {
+	case "Role", "ClusterRole":
+		return 1
+	case "RoleBinding", "ClusterRoleBinding":
+		return 2
+	case "CustomResourceDefinition":
+		return 3
+	default:
+		return 4
+	}
+}

operators/endpointmetrics/pkg/rendering/renderer.go

@@ -12,7 +12,6 @@ import (
 	"strings"
 
 	prometheusv1 "github.com/prometheus-operator/prometheus-operator/pkg/apis/monitoring/v1"
-	"golang.org/x/exp/slices"
 	v1 "k8s.io/api/apps/v1"
 	corev1 "k8s.io/api/core/v1"
 	"k8s.io/apimachinery/pkg/apis/meta/v1/unstructured"
@@ -251,27 +250,9 @@ func Render(
 		}
 	}
 
-	// Ordering resources to ensure they are applied in the correct order
-	slices.SortFunc(resources, func(a, b *unstructured.Unstructured) bool {
-		return (resourcePriority(a) - resourcePriority(b)) < 0
-	})
-
 	return resources, nil
 }
 
-func resourcePriority(resource *unstructured.Unstructured) int {
-	switch resource.GetKind() {
-	case "Role", "ClusterRole":
-		return 1
-	case "RoleBinding", "ClusterRoleBinding":
-		return 2
-	case "CustomResourceDefinition":
-		return 3
-	default:
-		return 4
-	}
-}
-
 func getDisabledMetrics(c runtimeclient.Client) (string, error) {
 	cm := &corev1.ConfigMap{}
 	err := c.Get(context.TODO(), types.NamespacedName{Name: operatorconfig.AllowlistConfigMapName,

operators/endpointmetrics/pkg/rendering/renderer_test.go

@@ -58,14 +58,6 @@ func TestRender(t *testing.T) {
 	}
 
 	printObjs(t, objs)
-
-	// ensure that objects are sorted
-	for i := 0; i < len(objs)-1; i++ {
-		if resourcePriority(objs[i]) > resourcePriority(objs[i+1]) {
-			t.Errorf("objects are not sorted")
-		}
-	}
-
 }
 
 func printObjs(t *testing.T, objs []*unstructured.Unstructured) {