From 42b204bf8a55b223047847f099292d1cfbfcda6d Mon Sep 17 00:00:00 2001
From: Hermes Agent service account
 <hermes@carousel-of-progress.local.mk-labs.cloud>
Date: Thu, 28 May 2026 14:06:24 -0500
Subject: [PATCH] fix(semaphore): make quadlet deployment self-contained with
 immediate daemon-reload + service start

---
 ansible/roles/semaphore/handlers/main.yml | 17 ++-------------
 ansible/roles/semaphore/tasks/main.yml    | 25 ++++++++++-------------
 2 files changed, 13 insertions(+), 29 deletions(-)

diff --git a/ansible/roles/semaphore/handlers/main.yml b/ansible/roles/semaphore/handlers/main.yml
index ddcec0a..383319c 100644
--- a/ansible/roles/semaphore/handlers/main.yml
+++ b/ansible/roles/semaphore/handlers/main.yml
@@ -1,16 +1,3 @@
 ---
-- name: reload systemd and start postgres
-  ansible.builtin.systemd:
-    daemon_reload: true
-    name: "{{ semaphore_postgres_container_name }}.service"
-    state: started
-    enabled: true
-  become: true
-
-- name: reload systemd and start semaphore
-  ansible.builtin.systemd:
-    daemon_reload: true
-    name: "{{ semaphore_container_name }}.service"
-    state: started
-    enabled: true
-  become: true
+# Handlers kept for potential future use or legacy paths.
+# Quadlet deployment now handles reload + start directly in the task block.
diff --git a/ansible/roles/semaphore/tasks/main.yml b/ansible/roles/semaphore/tasks/main.yml
index c848e34..d9d2f64 100644
--- a/ansible/roles/semaphore/tasks/main.yml
+++ b/ansible/roles/semaphore/tasks/main.yml
@@ -44,7 +44,6 @@
         group: root
         mode: "0644"
       become: true
-      notify: reload systemd and start postgres
 
     - name: Deploy Semaphore quadlet file
       ansible.builtin.template:
@@ -54,7 +53,17 @@
         group: root
         mode: "0644"
       become: true
-      notify: reload systemd and start semaphore
+
+    - name: Reload systemd and start quadlet services
+      ansible.builtin.systemd:
+        daemon_reload: true
+        name: "{{ item }}"
+        state: started
+        enabled: true
+      become: true
+      loop:
+        - "{{ semaphore_postgres_container_name }}.service"
+        - "{{ semaphore_container_name }}.service"
 
 - name: Fallback - legacy podman_container deployment (BoltDB)
   when: not (semaphore_use_postgres | default(false))
@@ -111,15 +120,3 @@
         state: started
         enabled: true
   become: true
-
-- name: Ensure quadlet services are started (postgres path)
-  when: semaphore_use_postgres | default(false)
-  ansible.builtin.systemd:
-    daemon_reload: true
-    name: "{{ item }}"
-    state: started
-    enabled: true
-  become: true
-  loop:
-    - "{{ semaphore_postgres_container_name }}.service"
-    - "{{ semaphore_container_name }}.service"