fix(systemd): Skip enable and use start directly to avoid validation errors

Author: fourdollars

lib/concourse_web.py

@@ -309,29 +309,23 @@ def start_service(self):
                 )
                 os.chown(CONCOURSE_BIN, concourse_uid, concourse_gid)
 
-            is_enabled_check = subprocess.run(
-                ["systemctl", "is-enabled", "concourse-server.service"],
+            is_active_check = subprocess.run(
+                ["systemctl", "is-active", "concourse-server.service"],
                 capture_output=True,
                 text=True,
             )
-            service_not_enabled = is_enabled_check.returncode != 0
+            service_not_active = is_active_check.returncode != 0
 
-            if service_not_enabled:
+            if service_not_active:
                 subprocess.run(
-                    ["systemctl", "enable", "concourse-server.service"],
+                    ["systemctl", "start", "concourse-server.service"],
                     check=True,
                     capture_output=True,
                     text=True,
                 )
-                logger.info("Web server service enabled")
-
-            subprocess.run(
-                ["systemctl", "start", "concourse-server.service"],
-                check=True,
-                capture_output=True,
-                text=True,
-            )
-            logger.info("Web server service started")
+                logger.info("Web server service started")
+            else:
+                logger.info("Web server service already active")
         except subprocess.CalledProcessError as e:
             stderr = (
                 e.stderr if hasattr(e, "stderr") and e.stderr else "No stderr available"