| 9 Jul 2025 |
Zhaofeng Li | but it's kind of rude to break stable like that | 00:42:46 |
hexa | yeah, true | 00:43:05 |
hexa | so we hack around with chmod in prestart? | 00:43:21 |
hexa | * so we hack around with chmod in prestart for now | 00:44:09 |
Zhaofeng Li | yeah, looks like the least dramatic option, though not ideal | 00:45:06 |
Zhaofeng Li | maybe we can migrate to a static user, but force everyone with stateVersion <= 25.11 to explicitly set stateDir to acknowledge the migration | 00:46:41 |
Zhaofeng Li | * maybe we can migrate to a static user in unstable, but force everyone with stateVersion <= 25.11 to explicitly set stateDir to acknowledge the migration | 00:47:13 |
hexa | diff --git a/nixos/modules/services/networking/kea.nix b/nixos/modules/services/networking/kea.nix
index 6e0af62425a4..6f815659f567 100644
--- a/nixos/modules/services/networking/kea.nix
+++ b/nixos/modules/services/networking/kea.nix
@@ -272,12 +272,13 @@ in
};
commonServiceConfig = {
- ExecReload = "${pkgs.coreutils}/bin/kill -HUP $MAINPID";
+ ExecStartPre = "${lib.getExe' pkgs.coreutils "chmod"} 0750 /run/kea";
+ ExecReload = "${lib.getExe' pkgs.coreutils "kill"} -HUP $MAINPID";
DynamicUser = true;
User = "kea";
ConfigurationDirectory = "kea";
RuntimeDirectory = "kea";
- RuntimeDirectoryMode = "750";
+ RuntimeDirectoryMode = "0750";
RuntimeDirectoryPreserve = true;
StateDirectory = "kea";
UMask = "0077";
| 00:47:46 |
hexa | this change nothing | 00:47:56 |
hexa | * this changes nothing | 00:48:38 |
Zhaofeng Li | it seems racy... I added the chmod to script (not ExecStartPre) and it failed once but worked the next two tries? | 00:52:03 |
Zhaofeng Li | what even is going on | 00:52:11 |
hexa | can only be systemd doing this | 00:52:45 |
Zhaofeng Li | this is beyond cursed | 00:54:40 |