repos: Clarify usage and which RHEL versions are available in repos#28521
Merged
openshift-merge-robot merged 5 commits intoopenshift:masterfrom Jun 3, 2022
Merged
repos: Clarify usage and which RHEL versions are available in repos#28521openshift-merge-robot merged 5 commits intoopenshift:masterfrom
openshift-merge-robot merged 5 commits intoopenshift:masterfrom
Conversation
Member
Author
Member
|
Looks great! Thanks for doing this. /approve |
Member
|
/cc @jupierce /approve |
openshift-ci
Bot
added
the
approved
travier
commented
May 13, 2022
Member
Author
|
/hold |
openshift-ci
Bot
added
do-not-merge/hold
Member
|
/uncc |
openshift-ci
Bot
removed
the
needs-rebase
Member
Author
|
/unhold |
openshift-ci
Bot
removed
the
do-not-merge/hold
Member
Author
|
Updated to use RHEL 8.6 EUS everywhere and include #26811. |
Member
|
Since we are doing clean up in this PR, couple of questions:
|
Contributor
|
Yes we still need aarch64 repo files because the "main" ones hard-code |
Member
Author
I think we still want that repo to be on the "correct" RHEL version for a given OCP release. I kept it for consumers that want the latest valid RHEL for a given OCP release without caring about the version itself. |
Member
Author
|
We might however want to remove some RHCOS specific repos from those files. I'll try that. |
From RHEL 8.6 onwards there is no advanced virtualization repository anymore. Signed-off-by: Eduardo Lima (Etrunko) <etrunko@redhat.com>
- Use RHEL 8.6 EUS repos - Use RHEL 8.6 TUS for NFV repo - Remove RHEL 8.5 repos
travier
force-pushed
the
repos
branch
2 times, most recently
from
88abf91 to
8e38d64
Compare
Member
Author
|
Updated this PR to focus on cleanups and the switch to RHEL 8.6 in 4.11 and later |
Member
|
/lgtm |
Contributor
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: hongkailiu, miabbott, 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 |
Contributor
|
@travier: 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/test-infra repository. I understand the commands that are listed here. |
Contributor
|
@travier: Updated the following 7 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/test-infra repository. |
jupierce
added a commit
to jupierce/release
that referenced
this pull request
Jun 6, 2022
These were removed in openshift#28521 but it prevents CI workloads from accessing packages built by ART and tagged into ART's candidate tag. Restoring until RHCOS can discuss the reasoning with us.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
6 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
For RHCOS, we want to be able to have a stable name for each RHEL versions available from a repo to help us during migrations from one RHEL minor to the next. Thus we will no longer used the un-versioned
rhel8repos and use minor versioned ones:rhel86,rhel90.Whether or not the URLs used for the repos are minor versioned is a different issue that we are waiting for confirmation. See openshift-eng/ocp-build-data#1576 (comment) & #28503 (comment)
Keep the un-versioned one
rhel8for compatibility for other potential users and make it use the latest released RHEL.