Add EXTENSIONS_YUM_REPO_NAMES build arg to openshift/os extensions images#81446
Add EXTENSIONS_YUM_REPO_NAMES build arg to openshift/os extensions images#81446redhat-chai-bot wants to merge 1 commit into
Conversation
…ages Add the EXTENSIONS_YUM_REPO_NAMES build arg to the rhel-coreos-extensions and rhel-coreos-10-extensions image definitions in both master and release-5.0 ci-operator configs. This enables the extensions Containerfile to install packages from the correct yum repos during CI builds. RHEL 9 extensions: rhel-9.8-baseos,rhel-9.8-appstream,rhel-9.8-server-ose-5.0,rhel-9.8-highavailability,rhel-9.8-fast-datapath RHEL 10 extensions: rhel-10.2-baseos,rhel-10.2-appstream,rhel-10.2-server-ose-5.0,rhel-10.2-highavailability,rhel-10.2-fast-datapath Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
|
@redhat-chai-bot, Interacting with pj-rehearseComment: Once you are satisfied with the results of the rehearsals, comment: |
|
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 (2)
WalkthroughTwo CI operator configuration YAML files are updated to add a new ChangesExtensions YUM repo config update
Estimated code review effort: 1 (Trivial) | ~3 minutes Related issues: None specified Related PRs: None specified Suggested labels: None specified Suggested reviewers: None specified 🚥 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 |
|
/pj-rehearse |
|
@joepvd: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel. |
|
@redhat-chai-bot: 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. |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: redhat-chai-bot, travier 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 |
openshift-ci
Bot
added
the
approved
3 participants
The openshift/os PR #1945 refactors the extensions build to use
dnf downloadinstead ofrpm-ostree compose. This introduces a newEXTENSIONS_YUM_REPO_NAMESbuild arg inextensions/Containerfilethat must be supplied at build time.Without this change, the CI extensions image builds (
rhel-coreos-extensionsandrhel-coreos-10-extensions) fail with:because the placeholder default value
overriddenis used literally as a dnf repo name.This PR adds the
EXTENSIONS_YUM_REPO_NAMESbuild arg to both extensions image definitions in the ci-operator configs foropenshift/os(masterandrelease-5.0branches), mirroring the existingYUM_REPO_NAMESpattern used by the node image builds.Changes
openshift-os-master.yaml: AddedEXTENSIONS_YUM_REPO_NAMESforrhel-coreos-extensions(RHEL 9) andrhel-coreos-10-extensions(RHEL 10)openshift-os-release-5.0.yaml: Same additionsJira: ART-19543
Summary by CodeRabbit
This PR updates OpenShift OS CI build configs so the extensions image builds for both
masterandrelease-5.0pass the newEXTENSIONS_YUM_REPO_NAMESbuild argument. In practice, this fixes the affectedopenshift/osCI jobs for the RHEL 9 and RHEL 10 extensions images by ensuringdnf downloaduses the correct YUM repositories, including the high-availability repos that were missing before.