fix topology-rbac and kubernetes-rbac tests in k8s jobs (#4656)

Author: sanketpathak

.ci/pipelines/lib/operators.sh

@@ -109,6 +109,25 @@ operator::install_pipelines() {
   return 0
 }
 
+# Install Tekton Pipelines (alternative to OpenShift Pipelines for Kubernetes)
+operator::install_tekton() {
+  local display_name="tekton-pipelines-webhook"
+  local ns="tekton-pipelines"
+
+  if ! kubectl get namespace "${ns}" &> /dev/null; then
+    log::info "Tekton Pipelines is not installed. Installing..."
+    kubectl apply -f https://storage.googleapis.com/tekton-releases/pipeline/latest/release.yaml
+  else
+    log::info "Tekton Pipelines namespace exists; ensuring webhook is ready before continuing."
+  fi
+
+  log::info "Waiting for Tekton Pipeline CRD and webhook (required before applying Pipeline manifests)..."
+  k8s_wait::crd "pipelines.tekton.dev" 120 5 || return 1
+  k8s_wait::deployment "${ns}" "${display_name}" 30 10 || return 1
+  k8s_wait::endpoint "${display_name}" "${ns}" 1800 10 || return 1
+  return 0
+}
+
 # Install Operator Lifecycle Manager if not present
 operator::install_olm() {
   if operator-sdk olm status > /dev/null 2>&1; then

.ci/pipelines/utils.sh

@@ -372,17 +372,13 @@ apply_yaml_files() {
   common::create_configmap_from_file "dynamic-global-header-config" "$project" \
     "dynamic-global-header-config.yaml" "$dir/resources/config_map/dynamic-global-header-config.yaml"
 
-  # Skip Topology resources for K8s deployments (AKS/EKS/GKE)
-  if [[ "$JOB_NAME" != *"aks"* && "$JOB_NAME" != *"eks"* && "$JOB_NAME" != *"gke"* ]]; then
-    # Create Deployment and Pipeline for Topology test.
-    oc apply -f "$dir/resources/topology_test/topology-test.yaml" --namespace="${project}"
-    if [[ -z "${IS_OPENSHIFT}" || "${IS_OPENSHIFT}" == "false" ]]; then
-      kubectl apply -f "$dir/resources/topology_test/topology-test-ingress.yaml" --namespace="${project}"
-    else
-      oc apply -f "$dir/resources/topology_test/topology-test-route.yaml" --namespace="${project}"
-    fi
+  # Create Deployment and Pipeline for Topology test.
+  oc apply -f "$dir/resources/topology_test/topology-test.yaml" --namespace="${project}"
+
+  if [[ -z "${IS_OPENSHIFT}" || "${IS_OPENSHIFT}" == "false" ]]; then
+    kubectl apply -f "$dir/resources/topology_test/topology-test-ingress.yaml" --namespace="${project}"
   else
-    log::info "Skipping Topology resources for K8s deployment (${JOB_NAME})"
+    oc apply -f "$dir/resources/topology_test/topology-test-route.yaml" --namespace="${project}"
   fi
 
   rm -rf "${tmpdir}"
@@ -478,12 +474,16 @@ cluster_setup_ocp_operator() {
 
 cluster_setup_k8s_operator() {
   operator::install_olm
+  # Install Tekton Pipelines for K8s deployments (AKS/EKS/GKE) for topology tests to work
+  operator::install_tekton
   # operator::install_postgres_k8s # Works with K8s but disabled in values file
 }
 
 cluster_setup_k8s_helm() {
-  log::info "No additional cluster setup required for K8s Helm deployment"
+  # Install Tekton Pipelines for K8s deployments (AKS/EKS/GKE) for topology tests to work
+  log::info "Installing Tekton Pipelines for K8s Helm deployment"
   # operator::install_olm
+  operator::install_tekton
   # operator::install_postgres_k8s # Works with K8s but disabled in values file
 }
 

e2e-tests/playwright/e2e/plugins/kubernetes/kubernetes-rbac.spec.ts

@@ -6,9 +6,6 @@ import { KUBERNETES_COMPONENTS } from "../../../support/page-objects/page-obj";
 import { KubernetesPage } from "../../../support/pages/kubernetes";
 
 test.describe("Test Kubernetes Plugin", () => {
-  // TODO: https://issues.redhat.com/browse/RHDHBUGS-2817
-  test.fixme(() => process.env.IS_OPENSHIFT === "false");
-
   let common: Common;
   let uiHelper: UIhelper;
   let catalog: Catalog;

e2e-tests/playwright/e2e/plugins/topology/topology-rbac.spec.ts

@@ -5,9 +5,6 @@ import { Catalog } from "../../../support/pages/catalog";
 import { Topology } from "../../../support/pages/topology";
 
 test.describe("Test Topology Plugin with RBAC", () => {
-  // TODO: https://issues.redhat.com/browse/RHDHBUGS-2817
-  test.fixme(() => process.env.IS_OPENSHIFT === "false");
-
   let common: Common;
   let uiHelper: UIhelper;
   let catalog: Catalog;