Fix: Use atomic writes for Mongoose webhooks to prevent rule loading failures

Author: TontonSancho

pirogue-admin/debian/changelog

@@ -1,3 +1,9 @@
+pirogue-admin (2.0.14) bookworm; urgency=medium
+
+  * Fix: Use atomic writes for Mongoose webhooks to prevent rule loading failures
+
+ -- Christophe Andral <christophe@andral.fr>  Mon, 30 Mar 2026 18:53:44 +0200
+
 pirogue-admin (2.0.13) bookworm; urgency=medium
 
   * Device monitoring management

pirogue-admin/pirogue_admin/daemon/servicers_impl.py

@@ -719,16 +719,20 @@ def AddDeviceMonitoring(self, request, context):
         payload = MessageToDict(request, preserving_proto_field_name=True)
         payload.pop('name', None)
 
-        yaml_file = Path(self._base_configuration_context.pirogue_working_root_dir,
-                         f"var/lib/mongoose/webhook.d/{monitoring_name}.yaml")
+        tmp_yaml_file = Path(f"/tmp/{monitoring_name}.yaml")
 
-        with open(yaml_file, 'w') as out_fd:
+        with open(tmp_yaml_file, 'w') as out_fd:
             yaml.safe_dump(payload, out_fd,
                            sort_keys=False,
                            default_flow_style=False,
                            encoding="utf-8",
                            allow_unicode=True)
 
+        yaml_file = Path(self._base_configuration_context.pirogue_working_root_dir,
+                         f"var/lib/mongoose/webhook.d/{monitoring_name}.yaml")
+
+        os.rename(tmp_yaml_file, yaml_file)
+
         return EMPTY
 
     def DeleteDeviceMonitoring(self, request, context):

pirogue-admin/pyproject.toml

@@ -9,7 +9,7 @@ namespaces = false
 
 [project]
 name = "pirogue-admin"
-version = "2.0.13"
+version = "2.0.14"
 description = "Admin management for the PiRogue"
 readme = "README.md"
 license = { text = "GPL-3.0-or-later" }