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

Ceph RBD: Regenerate the FS UUID for filesystem volumes only #12805

Merged
merged 5 commits into from
Feb 1, 2024

Conversation

roosterfish
Copy link
Contributor

See #12743 (comment)

Due to #12745 that issue didn't surface.

Actual type block volumes may contain the block.filesystem config key
which will trigger the UUID regneration.

Signed-off-by: Julian Pelizäus <julian.pelizaeus@canonical.com>
tomponline
tomponline previously approved these changes Feb 1, 2024
Copy link
Member

@tomponline tomponline left a comment

Choose a reason for hiding this comment

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

Good spot! thanks!

@tomponline
Copy link
Member

Looks like the GH runners dont like the xfs filesystem option ...maybe we need to install the xfs tooling?

@tomponline
Copy link
Member

xfsprogs

@roosterfish
Copy link
Contributor Author

Looks like the GH runners dont like the xfs filesystem option ...maybe we need to install the xfs tooling?

There was already a check for all the drivers except LVM and Ceph that didn't allow setting block.* config keys on --type block volumes. I have now copied this check from the ZFS driver to both LVM and Ceph and modified the test suite accordingly.

…umes

Signed-off-by: Julian Pelizäus <julian.pelizaeus@canonical.com>
Signed-off-by: Julian Pelizäus <julian.pelizaeus@canonical.com>
Signed-off-by: Julian Pelizäus <julian.pelizaeus@canonical.com>
Signed-off-by: Julian Pelizäus <julian.pelizaeus@canonical.com>
@tomponline tomponline merged commit a6bdb16 into canonical:main Feb 1, 2024
26 checks passed
@roosterfish roosterfish deleted the fix_ceph_fs_uuid_restore branch February 1, 2024 13:25
tomponline added a commit to tomponline/lxd-pkg-snap that referenced this pull request Feb 1, 2024
Lands fixes from:

 canonical/lxd#12745
 canonical/lxd#12777
 canonical/lxd#12805 (partial)

Signed-off-by: Thomas Parrott <thomas.parrott@canonical.com>
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