fix(otel-upstream): use opentelemetry-operator-system namespace in upstream step (#79419)

* fix(otel-upstream): use opentelemetry-operator-system namespace

The OpenTelemetry Operator upstream e2e tests now install the operator
into opentelemetry-operator-system (the OLM default) instead of the old
opentelemetry-operator namespace. Update all oc -n and CSV patch
commands in the upstream step to match.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

* fix(otel-upstream): use opentelemetry-operator-system in job configs

Update the operator install namespace from opentelemetry-operator to
opentelemetry-operator-system in the upstream OCP 4.12, 4.21, and 4.22
job configs. Aligns with the step script change and the upstream
operator repo standardization.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

* fix(otel-upstream): revert PROJECT_NAME to opentelemetry-operator

PROJECT_NAME is consumed by the Snyk SAST step as the Snyk project
name, not as an OCP namespace. Revert the inadvertent rename.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

* Run Tempo TLS profile tests last as an isolated chainsaw invocation

The TLS profile tests patch the cluster-wide APIServer resource, triggering
node-level TLS reconciliation that disrupts concurrently running tests. Run
them as a separate chainsaw invocation after all other suites complete,
matching the same any_errors pattern used in the OTel upstream test step.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

---------

Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>

Author: IshwarKanse

ci-operator/config/openshift/open-telemetry-opentelemetry-operator/openshift-open-telemetry-opentelemetry-operator-main__upstream-ocp-4.12-amd64.yaml

@@ -128,10 +128,10 @@ tests:
     - as: install
       cli: latest
       commands: |
-        oc create namespace opentelemetry-operator
-        oc label namespace opentelemetry-operator openshift.io/cluster-monitoring="true"
-        operator-sdk run bundle --timeout=5m --security-context-config restricted -n opentelemetry-operator "$OO_BUNDLE"
-        oc wait --for condition=Available -n opentelemetry-operator deployment opentelemetry-operator-controller-manager
+        oc create namespace opentelemetry-operator-system
+        oc label namespace opentelemetry-operator-system openshift.io/cluster-monitoring="true"
+        operator-sdk run bundle --timeout=5m --security-context-config restricted -n opentelemetry-operator-system "$OO_BUNDLE"
+        oc wait --for condition=Available -n opentelemetry-operator-system deployment opentelemetry-operator-controller-manager
       dependencies:
       - env: OO_BUNDLE
         name: opentelemetry-bundle

ci-operator/config/openshift/open-telemetry-opentelemetry-operator/openshift-open-telemetry-opentelemetry-operator-main__upstream-ocp-4.21-amd64.yaml

@@ -118,10 +118,10 @@ tests:
     - as: install
       cli: latest
       commands: |
-        oc create namespace opentelemetry-operator
-        oc label namespace opentelemetry-operator openshift.io/cluster-monitoring="true"
-        operator-sdk run bundle --timeout=5m --security-context-config restricted -n opentelemetry-operator "$OO_BUNDLE"
-        oc wait --for condition=Available -n opentelemetry-operator deployment opentelemetry-operator-controller-manager
+        oc create namespace opentelemetry-operator-system
+        oc label namespace opentelemetry-operator-system openshift.io/cluster-monitoring="true"
+        operator-sdk run bundle --timeout=5m --security-context-config restricted -n opentelemetry-operator-system "$OO_BUNDLE"
+        oc wait --for condition=Available -n opentelemetry-operator-system deployment opentelemetry-operator-controller-manager
       dependencies:
       - env: OO_BUNDLE
         name: opentelemetry-bundle

ci-operator/config/openshift/open-telemetry-opentelemetry-operator/openshift-open-telemetry-opentelemetry-operator-main__upstream-ocp-4.22-amd64.yaml

@@ -124,10 +124,10 @@ tests:
     - as: install
       cli: latest
       commands: |
-        oc create namespace opentelemetry-operator
-        oc label namespace opentelemetry-operator openshift.io/cluster-monitoring="true"
-        operator-sdk run bundle --timeout=5m --security-context-config restricted -n opentelemetry-operator "$OO_BUNDLE"
-        oc wait --for condition=Available -n opentelemetry-operator deployment opentelemetry-operator-controller-manager
+        oc create namespace opentelemetry-operator-system
+        oc label namespace opentelemetry-operator-system openshift.io/cluster-monitoring="true"
+        operator-sdk run bundle --timeout=5m --security-context-config restricted -n opentelemetry-operator-system "$OO_BUNDLE"
+        oc wait --for condition=Available -n opentelemetry-operator-system deployment opentelemetry-operator-controller-manager
       dependencies:
       - env: OO_BUNDLE
         name: opentelemetry-bundle

ci-operator/step-registry/distributed-tracing/tests/opentelemetry/upstream/distributed-tracing-tests-opentelemetry-upstream-commands.sh

@@ -47,13 +47,13 @@ fi
 any_errors=false
 
 # Set the operator environment variables required for tests execution.
-OTEL_CSV_NAME=$(oc get csv -n opentelemetry-operator | grep "opentelemetry-operator" | awk '{print $1}')
-oc -n opentelemetry-operator patch csv $OTEL_CSV_NAME --type=json -p '[
+OTEL_CSV_NAME=$(oc get csv -n opentelemetry-operator-system | grep "opentelemetry-operator" | awk '{print $1}')
+oc -n opentelemetry-operator-system patch csv $OTEL_CSV_NAME --type=json -p '[
   {"op":"add","path":"/spec/install/spec/deployments/0/spec/template/spec/containers/0/env/-","value":{"name":"RELATED_IMAGE_TARGET_ALLOCATOR","value":"'"${TARGETALLOCATOR_IMG}"'"}},
   {"op":"add","path":"/spec/install/spec/deployments/0/spec/template/spec/containers/0/env/-","value":{"name":"RELATED_IMAGE_OPERATOR_OPAMP_BRIDGE","value":"'"${OPERATOROPAMPBRIDGE_IMG}"'"}}
 ]'
 sleep 60
-if oc -n opentelemetry-operator get deployment opentelemetry-operator-controller-manager -o jsonpath='{.status.conditions[?(@.type=="Available")].status}' | grep -q "True"; then
+if oc -n opentelemetry-operator-system get deployment opentelemetry-operator-controller-manager -o jsonpath='{.status.conditions[?(@.type=="Available")].status}' | grep -q "True"; then
     echo "Operator deployment updated successfully, continuing script execution..."
 else
     echo "Operator deployment update failed, exiting with error."
@@ -98,13 +98,13 @@ tests/e2e-prometheuscr \
 tests/e2e-sidecar || any_errors=true
 
 # Set the operator environment variables for metadata filters tests.
-OTEL_CSV_NAME=$(oc get csv -n opentelemetry-operator | grep "opentelemetry-operator" | awk '{print $1}')
-oc -n opentelemetry-operator patch csv $OTEL_CSV_NAME --type=json -p '[
+OTEL_CSV_NAME=$(oc get csv -n opentelemetry-operator-system | grep "opentelemetry-operator" | awk '{print $1}')
+oc -n opentelemetry-operator-system patch csv $OTEL_CSV_NAME --type=json -p '[
   {"op":"add","path":"/spec/install/spec/deployments/0/spec/template/spec/containers/0/env/-","value":{"name":"ANNOTATIONS_FILTER","value":".*filter.out,config.*.gke.io.*"}},
   {"op":"add","path":"/spec/install/spec/deployments/0/spec/template/spec/containers/0/env/-","value":{"name":"LABELS_FILTER","value":".*filter.out"}}
 ]'
 sleep 60
-if oc -n opentelemetry-operator get deployment opentelemetry-operator-controller-manager -o jsonpath='{.status.conditions[?(@.type=="Available")].status}' | grep -q "True"; then
+if oc -n opentelemetry-operator-system get deployment opentelemetry-operator-controller-manager -o jsonpath='{.status.conditions[?(@.type=="Available")].status}' | grep -q "True"; then
     echo "Operator deployment updated successfully for metadata filters, continuing script execution..."
 else
     echo "Operator deployment update for metadata filters failed, exiting with error."
@@ -121,16 +121,16 @@ chainsaw test \
 tests/e2e-metadata-filters || any_errors=true
 
 # Set the operator environment variables with instrumentation images for e2e-instrumentation tests.
-OTEL_CSV_NAME=$(oc get csv -n opentelemetry-operator | grep "opentelemetry-operator" | awk '{print $1}')
-oc -n opentelemetry-operator patch csv $OTEL_CSV_NAME --type=json -p '[
+OTEL_CSV_NAME=$(oc get csv -n opentelemetry-operator-system | grep "opentelemetry-operator" | awk '{print $1}')
+oc -n opentelemetry-operator-system patch csv $OTEL_CSV_NAME --type=json -p '[
   {"op":"add","path":"/spec/install/spec/deployments/0/spec/template/spec/containers/0/env/-","value":{"name":"RELATED_IMAGE_AUTO_INSTRUMENTATION_JAVA","value":"'"${INSTRUMENTATION_JAVA_IMG}"'"}},
   {"op":"add","path":"/spec/install/spec/deployments/0/spec/template/spec/containers/0/env/-","value":{"name":"RELATED_IMAGE_AUTO_INSTRUMENTATION_NODEJS","value":"'"${INSTRUMENTATION_NODEJS_IMG}"'"}},
   {"op":"add","path":"/spec/install/spec/deployments/0/spec/template/spec/containers/0/env/-","value":{"name":"RELATED_IMAGE_AUTO_INSTRUMENTATION_PYTHON","value":"'"${INSTRUMENTATION_PYTHON_IMG}"'"}},
   {"op":"add","path":"/spec/install/spec/deployments/0/spec/template/spec/containers/0/env/-","value":{"name":"RELATED_IMAGE_AUTO_INSTRUMENTATION_DOTNET","value":"'"${INSTRUMENTATION_DOTNET_IMG}"'"}},
   {"op":"add","path":"/spec/install/spec/deployments/0/spec/template/spec/containers/0/env/-","value":{"name":"RELATED_IMAGE_AUTO_INSTRUMENTATION_APACHE_HTTPD","value":"'"${INSTRUMENTATION_APACHE_HTTPD_IMG}"'"}}
 ]'
 sleep 60
-if oc -n opentelemetry-operator get deployment opentelemetry-operator-controller-manager -o jsonpath='{.status.conditions[?(@.type=="Available")].status}' | grep -q "True"; then
+if oc -n opentelemetry-operator-system get deployment opentelemetry-operator-controller-manager -o jsonpath='{.status.conditions[?(@.type=="Available")].status}' | grep -q "True"; then
     echo "Operator deployment updated successfully with instrumentation images, continuing script execution..."
 else
     echo "Operator deployment update with instrumentation images failed, exiting with error."

ci-operator/step-registry/distributed-tracing/tests/tempo/upstream/distributed-tracing-tests-tempo-upstream-commands.sh

@@ -35,6 +35,9 @@ fi
 # Unset environment variable which conflicts with Chainsaw
 unset NAMESPACE
 
+# Initialize a variable to keep track of errors
+any_errors=false
+
 # Execute Tempo e2e tests
 chainsaw test \
 --quiet \
@@ -50,5 +53,23 @@ tests/e2e-openshift-ossm \
 tests/e2e-openshift-object-stores \
 tests/e2e-long-running \
 tests/e2e-openshift-tshirt-sizes \
-tests/operator-metrics \
-tests/e2e-openshift-tls-profile
+tests/operator-metrics || any_errors=true
+
+# Execute TLS profile tests last: they patch the cluster-wide APIServer resource,
+# triggering node-level TLS reconciliation that would disrupt concurrently running tests.
+chainsaw test \
+--quiet \
+--config .chainsaw-openshift.yaml \
+--report-name "junit_tempo_e2e_tls_profile" \
+--report-path "$ARTIFACT_DIR" \
+--report-format "XML" \
+--test-dir \
+tests/e2e-openshift-tls-profile || any_errors=true
+
+# Check if any errors occurred
+if $any_errors; then
+  echo "Tests failed, check the logs for more details."
+  exit 1
+else
+  echo "All the tests passed."
+fi