Skip to content

Add kernel command line for snapshots as config parameter#245

Merged
Schievel1 merged 4 commits into
masterfrom
improve-config
Jan 22, 2023
Merged

Add kernel command line for snapshots as config parameter#245
Schievel1 merged 4 commits into
masterfrom
improve-config

Conversation

@Schievel1

@Schievel1 Schievel1 commented Dec 5, 2022

Copy link
Copy Markdown
Collaborator

See #214 and #160.

Also improved default for GRUB_BTRFS_GBTRFS_DIRNAME to be GRUB_BTRFS_GRUB_DIRNAME (see #244)

This is completely untested, just an idea I had when reading issues.

@Schievel1 Schievel1 force-pushed the improve-config branch 4 times, most recently from 684521a to 2a9f34e Compare December 5, 2022 20:21
@Schievel1 Schievel1 marked this pull request as ready for review December 6, 2022 21:50
Fix #244

Signed-off-by: Pascal Jäger <pascal.jaeger@leimstift.de>
Fix #214, #160

Signed-off-by: Pascal Jäger <pascal.jaeger@leimstift.de>
Signed-off-by: Pascal Jäger <pascal.jaeger@leimstift.de>
@Schievel1 Schievel1 merged commit 87f030f into master Jan 22, 2023
Comment thread initramfs/readme.md
`mkinitcpio -P` (option -P means, all preset present in `/etc/mkinitcpio.d`)

#### Dracut based distros
Distributions that use Dracut to make their initramfs (many of the Fedora based Distros) simply have to pass either `rd.live.overlay.readonly=1` (to boot into the snapshot read only) or `rd.live.overlay.overlayfs=1` (to act like a livedisk, that is files can be changed but changes will be lost on the next boot) to their kernel command line in grub.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

This part does not seem to be true, Ubuntu is currently switching to Dracut and this solution does not work there... When switching back to initramfs, #250 or #359 work without any problems (you should merge at least one of them, they are basically the same, #250 is more user-friendly).

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