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

predictably arbitrary disk selection #2050

Merged
merged 2 commits into from
Aug 9, 2024

Conversation

dbungert
Copy link
Collaborator

@dbungert dbungert commented Aug 8, 2024

filesystem: arbitrary disk select more consistent

If two disks of the same size went to the disk matchers with the default
"largest" match, the one we would choose would be based on whatever
order in python they ended up, and across installs this means that we
could select arbitrarily among disks of a matching size.

Add sort criteria based on various unique identifiers. Make it work for
other disk-like devices, though the main target is disks and raids
because that's what can go to the matcher system.

If two disks of the same size went to the disk matchers with the default
"largest" match, the one we would choose would be based on whatever
order in python they ended up, and across installs this means that we
could select arbitrarily among disks of a matching size.

Add sort criteria based on various unique identifiers.  Make it work for
other disk-like devices, though the main target is disks and raids
because that's what can go to the matcher system.
@dbungert dbungert requested a review from mwhudson August 9, 2024 02:16
@dbungert dbungert merged commit 59ac2a8 into canonical:main Aug 9, 2024
12 checks passed
@dbungert dbungert deleted the predictably-arbitrary branch August 9, 2024 14:34
@dbungert
Copy link
Collaborator Author

dbungert commented Aug 9, 2024

@medicalwei - FYI

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