supybot.service: tidy up

modules/misc/ids.nix

@@ -136,6 +136,7 @@ in
     scanner = 59;
     nginx = 60;
     systemd-journal = 62;
+    supybot = 63;
 
     # When adding a gid, make sure it doesn't match an existing uid.
 

modules/services/networking/supybot.nix

@@ -10,8 +10,6 @@ in
 
 {
 
-  ###### interface
-
   options = {
 
     services.supybot = {
@@ -22,44 +20,33 @@ in
       };
 
       stateDir = mkOption {
-        default = "/var/lib/supybot";
-        description = "
-
-        ";
+        # Setting this to /var/lib/supybot caused useradd to fail
+        default = "/home/supybot";
+        description = "The root directory, logs and plugins are stored here";
       };
 
       configFile = mkOption {
         type = types.path;
-        default = /dev/null;
         description = ''
-          Verbatim contents of the supybot config, this can be
-          generated by supybot-wizard
+          Path to a supybot config file. This can be generated by
+          running supybot-wizard.
+
+          Note: all paths should include the full path to the stateDir
+          directory (backup conf data logs logs/plugins plugins tmp web).
         '';
       };
 
-      user = mkOption {
-        default = "supybot";
-        description = "User account under which supybot runs.";
-      };
-
-      group = mkOption {
-        default = "supybot";
-        description = "Group account under which supybot runs.";
     };
 
   };
 
-  };
-
-
-  ###### implementation
 
   config = mkIf cfg.enable {
 
     environment.systemPackages = [ pkgs.pythonPackages.limnoria ];
 
-    users.extraUsers = singleton
-      { name = cfg.user;
+    users.extraUsers = singleton {
+      name = "supybot";
       uid = config.ids.uids.supybot;
       group = "supybot";
       description = "Supybot IRC bot user";
@@ -67,27 +54,29 @@ in
       createHome = true;
     };
 
-    users.extraGroups.supybot = {};
+    users.extraGroups.supybot = {
+      name = "supybot";
+      gid = config.ids.gids.supybot;
+    };
 
-    systemd.services.supybot =
-      { description = "Supybot IRC bot";
+    systemd.services.supybot = {
+      description = "Supybot, an IRC bot";
       after = [ "network.target" ];
       wantedBy = [ "multi-user.target" ];
       path = [ pkgs.pythonPackages.limnoria ];
       preStart = ''
-          mkdir -m 0755 -p ${cfg.stateDir}
-          chown ${cfg.user}:${cfg.group} ${cfg.stateDir}
         cd ${cfg.stateDir}
-          mkdir -p logs/plugins backup conf data plugins tmp
+        mkdir -p backup conf data plugins logs/plugins tmp web
         ln -sf ${cfg.configFile} supybot.cfg
+        # This needs to be created afresh every time
         rm -f supybot.cfg.bak
       '';
-        serviceConfig =
-          { ExecStart =
-              "${pkgs.pythonPackages.limnoria}/bin/supybot ${cfg.stateDir}/supybot.cfg";
+
+      serviceConfig = {
+        ExecStart = "${pkgs.pythonPackages.limnoria}/bin/supybot ${cfg.stateDir}/supybot.cfg";
         PIDFile = "/run/supybot.pid";
-            User = "${cfg.user}";
-            Group = "${cfg.group}";
+        User = "supybot";
+        Group = "supybot";
         UMask = "0007";
         Restart = "on-abort";
         StartLimitInterval = "5m";
@@ -96,5 +85,4 @@ in
     };
 
   };
-
 }