app: make some options optional

Longhorn 7672 Signed-off-by: Derek Su <[email protected]>
longhorn · Feb 7, 2024 · 0c35f68 · 0c35f68
1 parent 21e0aae
commit 0c35f68
Showing 1 changed file with 4 additions and 6 deletions.
diff --git a/app/cmd/basic/bdev_nvme.go b/app/cmd/basic/bdev_nvme.go
@@ -37,19 +37,17 @@ func BdevNvmeAttachControllerCmd() cli.Command {
 				Required: true,
 			},
 			cli.StringFlag{
-				Name:     "subnqn",
-				Usage:    "NVMe-oF target subnqn. It can be the nvmf subsystem nqn",
-				Required: true,
+				Name:  "subnqn",
+				Usage: "NVMe-oF target subnqn. It can be the nvmf subsystem nqn",
 			},
 			cli.StringFlag{
 				Name:     "traddr",
 				Usage:    "NVMe-oF target address: a ip or BDF",
 				Required: true,
 			},
 			cli.StringFlag{
-				Name:     "trsvcid",
-				Usage:    "NVMe-oF target trsvcid: a port number",
-				Required: true,
+				Name:  "trsvcid",
+				Usage: "NVMe-oF target trsvcid: a port number",
 			},
 			cli.StringFlag{
 				Name:  "trtype",