spin-down-hdd

hosts/yodaHedgehog/configuration.nix

@@ -22,7 +22,7 @@
       ../../modules/btrbk
       ./btrbk-config.nix
       ../../modules/de-p1st-monitor.nix
-      #../../modules/spin-down.nix
+      #../../modules/spin-down-hdd.nix
 
       ../../modules/btrfs-scrub.nix
       ../../modules/btrfs-mount-options.nix

hosts/yodaNas/configuration.nix

@@ -26,7 +26,7 @@
       ./btrbk-config.nix
       ../../modules/netcup-dns.nix
       ../../modules/de-p1st-monitor.nix
-      ../../modules/spin-down.nix
+      ../../modules/spin-down-hdd.nix
 
       ../../modules/btrfs-scrub.nix
       ../../modules/btrfs-mount-options.nix
@@ -36,6 +36,14 @@
   boot.initrd.luks.devices."luks-3d974bd0-f373-469b-8e9c-2d5516e9f0f5".allowDiscards = true;
   yoda.btrfsFileSystems = ["/" "/mnt/data" "/mnt/backup"];
   #yoda.btrfsMounts = yoda.btrfsFileSystems;
+  yoda.spin-down-hdd = [
+    # 4tb1
+    "ata-WDC_WD40EFRX-68N32N0_WD-WCC7K0CPF0N1"
+    # 3tb1
+    "ata-WDC_WD30EFRX-68EUZN0_WD-WCC4N1173157"
+    # 3tb2
+    "ata-WDC_WD30EFRX-68EUZN0_WD-WMC4N0564095"
+  ];
 
   boot.kernelParams = [
     # Microarchitectural Data Sampling (MDS), see https://www.kernel.org/doc/html/latest/admin-guide/hw-vuln/mds.html#mitigation-control-on-the-kernel-command-line

hosts/yodaYoga/configuration.nix

@@ -10,22 +10,14 @@
       ../../modules/headless.nix
 
       ../../modules/ssh-fde-unlock.nix
-      #../../modules/git.nix
       ../../modules/zsh.nix
-      #../../modules/ssh-client.nix
       ../../modules/ssh-server.nix
-
-      #../../modules/podman.nix
       ../../modules/docker.nix
-      #../../modules/docker-pushrm.nix
 
       ../../modules/lid-switch-handling.nix
       ../../modules/sendmail-mta.nix
       ../../modules/journalwatch.nix
 
-      #../../modules/btrbk
-      #../../modules/spin-down.nix
-
       ../../modules/btrfs-scrub.nix
       ../../modules/btrfs-mount-options.nix
     ];

modules/btrfs-mount-options.nix

@@ -35,8 +35,8 @@ in
   config = mkIf (length cfg > 0) {
     # For each element (e.g. "/") in `cfg` create this config:
     #   fileSystems."/".options = [ "compress=zstd" "noatime" "commit=120" ];
-    fileSystems = builtins.listToAttrs (builtins.map (fs: {
+    fileSystems = builtins.listToAttrs (builtins.map (btrfs_mount_path: {
-      name = fs;
+      name = btrfs_mount_path;
       value = { options = [ "compress=zstd" "noatime" "commit=120" ]; };
     }) cfg);
   };

modules/spin-down-hdd.nix

@@ -1,5 +1,3 @@
-{ config, pkgs, ... }:
-
 # hdparm - get/set SATA/IDE device parameters
 #   -q  Handle the next option quietly, suppressing normal output (but not error messages).
 #   -I  Print device information.
@@ -61,32 +59,45 @@
 # - The standby timeout has been set before already, otherwise it would be spinning.
 # - Don't set the standby timeout again, as this will spin up the disk.
 
+{ lib, config, options, pkgs, modulesPath, ... }:
+with lib;
+let
+  cfg = config.yoda.spin-down-hdd;
+in
 {
+  options = {
+    yoda.spin-down-hdd = mkOption {
+      type = types.listOf types.str;
+      default = [ ];
+      example = ["ata-WDC_WD40EFRX-68N32N0_WD-WCC7K0CPF0N1"];
+      description = ''
+        List with IDs (/dev/disk/by-id/<ID>) of HDDs to spin down after 10 minutes.
+      '';
+    };
+  };
+
+  config = mkIf (length cfg > 0) {
+    # TODO: Is this required or is it enough to use `${pkgs.hdparm}` in the systemd service script?
     environment.systemPackages = with pkgs; [
       hdparm
     ];
-  systemd.services."hdparm-4tb1" = {
+
+    # For each element (e.g. "ata-WDC_WD40EFRX-68N32N0_WD-WCC7K0CPF0N1") in `cfg` create this config:
+    #   systemd.services."hdparm-ata-WDC_WD40EFRX-68N32N0_WD-WCC7K0CPF0N1" = {
+    #     description = ...;
+    #     wantedBy = ...;
+    #     ...
+    #   };
+    systemd.services = builtins.listToAttrs (builtins.map (id: {
+      name = "hdparm-${id}";
+      value = {
         description = "Spin down inactive HDD";
         wantedBy = [ "multi-user.target" ];
         serviceConfig.Type = "oneshot";
         script = ''
-      ${pkgs.hdparm}/bin/hdparm -C /dev/disk/by-id/ata-WDC_WD40EFRX-68N32N0_WD-WCC7K0CPF0N1 | ${pkgs.busybox}/bin/grep 'standby' || ${pkgs.hdparm}/bin/hdparm -S 24 /dev/disk/by-id/ata-WDC_WD40EFRX-68N32N0_WD-WCC7K0CPF0N1
+          ${pkgs.hdparm}/bin/hdparm -C /dev/disk/by-id/${id} | ${pkgs.busybox}/bin/grep 'standby' || ${pkgs.hdparm}/bin/hdparm -S 24 /dev/disk/by-id/${id}
         '';
       };
-  systemd.services."hdparm-3tb1" = {
+    }) cfg);
-    description = "Spin down inactive HDD";
-    wantedBy = [ "multi-user.target" ];
-    serviceConfig.Type = "oneshot";
-    script = ''
-      ${pkgs.hdparm}/bin/hdparm -C /dev/disk/by-id/ata-WDC_WD30EFRX-68EUZN0_WD-WCC4N1173157 | ${pkgs.busybox}/bin/grep 'standby' || ${pkgs.hdparm}/bin/hdparm -S 24 /dev/disk/by-id/ata-WDC_WD30EFRX-68EUZN0_WD-WCC4N1173157
-    '';
-  };
-  systemd.services."hdparm-3tb2" = {
-    description = "Spin down inactive HDD";
-    wantedBy = [ "multi-user.target" ];
-    serviceConfig.Type = "oneshot";
-    script = ''
-      ${pkgs.hdparm}/bin/hdparm -C /dev/disk/by-id/ata-WDC_WD30EFRX-68EUZN0_WD-WMC4N0564095 | ${pkgs.busybox}/bin/grep 'standby' || ${pkgs.hdparm}/bin/hdparm -S 24 /dev/disk/by-id/ata-WDC_WD30EFRX-68EUZN0_WD-WMC4N0564095
-    '';
   };
 }