fix: unrelated fix to make certain resync cmd parms mandatory

Seagate · Oct 17, 2024 · 21f206d · 21f206d
1 parent 1b3d95a
commit 21f206d
Show file tree

Hide file tree

Showing 4 changed files with 4 additions and 1 deletion.
diff --git a/cmd/cfm-cli/cmd/resync.go b/cmd/cfm-cli/cmd/resync.go
@@ -6,7 +6,6 @@ import (
 	"github.com/spf13/cobra"
 )
 
-// deleteCmd represents the delete command
 var resyncCmd = &cobra.Command{
 	Use:   "resync <device> [<flags>]",
 	Short: "Resynchronize the cfm service to the specified hardware",

diff --git a/cmd/cfm-cli/cmd/resyncAppliance.go b/cmd/cfm-cli/cmd/resyncAppliance.go
@@ -40,6 +40,7 @@ func init() {
 	initCommonPersistentFlags(resyncApplianceCmd)
 
 	resyncApplianceCmd.Flags().StringP(flags.APPLIANCE_ID, flags.APPLIANCE_ID_SH, flags.ID_DFLT, "ID of composable memory appliance (CMA)")
+	resyncApplianceCmd.MarkFlagRequired(flags.APPLIANCE_ID)
 
 	//Add command to parent
 	resyncCmd.AddCommand(resyncApplianceCmd)

diff --git a/cmd/cfm-cli/cmd/resyncBlade.go b/cmd/cfm-cli/cmd/resyncBlade.go
@@ -40,7 +40,9 @@ func init() {
 	initCommonPersistentFlags(resyncBladeCmd)
 
 	resyncBladeCmd.Flags().StringP(flags.APPLIANCE_ID, flags.APPLIANCE_ID_SH, flags.ID_DFLT, "ID of blade's appliance")
+	resyncBladeCmd.MarkFlagRequired(flags.APPLIANCE_ID)
 	resyncBladeCmd.Flags().StringP(flags.BLADE_ID, flags.BLADE_ID_SH, flags.ID_DFLT, "ID of blade")
+	resyncBladeCmd.MarkFlagRequired(flags.BLADE_ID)
 
 	//Add command to parent
 	resyncCmd.AddCommand(resyncBladeCmd)

diff --git a/cmd/cfm-cli/cmd/resyncHost.go b/cmd/cfm-cli/cmd/resyncHost.go
@@ -40,6 +40,7 @@ func init() {
 	initCommonPersistentFlags(resyncHostCmd)
 
 	resyncHostCmd.Flags().StringP(flags.HOST_ID, flags.HOST_ID_SH, flags.ID_DFLT, "ID of CXL host")
+	resyncHostCmd.MarkFlagRequired(flags.HOST_ID)
 
 	//Add command to parent
 	resyncCmd.AddCommand(resyncHostCmd)