Skip to content

Update 69-vdev.rules.in to add support to alias md-type devices #17345

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: master
Choose a base branch
from

Conversation

a-d-j-i
Copy link

@a-d-j-i a-d-j-i commented May 17, 2025

Motivation and Context

If you run zfs over an md-type device and want to add aliases

Description

Add support to alias md-type devices in udev rules.

How Has This Been Tested?

Just tested it locally by re-triggering udev in my setup.

Types of changes

  • [ x ] Bug fix (non-breaking change which fixes an issue)

Checklist:

@a-d-j-i a-d-j-i changed the title Update 69-vdev.rules.in Update 69-vdev.rules.in to add support to alias md-type devices May 17, 2025
@amotin amotin added the Status: Code Review Needed Ready for review and testing label May 19, 2025
Copy link
Contributor

@behlendorf behlendorf left a comment

Choose a reason for hiding this comment

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

While probably not a common set up, I agree we should be able to use the vdev rules to create aliases for md devices. Looks good.

@behlendorf behlendorf added Status: Accepted Ready to integrate (reviewed, tested) and removed Status: Code Review Needed Ready for review and testing labels May 19, 2025
@github-actions github-actions bot removed the Status: Accepted Ready to integrate (reviewed, tested) label May 19, 2025
@tonyhutter
Copy link
Contributor

Please remove the merge commit.

Add support to alias md-type devices in udev rules.

Signed-off-by: Andres <[email protected]>
@behlendorf behlendorf added the Status: Accepted Ready to integrate (reviewed, tested) label May 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status: Accepted Ready to integrate (reviewed, tested)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants