From e49556e740e76da7da3194363384fa48db8d4db3 Mon Sep 17 00:00:00 2001 From: Konstantinos Karampogias Date: Fri, 8 May 2026 15:11:04 +0200 Subject: [PATCH 1/2] pf-status-relay-operator: migrate bundle to file-based catalog ci-operator emits deprecation warning: WARN: Building index images is deprecated and will be removed from ci-operator soon. See https://docs.ci.openshift.org/how-tos/ testing-operator-sdk-operators/#moving-to-file-based-catalog Fix: add skip_building_index: true to operator bundle definition. ci-index-operator-bundle job replaced by ci-bundle-operator-bundle which builds the bundle directly without generating an index image. e2e-validation already uses OO_BUNDLE so no test changes needed. Assisted-By: Claude --- .../openshift-pf-status-relay-operator-main.yaml | 1 + ...hift-pf-status-relay-operator-main-presubmits.yaml | 11 +++++------ 2 files changed, 6 insertions(+), 6 deletions(-) diff --git a/ci-operator/config/openshift/pf-status-relay-operator/openshift-pf-status-relay-operator-main.yaml b/ci-operator/config/openshift/pf-status-relay-operator/openshift-pf-status-relay-operator-main.yaml index 02ae9e2158814..993b89a4e675b 100644 --- a/ci-operator/config/openshift/pf-status-relay-operator/openshift-pf-status-relay-operator-main.yaml +++ b/ci-operator/config/openshift/pf-status-relay-operator/openshift-pf-status-relay-operator-main.yaml @@ -17,6 +17,7 @@ operator: bundles: - as: operator-bundle dockerfile_path: bundle.Dockerfile + skip_building_index: true substitutions: - pullspec: quay.io/openshift/origin-pf-status-relay-operator:.* with: pipeline:pf-status-relay-operator diff --git a/ci-operator/jobs/openshift/pf-status-relay-operator/openshift-pf-status-relay-operator-main-presubmits.yaml b/ci-operator/jobs/openshift/pf-status-relay-operator/openshift-pf-status-relay-operator-main-presubmits.yaml index 278e04a1ffcd6..71ccca396f8f1 100644 --- a/ci-operator/jobs/openshift/pf-status-relay-operator/openshift-pf-status-relay-operator-main-presubmits.yaml +++ b/ci-operator/jobs/openshift/pf-status-relay-operator/openshift-pf-status-relay-operator-main-presubmits.yaml @@ -5,21 +5,20 @@ presubmits: branches: - ^main$ - ^main- - cluster: build06 - context: ci/prow/ci-index-operator-bundle + context: ci/prow/ci-bundle-operator-bundle decorate: true labels: ci.openshift.io/generator: prowgen pj-rehearse.openshift.io/can-be-rehearsed: "true" - name: pull-ci-openshift-pf-status-relay-operator-main-ci-index-operator-bundle - rerun_command: /test ci-index-operator-bundle + name: pull-ci-openshift-pf-status-relay-operator-main-ci-bundle-operator-bundle + rerun_command: /test ci-bundle-operator-bundle spec: containers: - args: - --gcs-upload-secret=/secrets/gcs/service-account.json - --image-import-pull-secret=/etc/pull-secret/.dockerconfigjson - --report-credentials-file=/etc/report/credentials - - --target=ci-index-operator-bundle + - --target=operator-bundle command: - ci-operator image: quay-proxy.ci.openshift.org/openshift/ci:ci_ci-operator_latest @@ -52,7 +51,7 @@ presubmits: - name: result-aggregator secret: secretName: result-aggregator - trigger: (?m)^/test( | .* )ci-index-operator-bundle,?($|\s.*) + trigger: (?m)^/test( | .* )ci-bundle-operator-bundle,?($|\s.*) - agent: kubernetes always_run: true branches: From b3d723ffd9a9611e886913f20da426c543338c39 Mon Sep 17 00:00:00 2001 From: Konstantinos Karampogias Date: Fri, 8 May 2026 15:13:54 +0200 Subject: [PATCH 2/2] pf-status-relay-operator: capture operator logs in e2e-validation Operator pod logs were not preserved in CI artifacts. gather-extra collects all-namespaces but the operator namespace is cleaned up before collection reaches it, leaving the artifact directory empty. Fix: dump controller-manager logs to ARTIFACT_DIR at end of e2e-test step after deployment is confirmed Available. Logs are uploaded to GCS automatically and available in the job artifacts. Assisted-By: Claude --- .../openshift-pf-status-relay-operator-main.yaml | 1 + .../openshift-pf-status-relay-operator-main-presubmits.yaml | 1 + 2 files changed, 2 insertions(+) diff --git a/ci-operator/config/openshift/pf-status-relay-operator/openshift-pf-status-relay-operator-main.yaml b/ci-operator/config/openshift/pf-status-relay-operator/openshift-pf-status-relay-operator-main.yaml index 993b89a4e675b..9a2cd81cdb151 100644 --- a/ci-operator/config/openshift/pf-status-relay-operator/openshift-pf-status-relay-operator-main.yaml +++ b/ci-operator/config/openshift/pf-status-relay-operator/openshift-pf-status-relay-operator-main.yaml @@ -72,6 +72,7 @@ tests: cli: latest commands: | oc wait --for condition=Available -n openshift-pf-status-relay-operator deployment pf-status-relay-operator-controller-manager + oc logs -n openshift-pf-status-relay-operator deployment/pf-status-relay-operator-controller-manager --all-containers > "${ARTIFACT_DIR}/operator-controller-manager.log" 2>&1 || true from: src resources: requests: diff --git a/ci-operator/jobs/openshift/pf-status-relay-operator/openshift-pf-status-relay-operator-main-presubmits.yaml b/ci-operator/jobs/openshift/pf-status-relay-operator/openshift-pf-status-relay-operator-main-presubmits.yaml index 71ccca396f8f1..362b6e414eea5 100644 --- a/ci-operator/jobs/openshift/pf-status-relay-operator/openshift-pf-status-relay-operator-main-presubmits.yaml +++ b/ci-operator/jobs/openshift/pf-status-relay-operator/openshift-pf-status-relay-operator-main-presubmits.yaml @@ -5,6 +5,7 @@ presubmits: branches: - ^main$ - ^main- + cluster: build06 context: ci/prow/ci-bundle-operator-bundle decorate: true labels: