From: John Mulligan <jmulligan@redhat.com>
Date: Mon, 10 Nov 2025 21:15:29 +0000 (-0500)
Subject: cephadm: add pid file directory option to cfg watch sidecar
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=0f8163f1ddc7b1e00c3889e27b61d74d3d1a40d8;p=ceph-ci.git

cephadm: add pid file directory option to cfg watch sidecar

This will enable the config watch sidecar to signal processes
with a SIGHUP to tell them to reload configuration when config
watch has detected a configuration change. Currently only used
by keybridge.

Resolves: rhbz#2412278
Signed-off-by: John Mulligan <jmulligan@redhat.com>
(cherry picked from commit 54f2c897a68daf93252ea20d070d45e561f26718)
---

diff --git a/src/cephadm/cephadmlib/daemons/smb.py b/src/cephadm/cephadmlib/daemons/smb.py
index 24622d70f2b..46988c5791f 100644
--- a/src/cephadm/cephadmlib/daemons/smb.py
+++ b/src/cephadm/cephadmlib/daemons/smb.py
@@ -408,7 +408,11 @@ class ConfigWatchContainer(SambaContainerCommon):
         return 'configwatch'
 
     def args(self) -> List[str]:
-        return super().args() + ['update-config', '--watch']
+        return super().args() + [
+            'update-config',
+            '--watch',
+            f'--signal-pids-dir={_WANT_SIGNAL_DIR}',
+        ]
 
 
 class SMBMetricsContainer(ContainerCommon):