fix: set CNV_SUBSCRIPTION_SOURCE for kubevirt AWS e2e jobs#78056
fix: set CNV_SUBSCRIPTION_SOURCE for kubevirt AWS e2e jobs#78056bryan-cox wants to merge 1 commit intoopenshift:mainfrom
Conversation
PR openshift#77820 switched e2e-kubevirt-aws-ovn-reduced and e2e-kubevirt-aws-ovn from the hypershift-kubevirt-e2e-nested workflow to hypershift-kubevirt-e2e-aws. The nested workflow set CNV_SUBSCRIPTION_SOURCE: redhat-operators in its steps.env, but the aws workflow does not. Without this override, the hypershift-kubevirt-install step ref defaults to cnv-prerelease-catalog-source, which causes the install script to derive a nightly catalog image tag from the OCP cluster version (quay.io/openshift-cnv/nightly-catalog:5.0). This image does not exist, so the CatalogSource pod enters ImagePullBackOff, the OLM subscription never resolves, and the step fails after exhausting 30 retries. Set CNV_SUBSCRIPTION_SOURCE: redhat-operators explicitly in both kubevirt AWS job configs so the install script uses the stable channel from the redhat-operators catalog instead of the nonexistent nightly prerelease catalog image. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
|
@bryan-cox: GitHub didn't allow me to request PR reviews from the following users: openshift/hypershift-maintainers. Note that only openshift members and repo collaborators can review this PR, and authors cannot review their own PRs. DetailsIn response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository: openshift/coderabbit/.coderabbit.yaml Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
WalkthroughEnvironment variable Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes 🚥 Pre-merge checks | ✅ 10✅ Passed checks (10 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
|
/pj-rehearse pull-ci-openshift-hypershift-main-e2e-kubevirt-aws-ovn-reduced |
|
@bryan-cox: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel. |
|
[REHEARSALNOTIFIER]
Prior to this PR being merged, you will need to either run and acknowledge or opt to skip these rehearsals. Interacting with pj-rehearseComment: Once you are satisfied with the results of the rehearsals, comment: |
|
/pj-rehearse pull-ci-openshift-hypershift-main-e2e-kubevirt-aws-ovn |
|
@bryan-cox: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel. |
|
/lgtm |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: bryan-cox, sdminonne The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
|
@bryan-cox: The following tests failed, say
Full PR test history. Your PR dashboard. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here. |
|
/close |
|
@bryan-cox: Closed this PR. DetailsIn response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
Summary
CNV_SUBSCRIPTION_SOURCE: redhat-operatorsin bothe2e-kubevirt-aws-ovn-reducedande2e-kubevirt-aws-ovnjob configsRoot Cause
PR #77820 (merged 2026-04-20) switched these jobs from
workflow: hypershift-kubevirt-e2e-nestedtoworkflow: hypershift-kubevirt-e2e-aws. The old nested workflow setCNV_SUBSCRIPTION_SOURCE: redhat-operatorsin itssteps.env, but the new aws workflow does not.Without this override, the
hypershift-kubevirt-installstep ref defaults tocnv-prerelease-catalog-source. Since the jobs aren't triggered via gangway API,CNV_PRERELEASE_CATALOG_IMAGEandCNV_SUBSCRIPTION_CHANNELare both empty, causing the install script to fall through to theelsebranch:The IPI cluster runs OCP 5.0, so this constructs
quay.io/openshift-cnv/nightly-catalog:5.0— an image tag that does not exist. The CatalogSource pod entersImagePullBackOff, the OLM subscription never resolves a CSV, and the step exhausts 30 retries over ~15 minutes before failing.Evidence
All three most recent runs of
e2e-kubevirt-aws-ovn-reduced(against PRs #8278, #8288, #8247) show identical failure inhypershift-kubevirt-install:No e2e tests ever ran — the failure is entirely in the pre-phase.
Fix
Explicitly set
CNV_SUBSCRIPTION_SOURCE: redhat-operatorsin both job configs, matching the pattern already used bye2e-azure-kubevirt-ovn. This causes the install script to use thestablechannel from theredhat-operatorscatalog instead of the nonexistent nightly prerelease catalog image.Job History
/cc @openshift/hypershift-maintainers
🤖 Generated with Claude Code
Summary by CodeRabbit