From b7f376b6b319c911f4d5456608a49283c9db3e4c Mon Sep 17 00:00:00 2001 From: reachableceo Date: Thu, 3 Sep 2026 06:33:00 -0500 Subject: [PATCH] fix(deploy): sha-watch deploys must restart core - reload kills webhook triggers [#345] Found live 2026-09-03: after reload-only deploys, webhook-triggered automations (doorman ingest, deploy fast path) silently stop firing while the endpoint still returns HTTP 200. Proven: core restart restores them. Sha-watch now restarts core on every new master HEAD (same semantics as the webhook fast path for packages/** changes). https://projects.knownelement.com/issues/345 --- automations.yaml | 19 +++++++++++++------ 1 file changed, 13 insertions(+), 6 deletions(-) diff --git a/automations.yaml b/automations.yaml index 5d61ed9..a7dd578 100644 --- a/automations.yaml +++ b/automations.yaml @@ -330,9 +330,15 @@ - id: pfv_deploy_sha_watch alias: PFV deploy - gitea sha watch safety net description: >- - Master HEAD sha changed -> pull + reload_all, no core restart. - Interim deploy trigger while gitea webhook delivery is blocked by - its SSRF host allowlist [#734]. + Master HEAD sha changed -> pull + CORE RESTART. Interim deploy + trigger while gitea webhook delivery is blocked by its SSRF host + allowlist [#734]. Restart (not reload) is REQUIRED: automation + reload loses webhook-trigger registration - found live 2026-09-03 + (#345): after any reload-only deploy, webhook automations (doorman + scan ingest, this pipeline's own fast path) silently stop firing + while the endpoint still returns HTTP 200. Core restart + re-registers them. Accepted trade-off: ~1 min HA outage per + master deploy. mode: single max_exceeded: silent triggers: @@ -353,7 +359,6 @@ hours: 0 minutes: 0 seconds: 20 - - action: homeassistant.reload_all - action: notify.send_message target: entity_id: @@ -362,5 +367,7 @@ data: title: PFV deploy (sha watch) message: >- - New master HEAD pulled + reloaded at {{ now().strftime('%H:%M:%S') }} - - no core restart. HEAD {{ states('sensor.pfv_deploy_gitea_master_head')[:7] }} + New master HEAD {{ states('sensor.pfv_deploy_gitea_master_head')[:7] }} + pulled. Restarting HA core (webhook re-registration) - back in + ~1 min. + - action: homeassistant.restart