Merge pull request #366 from ritmun/allow-pods

remove operators from webhook check

Author: openshift-merge-bot[bot]

test/e2e/managed_cluster_validating_webhooks_test.go

@@ -5,7 +5,6 @@ package osde2etests
 
 import (
 	"context"
-	"slices"
 	"strconv"
 	"strings"
 	"time"
@@ -247,47 +246,6 @@ var _ = Describe("Managed Cluster Validating Webhooks", Ordered, func() {
 			Expect(err).NotTo(HaveOccurred())
 		}, SpecTimeout(createPodWaitDuration.Seconds()+deletePodWaitDuration.Seconds()))
 
-		It("prevents operators workloads from being scheduled on worker or master nodes", func(ctx context.Context) {
-			operators := map[string]string{
-				"cloud-ingress-operator":          "openshift-cloud-ingress-operator",
-				"configure-alertmanager-operator": "openshift-monitoring",
-				"custom-domains-operator":         "openshift-custom-domains-operator",
-				"managed-velero-operator":         "openshift-velero",
-				"must-gather-operator":            "openshift-must-gather-operator",
-				"osd-metrics-exporter":            "openshift-osd-metrics",
-				"rbac-permissions-operator":       "openshift-rbac-permissions",
-			}
-
-			var podList v1.PodList
-			err := client.WithNamespace(metav1.NamespaceAll).List(ctx, &podList)
-			Expect(err).ShouldNot(HaveOccurred(), "unable to list pods")
-			Expect(len(podList.Items)).To(BeNumerically(">", 0), "found no pods")
-
-			var nodeList v1.NodeList
-			selectInfraNodes := resources.WithLabelSelector(labels.FormatLabels(map[string]string{"node-role.kubernetes.io": "infra"}))
-			err = client.List(ctx, &nodeList, selectInfraNodes)
-			Expect(err).ShouldNot(HaveOccurred(), "unable to list infra nodes ")
-
-			nodeNames := []string{}
-			for _, node := range nodeList.Items {
-				nodeNames = append(nodeNames, node.GetName())
-			}
-
-			violators := []string{}
-			for _, pod := range podList.Items {
-				for operator, namespace := range operators {
-					if pod.GetNamespace() != namespace {
-						continue
-					}
-					if strings.HasPrefix(pod.GetName(), operator) && !strings.HasPrefix(pod.GetName(), operator+"-registry") {
-						if !slices.Contains(nodeNames, pod.Spec.NodeName) {
-							violators = append(violators, pod.GetNamespace()+"/"+pod.GetName())
-						}
-					}
-				}
-			}
-			Expect(violators).To(HaveLen(0), "found pods in violation %v", violators)
-		})
 	})
 
 	Describe("sre-techpreviewnoupgrade-validation", func() {