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

storaged: btrfs: allow btrfs subvolumes to be deleted #19872

Merged
merged 1 commit into from
Jan 30, 2024

Conversation

jelly
Copy link
Member

@jelly jelly commented Jan 22, 2024


Btrfs subvolume deletion

Allow created subvolumes to be deleted

image

@jelly jelly added the no-test For doc/workflow changes, or experiments which don't need a full CI run, label Jan 22, 2024
@jelly jelly removed the no-test For doc/workflow changes, or experiments which don't need a full CI run, label Jan 22, 2024
@jelly jelly marked this pull request as ready for review January 22, 2024 16:09
@jelly jelly requested a review from mvollmer January 22, 2024 16:09
pkg/storaged/btrfs/subvolume.jsx Outdated Show resolved Hide resolved
pkg/storaged/btrfs/subvolume.jsx Outdated Show resolved Hide resolved
pkg/storaged/btrfs/subvolume.jsx Show resolved Hide resolved
pkg/storaged/btrfs/subvolume.jsx Outdated Show resolved Hide resolved
pkg/storaged/btrfs/subvolume.jsx Show resolved Hide resolved
pkg/storaged/btrfs/subvolume.jsx Outdated Show resolved Hide resolved
pkg/storaged/btrfs/subvolume.jsx Outdated Show resolved Hide resolved
pkg/storaged/btrfs/subvolume.jsx Outdated Show resolved Hide resolved
pkg/storaged/btrfs/subvolume.jsx Outdated Show resolved Hide resolved
pkg/storaged/dialog.jsx Outdated Show resolved Hide resolved
pkg/storaged/btrfs/subvolume.jsx Outdated Show resolved Hide resolved
pkg/storaged/btrfs/subvolume.jsx Outdated Show resolved Hide resolved
pkg/storaged/btrfs/subvolume.jsx Outdated Show resolved Hide resolved
pkg/storaged/dialog.jsx Show resolved Hide resolved
pkg/storaged/btrfs/subvolume.jsx Outdated Show resolved Hide resolved
@jelly jelly force-pushed the btrfs-subvolume-deletion branch 4 times, most recently from f01d3c2 to 40c0c90 Compare January 29, 2024 16:56
Deletion is mostly similar to creation, except we allow a user to
delete a parent and it's children from the UI unlike creation where we
only allow a child to be created.
let name;
if (block_stratis) {
name = block_stratis.Devnode;
} else if (block_btrfs && use.name) {
Copy link
Contributor

Choose a reason for hiding this comment

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

This added line is not executed by any test. Details

@mvollmer mvollmer self-assigned this Jan 30, 2024
Copy link
Member

@mvollmer mvollmer left a comment

Choose a reason for hiding this comment

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

Thanks!

@mvollmer mvollmer merged commit ac54438 into cockpit-project:main Jan 30, 2024
92 of 93 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release-blocker Targetted for next release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants