From a6cf555b00434bc60872a8897d2ece4272c69704 Mon Sep 17 00:00:00 2001
From: ItsDrike <itsdrike@protonmail.com>
Date: Mon, 8 Apr 2024 01:53:52 +0200
Subject: [PATCH] Add a todo for semi-persistent/expiring roots

---
 system/impermanence/autowipe.nix | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/system/impermanence/autowipe.nix b/system/impermanence/autowipe.nix
index f0ba6a5..71618d5 100644
--- a/system/impermanence/autowipe.nix
+++ b/system/impermanence/autowipe.nix
@@ -18,6 +18,12 @@ in
         unitConfig.DefaultDependencies = "no";
         serviceConfig.Type = "oneshot";
         script = let
+          # TODO: Consider adding support for moving the subvolume to something like old-roots
+          #      instead of deleting it. This would allow for easier recovery in case of a mistake.
+          # The subvolume can live here for a certain amount of time, before it's deleted, or there
+          # can be a certain amount of total roots kept. See the example configuration here:
+          # <https://github.com/nix-community/impermanence> which demonstrates keepping 30 days
+          # worth of old roots.
           wipeScript = devicePath: subvolumes: ''
             # Mount the BTRFS device root to a temporary mount point
             echo "Mounting BTRFS root from ${devicePath} to /mnt"