From: John Mulligan <jmulligan@redhat.com>
Date: Mon, 10 Nov 2025 21:15:19 +0000 (-0500)
Subject: cephadm: add pidfile option to smb keybridge sidecar
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=d0c05583f9bc74d163f30749ca89e7d798a918f3;p=ceph-ci.git

cephadm: add pidfile option to smb keybridge sidecar

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

diff --git a/src/cephadm/cephadmlib/daemons/smb.py b/src/cephadm/cephadmlib/daemons/smb.py
index d4941a34d3b..24622d70f2b 100644
--- a/src/cephadm/cephadmlib/daemons/smb.py
+++ b/src/cephadm/cephadmlib/daemons/smb.py
@@ -56,6 +56,7 @@ _SCC = '/usr/bin/samba-container'
 _NODES_SUBCMD = [_SCC, 'ctdb-list-nodes']
 _MUTEX_SUBCMD = [_SCC, 'ctdb-rados-mutex']  # requires rados uri
 _ETC_SAMBA_TLS = '/etc/samba/tls'
+_WANT_SIGNAL_DIR = '/run/want_update_signal'
 
 
 class Features(enum.Enum):
@@ -462,6 +463,7 @@ class KeyBridgeContainer(SambaContainerCommon):
     def args(self) -> List[str]:
         args = super().args()
         assert self.cfg.keybridge, 'keybridge is not configured'
+        args.append(f'--pidfile={_WANT_SIGNAL_DIR}/keybridge.pid')
         args.append('keybridge')
         if self.cfg.keybridge.tls_files:
             cert_path = self.cfg.keybridge.tls_files.cert_interior_path