permown: simplify and fixed issues

configs/pepe/syncthing.nix

@@ -81,16 +81,14 @@
     };
   };
 
-  # todo not working properly
+  system.permown."/media/syncthing" = {
-  #systemd.services."permown._media_syncthing" = {
+    owner = "syncthing";
-  #  bindsTo = [ "media.mount" ];
+    group = "syncthing";
-  #  after = [ "media.mount" ];
+  };
-  #};
+  systemd.services."permown._media_syncthing" = {
-  #system.permown."/media/syncthing" = {
+    bindsTo = [ "media.mount" ];
-  #  owner = "syncthing";
+    after = [ "media.mount" ];
-  #  group = "syncthing";
+  };
-  #  umask = "0003";
-  #};
   systemd.services."syncthing" = {
     bindsTo = [ "media.mount" ];
     after = [ "media.mount" ];

configs/porani/syncthing.nix

@@ -42,14 +42,13 @@
     };
   };
 
-  systemd.services."permown._media" = {
-    bindsTo = [ "media.mount" ];
-    after = [ "media.mount" ];
-  };
   system.permown."/media" = {
     owner = "syncthing";
     group = "syncthing";
-    umask = "0007";
+  };
+  systemd.services."permown._media" = {
+    bindsTo = [ "media.mount" ];
+    after = [ "media.mount" ];
   };
   systemd.services."syncthing" = {
     bindsTo = [ "media.mount" ];

configs/sterni/syncthing.nix

@@ -65,4 +65,9 @@
       };
     };
   };
+
+  system.permown."/home/palo/music-library" = {
+    owner = "palo";
+    group = "users";
+  };
 }

configs/workhorse/borg.nix

@@ -5,7 +5,6 @@
       quota = "100G";
       allowSubRepos = true;
       authorizedKeys = [
-        # todo rename
         (lib.fileContents <common_secrets/backup/ssh_rsa.pub>)
         (lib.fileContents <assets/ssh/card_rsa.pub>)
       ];

configs/workhorse/syncthing.nix

@@ -100,17 +100,14 @@
     };
   };
 
-  # todo now working properly
+  system.permown."/media/syncthing" = {
-  # -------------------------
+    owner = "syncthing";
-  #systemd.services."permown._media_syncthing" = {
+    group = "syncthing";
-  #  bindsTo = [ "media.mount" ];
+  };
-  #  after = [ "media.mount" ];
+  systemd.services."permown._media_syncthing" = {
-  #};
+    bindsTo = [ "media.mount" ];
-  #system.permown."/media/syncthing" = {
+    after = [ "media.mount" ];
-  #  owner = "syncthing";
+  };
-  #  group = "syncthing";
-  #  umask = "0002";
-  #};
   systemd.services."syncthing" = {
     bindsTo = [ "media.mount" ];
     after = [ "media.mount" ];

modules/system/permown.nix

@@ -1,6 +1,11 @@
 { config, pkgs, lib, ... }:
+
 with lib;
-let cfg = config.system.permown;
+
+let
+  cfg = config.system.permown;
+  nameGenerator = path: "permown.${replaceStrings [ "/" ] [ "_" ] path}";
+
 in {
 
   options.system.permown = mkOption {
@@ -10,11 +15,11 @@ in {
         options = {
           directory-mode = mkOption {
             default = "=rwx";
-            type = types.str; # TODO
+            type = types.str;
           };
           file-mode = mkOption {
             default = "=rw";
-            type = types.str; # TODO
+            type = types.str;
           };
           group = mkOption {
             apply = x: if x == null then "" else x;
@@ -30,69 +35,58 @@ in {
             default = "0027";
             type = types.str;
           };
+          timer = mkOption {
+            default = "hourly";
+            type = types.str;
+            description =
+              "OnCalendar string on how frequent should this command run";
+          };
         };
       }));
   };
 
   config = let plans = lib.attrValues cfg;
+
   in mkIf (plans != [ ]) {
 
     system.activationScripts.permown = let
-      mkdir = plan: # sh
+      mkdir = { path, ... }: ''
-        ''
+        ${pkgs.coreutils}/bin/mkdir -p ${path}
-          ${pkgs.coreutils}/bin/mkdir -p ${plan.path}
+      '';
-        '';
     in concatMapStrings mkdir plans;
 
-    # genAttrs' = names: f: listToAttrs (map f names);
+    systemd.services = listToAttrs (flip map plans
+      ({ path, directory-mode, file-mode, owner, group, umask, ... }: {
+        name = nameGenerator path;
+        value = {
+          environment = {
+            DIR_MODE = directory-mode;
+            FILE_MODE = file-mode;
+            OWNER_GROUP = "${owner}:${group}";
+            ROOT_PATH = path;
+          };
+          path = [ pkgs.coreutils pkgs.findutils pkgs.inotifyTools ];
+          serviceConfig = {
+            ExecStart = pkgs.writers.writeDash "permown" ''
+              set -efu
+              find "$ROOT_PATH" -exec chown -h "$OWNER_GROUP" {} +
+              find "$ROOT_PATH" -type d -exec chmod "$DIR_MODE" {} +
+              find "$ROOT_PATH" -type f -exec chmod "$FILE_MODE" {} +
+            '';
+            PrivateTmp = true;
+            Restart = "always";
+            RestartSec = 10;
+            UMask = umask;
+          };
+          wantedBy = [ "multi-user.target" ];
+        };
+      }));
 
-    systemd.services = listToAttrs (flip map plans (plan: {
+    systemd.timers = listToAttrs (flip map plans ({ path, timer, ... }: {
-      name = "permown.${replaceStrings [ "/" ] [ "_" ] plan.path}";
+      name = nameGenerator path;
       value = {
-        environment = {
-          DIR_MODE = plan.directory-mode;
-          FILE_MODE = plan.file-mode;
-          OWNER_GROUP = "${plan.owner}:${plan.group}";
-          ROOT_PATH = plan.path;
-        };
-        path = [ pkgs.coreutils pkgs.findutils pkgs.inotifyTools ];
-        serviceConfig = {
-          ExecStart = pkgs.writers.writeDash "permown" ''
-            set -efu
-
-            find "$ROOT_PATH" -exec chown -h "$OWNER_GROUP" {} +
-            find "$ROOT_PATH" -type d -exec chmod "$DIR_MODE" {} +
-            find "$ROOT_PATH" -type f -exec chmod "$FILE_MODE" {} +
-
-            paths=/tmp/paths
-            rm -f "$paths"
-            mkfifo "$paths"
-
-            inotifywait -mrq -e CREATE --format %w%f "$ROOT_PATH" > "$paths" &
-            inotifywaitpid=$!
-
-            trap cleanup EXIT
-            cleanup() {
-              kill "$inotifywaitpid"
-            }
-
-            while read -r path; do
-              if test -d "$path"; then
-                cleanup
-                exec "$0" "$@"
-              fi
-              chown -h "$OWNER_GROUP" "$path"
-              if test -f "$path"; then
-                chmod "$FILE_MODE" "$path"
-              fi
-            done < "$paths"
-          '';
-          PrivateTmp = true;
-          Restart = "always";
-          RestartSec = 10;
-          UMask = plan.umask;
-        };
         wantedBy = [ "multi-user.target" ];
+        timerConfig.OnCalendar = timer;
       };
     }));