openshift/os: add YUM_REPO_NAMES and EXTENSIONS_YUM_REPO_NAMES to extensions images#81704
Conversation
…names The openshift/os repo switched its c10s.repo to point directly at mirror.stream.centos.org URLs (commit 64af4d8 in openshift/os#1945), making the separate c10s-mirror.repo and its -mirror suffixed repo names obsolete. Update YUM_REPO_NAMES for the stream-coreos node image build to match: c10s-baseos and c10s-appstream instead of c10s-baseos-mirror and c10s-appstream-mirror. Assisted-by: <anthropic/claude-opus-4.6>
…ensions images openshift/os#1945 refactors the extensions build to use dnf download instead of rpm-ostree compose. The extensions Containerfile now requires two build args: - YUM_REPO_NAMES: base repos (same as the node image build) - EXTENSIONS_YUM_REPO_NAMES: additional repos needed only for extensions (highavailability for two-node-ha, nfv for kernel-rt) The build script combines them at runtime: repo_list="${YUM_REPO_NAMES},${EXTENSIONS_YUM_REPO_NAMES}" Add both build args to all extensions image definitions. Assisted-by: <anthropic/claude-opus-4.6>
|
/pj-rehearse auto-ack |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository YAML (base), Central YAML (inherited) Review profile: CHILL Plan: Enterprise Run ID: 📒 Files selected for processing (5)
WalkthroughThis PR adds ChangesExtensions repo build args
Estimated code review effort: 1 (Trivial) | ~5 minutes Possibly related PRs
Suggested labels: 🚥 Pre-merge checks | ✅ 15✅ Passed checks (15 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
openshift-ci
Bot
added
the
approved
|
[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: |
|
@dustymabe: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel. |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: dustymabe, joelcapitao 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 |
|
/override ci/rehearse/openshift/os/master/okd-scos-images These are going to fail until openshift/os#1945 merges anyway so we can ignore them here. |
|
@dustymabe: dustymabe unauthorized: /override is restricted to Repo administrators, approvers in top level OWNERS file, and the following github teams:openshift: openshift-release-oversight openshift-staff-engineers openshift-sustaining-engineers. 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. |
|
/pj-rehearse ack |
|
@dustymabe: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel. |
openshift-merge-bot
Bot
added
the
rehearsals-ack
openshift-merge-bot
Bot
merged commit 0669011
into
openshift:main
|
@dustymabe: 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. |
2 participants
openshift/os#1945 refactors the extensions build to use dnf download
instead of rpm-ostree compose. The extensions Containerfile now
requires two build args:
extensions (highavailability for two-node-ha, nfv for kernel-rt)
The build script combines them at runtime:
repo_list="${YUM_REPO_NAMES},${EXTENSIONS_YUM_REPO_NAMES}"
Add both build args to all extensions image definitions.
Assisted-by: <anthropic/claude-opus-4.6>
Obsoletes #81446
Summary by CodeRabbit
This updates OpenShift CI release and master configs for
openshift/osso extensions image builds receive the new repository inputs required by the revised build flow. In practice, the affected CI jobs now pass both the baseYUM_REPO_NAMESand the extensions-onlyEXTENSIONS_YUM_REPO_NAMESvalues, allowing builds for high-availability and NFV extensions to resolve repos correctly across the supported RHEL 9, RHEL 10, and OKD SCOS variants.It also updates the OKD SCOS node image configuration to use the new non-mirror CentOS stream repository names, matching the upstream
c10s.repochange and removing reliance on the old*-mirrorrepo naming.