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

support identifying existing devices by UUID #44

Open
dwlehman opened this issue Oct 11, 2019 · 2 comments
Open

support identifying existing devices by UUID #44

dwlehman opened this issue Oct 11, 2019 · 2 comments
Assignees

Comments

@dwlehman
Copy link
Collaborator

The current codebase does not make any attempt to identify devices by descriptors other than name. While we cannot specify UUID for new devices, we can support lookup/id of existing devices by a uuid field.

@dwlehman dwlehman self-assigned this Oct 11, 2019
@pcahyna
Copy link
Member

pcahyna commented Oct 11, 2019

If you mean the UUID field in the blkid output, I would say this is a bad idea. This data is derived from the content of the disk, not from the disk itself. And the content of the disk is going to change when the role is applied, making the playbook non-idempotent.
It can be a good idea only if you are sure you want to change the attributes of an existing filesystem without reformatting it.

@pcahyna
Copy link
Member

pcahyna commented Oct 14, 2019

Also, why can't we specify it for new devices? mkfs.ext4, mkfs.xfs and mkswap all support it.

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

No branches or pull requests

2 participants