Skip to content

Remove btrfs default structure prompt on default disk layout#3615

Closed
svartkanin wants to merge 1 commit intoarchlinux:masterfrom
svartkanin:remove-btrfs-question
Closed

Remove btrfs default structure prompt on default disk layout#3615
svartkanin wants to merge 1 commit intoarchlinux:masterfrom
svartkanin:remove-btrfs-question

Conversation

@svartkanin
Copy link
Copy Markdown
Collaborator

There was a previous issue related to an installation with BTRFS but no subvolumes #3477 (comment). In the default disk setup mode that is possible as there is a question to add the subvolumes or not which doesn't make much sense as choosing "no" will not actually work.

This removes the question if subolumes should be added in the default disk setup but still keeps the prompt in the manual partitioning menu as users are expected to be more advanced in there

@svartkanin svartkanin requested a review from Torxed as a code owner June 20, 2025 08:42
Copy link
Copy Markdown
Member

@Torxed Torxed left a comment

Choose a reason for hiding this comment

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

I still think they should have the option to choose, but if it's broken then we should merge this and circle back later.

@svartkanin
Copy link
Copy Markdown
Collaborator Author

What would be the actual flow though if someone chooses not to use the default structure? There's no other option at that point to add subvolumes

@Torxed
Copy link
Copy Markdown
Member

Torxed commented Jun 22, 2025

My understanding is that people use btrfs as a normal filesystem (like ext4) if no subvolumes were selected, and they can add some manually later. And if they select subvolumes they'll get some sensible defaults :)

@svartkanin
Copy link
Copy Markdown
Collaborator Author

Okay I tried to reproduce the problem mentioned without specifying subvolumes but everything seems to work fine. I'll decline this for now as it seems to be a working use case

@svartkanin svartkanin closed this Jun 27, 2025
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