Merge ~xnox/curtin:rocky into curtin:master

Proposed by Dimitri John Ledkov
Status: Merged
Approved by: Dan Bungert
Approved revision: 9ef03d7f83de26f4c130df4a7d741b27375e01fc
Merge reported by: Server Team CI bot
Merged at revision: not available
Proposed branch: ~xnox/curtin:rocky
Merge into: curtin:master
Diff against target: 36 lines (+4/-3)
2 files modified
curtin/distro.py (+3/-2)
pylintrc (+1/-1)
Reviewer Review Type Date Requested Status
Dan Bungert Approve
Server Team CI bot continuous-integration Approve
Review via email: mp+415604@code.staging.launchpad.net

Commit message

Add rocky linux as a RHEL-like variant

To post a comment you must log in.
Revision history for this message
Dimitri John Ledkov (xnox) wrote :

Entirely untested, and coded based on inspecting RPMs empirically.

Revision history for this message
Server Team CI bot (server-team-bot) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
Server Team CI bot (server-team-bot) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
Server Team CI bot (server-team-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
Francesco Santagata (phrancesco) wrote :

Hi,

I patched distro.py inside this one curtin package:

https://launchpad.net/~dbungert/+archive/ubuntu/curtin-lp-1955671

It's working.

I tested on two VMs (1 Bios + 1 UEFI).

With this "https://launchpad.net/~dbungert/+archive/ubuntu/curtin-lp-1955671"
BIOS Centos 8.4 OK, Rocky 8.4 OK
UEFI Centos 8.4 OK, Rocky 8.4 KO

With your latest patch of distro.py Rocky 8.4 on UEFI is working.

Minor Issue, at boot time it tests a lot of efi files before finding the right one...
The first time is really slow, then the "scan" goes much faster.

I have a screen capture of it.

BR
Francesco

PS
please let me know if you need a more clean test... got some problems with "git+ssh" a lost a little bit of focus.

Revision history for this message
Dan Bungert (dbungert) wrote :

Thanks Dimitri!
Tested by Francesco as mentioned on linked bug.

review: Approve

There was an error fetching revisions from git servers. Please try again in a few minutes. If the problem persists, contact Launchpad support.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches