-
-
Notifications
You must be signed in to change notification settings - Fork 42
Adding ZFS Snapshot support #32
base: master
Are you sure you want to change the base?
Conversation
I think we might have to add a update-grub into it. |
Cool, thanks for testing that. I'll add in an update-grub and see how it works. |
Hmm... update-grub wasn't enough to fix it. I don't think this issue is directly related to the snapshot being taken. It seems something happens when switching to devel-repo with zfs on root? Will try to look further into this... Edit: |
It seems to be a bug that as soon as it takes another snapshot after switching to rolling that it is what breaks it. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If a feature like this is added then snapshots should be made using zsys
.
Ah, I was unaware of zsys. Let me see what I can do with it. |
Doesn't ZSys already take a snapshot every time a update is invoked? |
It does, but not until after we modify a bunch of config files in this case. I was thinking would be good to have a snapshot taken before we touch anything. |
Using zsys now. Still not sure if we should prompt the user for a zfs snapshot. But I think it's ready for review :) |
Very good point and manual snapshot is also needed when you consider the amount of desktop changes people make outside of updates. Thanks for working on this as it will save a lot of headaches , i find ZFS so confusing. :) |
A first pass at adding in ZFS snapshot support.
I'm not sure if this is the best way to detect if ZFS is installed on root. Happy to take feedback and make edits. Especially in regards to snapshot names, zfs on root detection, and output messaging. Also, should we prompt the user if a snapshot is desired or just automatically take the snapshot as it is currently coded? My thought is to just take the snapshot as there is no downside that I'm aware of.