cephadm: propagate smbmetrics image via config_blobs

Current code assumes single image for each daemon-service. However,
Samba's metrics-exporter requires different image then the one used by
main samba server. Use the auxiliary 'config_blobs' map to propagate
the metrics image downwards to samba-service, plus metrics port number.

Signed-off-by: Shachar Sharon <[email protected]>

Author: synarete

src/pybind/mgr/cephadm/services/smb.py

@@ -16,6 +16,7 @@
 
 class SMBService(CephService):
     TYPE = 'smb'
+    DEFAULT_EXPORTER_PORT = 9922
     smb_pool = '.smb'  # minor layering violation. try to clean up later.
 
     def config(self, spec: ServiceSpec) -> None:
@@ -79,6 +80,11 @@ def generate_config(
                 smb_spec, daemon_spec.daemon_id, ceph_users
             )
         )
+        config_blobs['metrics_image'] = (
+            self.mgr.container_image_samba_metrics
+        )
+        config_blobs['metrics_port'] = SMBService.DEFAULT_EXPORTER_PORT
+
         logger.debug('smb generate_config: %r', config_blobs)
         self._configure_cluster_meta(smb_spec, daemon_spec)
         return config_blobs, []

src/pybind/mgr/cephadm/tests/test_services.py

@@ -3175,6 +3175,8 @@ def test_deploy_smb(
                 'config': '',
                 'keyring': '[client.smb.config.tango.briskly]\nkey = None\n',
                 'config_auth_entity': 'client.smb.config.tango.briskly',
+                'metrics_image': 'quay.io/samba.org/samba-metrics:latest',
+                'metrics_port': 9922,
             },
         }
         with with_host(cephadm_module, 'hostx'):
@@ -3245,6 +3247,8 @@ def test_deploy_smb_join_dns(
                     '[client.smb.fs.fs2.share3]\nkey = None\n'
                 ),
                 'config_auth_entity': 'client.smb.config.tango.briskly',
+                'metrics_image': 'quay.io/samba.org/samba-metrics:latest',
+                'metrics_port': 9922,
             },
         }
         with with_host(cephadm_module, 'hostx'):