-
Notifications
You must be signed in to change notification settings - Fork 33
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
images: Add rhel-10 #6116
base: main
Are you sure you want to change the base?
images: Add rhel-10 #6116
Conversation
mvollmer
commented
Mar 20, 2024
•
edited by cockpituous
edited by cockpituous
- FAIL: image-refresh rhel-10
There are no cloud images yet. Also, the repo URL needs to include "-Public-Beta", if this is indeed the rhel-10 that we are looking for. |
Too early indeed, there's not even c10s images yet either. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Images exist now, so we can resume this.
So the repo urls are a bit funky now: /rhel-10/nightly/RHEL-10-Public-Beta/latest-RHEL-10/ instead of rhel-10/nightly/RHEL-10/latest-RHEL-10.0/compose/BaseOS/x86_64/os/repodata/repomd.xml Do I need to complain somewhere? :) |
664d9a9
to
bc00378
Compare
We got further but need some investigation.
|
@jelly That's known -- we already have a centos-10 special case here: https://github.com/cockpit-project/bots/blob/main/images/scripts/rhel.setup#L234 It needs to be extended to also cover rhel-10. |
2a938bf
to
4ace697
Compare
images/scripts/rhel.setup
Outdated
@@ -54,17 +54,23 @@ EOF | |||
VERSION=${IMAGE#rhel-} | |||
MAJOR=${VERSION%%-*} | |||
VERSION=${VERSION/-/.} | |||
echo "VERSION=$VERSION" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Already dropped locally.
This errors, and stops the script. |
4ace697
to
365679c
Compare