NO-ISSUE: fix mirror2 baseurls in ocp-5.0-rhel102.repo#79351
Conversation
The baseurls used incorrect path format (rhel-10.2-*-rpms) instead of the correct format (rhel-102-*) matching the 4.21/4.22 repo configs. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
openshift-ci-robot
added
the
jira/valid-reference
|
@ashwindasr: This pull request explicitly references no jira issue. 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 openshift-eng/jira-lifecycle-plugin repository. |
openshift-merge-bot
Bot
added
the
rehearsals-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 (1)
WalkthroughThis PR updates repository mirror URLs in the OCP 5.0 RHEL 10.2 configuration file. Six yum/dnf repository sections have their ChangesRepository Mirror Configuration
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Suggested labels
🚥 Pre-merge checks | ✅ 12✅ Passed checks (12 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@core-services/release-controller/_repos/ocp-5.0-rhel102.repo`:
- Line 3: The repo file change at baseurl =
https://mirror2.openshift.com/enterprise/reposync/5.0/rhel-102-baseos/ needs
validation with the repo-file validator; run /test-repo-files (or the CI job
that runs it) against
core-services/release-controller/_repos/ocp-5.0-rhel102.repo and the other
affected ocp-*.repo edits (lines referenced: 3, 14, 25, 36, 58, 70), capture and
attach the full validator output or CI job link to the PR, and if the validator
reports issues, correct the baseurl/CDN patterns accordingly and re-run until it
passes before merging.
- Line 58: The early-kernel repository's baseurl is incorrect: update the
baseurl value used in the early-kernel repo block (the line currently
referencing rhel-10-server-ose-rpms) to use the rhel-102-early-kernel path;
specifically replace the existing baseurl string with
https://mirror2.openshift.com/enterprise/reposync/5.0/rhel-102-early-kernel/ so
the early-kernel repo matches the 4.21/4.22 pattern.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Repository YAML (base), Central YAML (inherited)
Review profile: CHILL
Plan: Enterprise
Run ID: 51046597-791d-4109-ac78-f3c853fd61c3
📒 Files selected for processing (1)
core-services/release-controller/_repos/ocp-5.0-rhel102.repo
Matches the 4.21/4.22 pattern instead of incorrectly reusing the rhel-10-server-ose-rpms path. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
|
[REHEARSALNOTIFIER] Note: If this PR includes changes to step registry files ( Interacting with pj-rehearseComment: Once you are satisfied with the results of the rehearsals, comment: |
|
/assign @joepvd |
|
/lgtm |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: ashwindasr, dustymabe, jupierce 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
|
@ashwindasr: all tests passed! 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. |
|
@ashwindasr: Updated the following 15 configmaps:
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. |
5 participants
Summary
core-services/release-controller/_repos/ocp-5.0-rhel102.reporhel-10.2-*-rpmsinstead ofrhel-102-*(matching the existing 4.21/4.22 configs)Test plan
https://mirror2.openshift.com/enterprise/reposync/5.0/rhel-102-baseos/repodata/)🤖 Generated with Claude Code
Summary
This PR fixes incorrect mirror2 baseurl paths in the OpenShift 5.0 RHEL 10.2 repository configuration used by CI to fetch packages.
What changed
Edited core-services/release-controller/_repos/ocp-5.0-rhel102.repo to correct mirror2 baseurl paths:
The rhel-10.2-server-ose-5.0 section points to the enterprise server OSE path (rhel-10-server-ose-rpms) as expected. Other repo settings were unchanged.
Impact
CI jobs and release tooling that rely on these mirror2 URLs for OCP 5.0 / RHEL 10.2 will now resolve and access the correct package repositories. This aligns the 5.0 rhel102 repo layout with the working conventions used for OCP 4.21/4.22, preventing package resolution failures during builds and tests.