Merge pull request ceph#61765 from Kushal-deb/fix_orch_ls_to_display_smb_ports

cephadm: ceph orch ls to Display SMB Service Ports

Reviewed-by: Adam King <[email protected]>
Reviewed-by: John Mulligan <[email protected]>

Author: adk3798

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

@@ -3738,10 +3738,12 @@ def test_deploy_smb(
             'name': 'smb.tango.briskly',
             'image': '',
             'deploy_arguments': [],
-            'params': {},
+            'params': {
+                "tcp_ports": [445, 9922]
+            },
             'meta': {
                 'service_name': 'smb',
-                'ports': [],
+                'ports': [445, 9922],
                 'ip': None,
                 'deployed_by': [],
                 'rank': None,
@@ -3801,10 +3803,12 @@ def test_deploy_smb_join_dns(
             'name': 'smb.tango.briskly',
             'image': '',
             'deploy_arguments': [],
-            'params': {},
+            'params': {
+                'tcp_ports': [445, 9922]
+            },
             'meta': {
                 'service_name': 'smb',
-                'ports': [],
+                'ports': [445, 9922],
                 'ip': None,
                 'deployed_by': [],
                 'rank': None,

src/python-common/ceph/deployment/service_spec.py

@@ -3179,6 +3179,9 @@ def _derive_cluster_uri(self, uri: str, objname: str) -> str:
         uri = 'rados://' + '/'.join(parts)
         return uri
 
+    def get_port_start(self) -> List[int]:
+        return [445, 9922]  # SMB service runs on port 445, and smbmetrics uses 9922
+
     def strict_cluster_ip_specs(self) -> List[Dict[str, Any]]:
         return [s.to_strict() for s in (self.cluster_public_addrs or [])]