Skip to content

Commit c4d13eb

Browse files
authored
Merge pull request ceph#65841 from Hezko/nvmeof-cli-ns-list-no-nqn-fix
mgr/dashboard: ns list now support not passing nqn param Reviewed-by: Afreen Misbah <[email protected]>
2 parents b2747d6 + 0419a1a commit c4d13eb

File tree

2 files changed

+4
-4
lines changed

2 files changed

+4
-4
lines changed

src/pybind/mgr/dashboard/controllers/nvmeof.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -409,14 +409,14 @@ class NVMeoFNamespace(RESTController):
409409
@EndpointDoc(
410410
"List all NVMeoF namespaces in a subsystem",
411411
parameters={
412-
"nqn": Param(str, "NVMeoF subsystem NQN"),
412+
"nqn": Param(str, "NVMeoF subsystem NQN", True, None),
413413
"nsid": Param(str, "NVMeoF Namespace ID to filter by", True, None),
414414
"gw_group": Param(str, "NVMeoF gateway group", True, None),
415415
},
416416
)
417417
@convert_to_model(model.NamespaceList)
418418
@handle_nvmeof_error
419-
def list(self, nqn: str, nsid: Optional[str] = None,
419+
def list(self, nqn: Optional[str] = None, nsid: Optional[str] = None,
420420
gw_group: Optional[str] = None, traddr: Optional[str] = None):
421421
return NVMeoFClient(gw_group=gw_group, traddr=traddr).stub.list_namespaces(
422422
NVMeoFClient.pb2.list_namespaces_req(subsystem=nqn,

src/pybind/mgr/dashboard/openapi.yaml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -9366,10 +9366,10 @@ paths:
93669366
/api/nvmeof/subsystem/{nqn}/namespace:
93679367
get:
93689368
parameters:
9369-
- description: NVMeoF subsystem NQN
9369+
- allowEmptyValue: true
9370+
description: NVMeoF subsystem NQN
93709371
in: path
93719372
name: nqn
9372-
required: true
93739373
schema:
93749374
type: string
93759375
- allowEmptyValue: true

0 commit comments

Comments
 (0)