Skip to content

Test VDO support on RHEL 10 #22008

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

jelly
Copy link
Member

@jelly jelly commented May 15, 2025

@jelly jelly added the no-test For doc/workflow changes, or experiments which don't need a full CI run, label May 15, 2025
@jelly jelly requested a review from mvollmer May 15, 2025 10:38
@@ -1354,6 +1354,7 @@ function legacy_vdo_overlay() {
function start() {
let buf = "";

// legacy vdo monitoring (RHEL-8)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The whole function is called legacy_vdo_overlay() with a comment re RHEL 8 at the top, I don't think we need this here in addition... or?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Okido re-pushed

@jelly jelly removed the no-test For doc/workflow changes, or experiments which don't need a full CI run, label May 16, 2025
Copy link
Member

@mvollmer mvollmer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Upps, the test changes have disappeared...

kmod-kvdo has been replaced in RHEL 10 with the in kernel dm-vdo
module. For support it still needs the `vdo` package for `vdoformat`
which lvm2 utilities call.

https://issues.redhat.com/browse/RHELDOCS-19842
@mvollmer mvollmer self-requested a review May 16, 2025 07:11
Copy link
Member

@mvollmer mvollmer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants