mgr/dashboard: Allow empty port value when adding a listener in NVMEoF CLI

Fixes: https://tracker.ceph.com/issues/76410
Signed-off-by: Gil Bregman <gbregman@il.ibm.com>

diff --git a/src/pybind/mgr/dashboard/controllers/nvmeof.py b/src/pybind/mgr/dashboard/controllers/nvmeof.py
index 7f58db0b056431f696d29aac593ea2df9077aad9..794d6e4b5774b0ae9d05de280e0db1490bb29d22 100644 (file)
--- a/src/pybind/mgr/dashboard/controllers/nvmeof.py
+++ b/src/pybind/mgr/dashboard/controllers/nvmeof.py
@@ -855,7 +855,7 @@ else:
                 "nqn": Param(str, "NVMeoF subsystem NQN"),
                 "host_name": Param(str, "NVMeoF hostname"),
                 "traddr": Param(str, "NVMeoF transport address"),
-                "trsvcid": Param(int, "NVMeoF transport service port", True, 4420),
+                "trsvcid": Param(int, "NVMeoF transport service port", True, None),
                 "adrfam": Param(int, "NVMeoF address family (0 - IPv4, 1 - IPv6)", True, 0),
                 "gw_group": Param(str, "NVMeoF gateway group", True, None),
                 "server_address": Param(str, "NVMeoF gateway address", True, None),
@@ -873,7 +873,7 @@ else:
             nqn: str,
             host_name: str,
             traddr: str,
-            trsvcid: int = 4420,
+            trsvcid: Optional[int] = None,
             adrfam: int = 0,  # IPv4,
             gw_group: Optional[str] = None,
             server_address: Optional[str] = None,
@@ -889,7 +889,7 @@ else:
                     nqn=nqn,
                     host_name=host_name,
                     traddr=traddr,
-                    trsvcid=int(trsvcid),
+                    trsvcid=int(trsvcid) if trsvcid is not None else None,
                     adrfam=int(adrfam),
                     secure=str_to_bool(secure),
                     verify_host_name=str_to_bool(verify_host_name),
@@ -917,7 +917,7 @@ else:
                 "nqn": Param(str, "NVMeoF subsystem NQN"),
                 "host_name": Param(str, "NVMeoF hostname"),
                 "traddr": Param(str, "NVMeoF transport address"),
-                "trsvcid": Param(int, "NVMeoF transport service port", True, 4420),
+                "trsvcid": Param(int, "NVMeoF transport service port"),
                 "adrfam": Param(int, "NVMeoF address family (0 - IPv4, 1 - IPv6)", True, 0),
                 "gw_group": Param(str, "NVMeoF gateway group", True, None),
                 "server_address": Param(str, "NVMeoF gateway address", True, None),
@@ -930,7 +930,7 @@ else:
             nqn: str,
             host_name: str,
             traddr: str,
-            trsvcid: int = 4420,
+            trsvcid: int,
             adrfam: int = 0,  # IPv4
             force: bool = False,
             gw_group: Optional[str] = None,

diff --git a/src/pybind/mgr/dashboard/openapi.yaml b/src/pybind/mgr/dashboard/openapi.yaml
index a43c312b4e67b742a2c5d162d5a45bd78ba180b2..d0c4476e3be66fe48c1d42cd0f54c0e2e55cc5e5 100644 (file)
--- a/src/pybind/mgr/dashboard/openapi.yaml
+++ b/src/pybind/mgr/dashboard/openapi.yaml
@@ -14004,7 +14004,6 @@ paths:
                   description: NVMeoF transport address
                   type: string
                 trsvcid:
-                  default: 4420
                   description: NVMeoF transport service port
                   type: integer
                 verify_host_name:
@@ -14049,7 +14048,7 @@ paths:
       summary: Create a new NVMeoF listener
       tags:
       - NVMe-oF Subsystem Listener
-  /api/nvmeof/subsystem/{nqn}/listener/{host_name}/{traddr}:
+  /api/nvmeof/subsystem/{nqn}/listener/{host_name}/{traddr}/{trsvcid}:
     delete:
       parameters:
       - description: NVMeoF subsystem NQN
@@ -14070,10 +14069,10 @@ paths:
         required: true
         schema:
           type: string
-      - default: 4420
-        description: NVMeoF transport service port
-        in: query
+      - description: NVMeoF transport service port
+        in: path
         name: trsvcid
+        required: true
         schema:
           type: integer
       - default: 0