Skip to content
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

enhance scan disks for md #1063 #1071

Merged
merged 3 commits into from
Dec 30, 2015

Conversation

phillxnet
Copy link
Member

In the case of md devices such as software raid ie mdadm and some hardware raid, the md devices and their respective partitions have repeat listings in lsblk, this accidentally triggers the duplicate device by serial number mechanism as the entries are identical; the consequence of this is a fail-safe that overrides the serial on the second occurrence (which has the same dev name) with a unique but fake serial. Avoid this udev assigned serial accidental override by only processing the first occurrence of each device by name in the lsblk output.

@schakrava I will have another go at testing this on a software raid system disk tomorrow (UK).

md* (multi device) block devices such as software raid
and some hardware raid devices are listed multiple times
in lsblk output, this causes them to be labeled as duplicate
devices due to shared serial number and any serial number
retrieved on the first occurrence is over-ridden on the second
occurrence by a fake to denote serial, and hence device,
duplication. This is incorrect in the case of these md devices.
@phillxnet
Copy link
Member Author

Haven't managed to install system disk on software raid as installer doesn't offer btrfs with raid. This would be a nice one to work out though. Can be done via installer if one selects eg XFS as root but then we arn't testing Rockstor as is. Also doesn't seem to present identically as original poster's system as their intel motherboard hardware md is presented as a partitioned md where as the installer goes with multiple md devices in software raid. So not equivalent. May revisit this attempt but taking too long. Would be nice to document a working solution when found though.

@schakrava
Copy link
Member

Thanks @phillxnet!

schakrava added a commit that referenced this pull request Dec 30, 2015
@schakrava schakrava merged commit 97ac482 into rockstor:master Dec 30, 2015
@phillxnet
Copy link
Member Author

Cheers @schakrava

@phillxnet phillxnet deleted the 1063_enhance_scan_disks_for_md branch December 30, 2015 23:00
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