From ab6c6d2a05beff6b816593cc79234819c4bcc936 Mon Sep 17 00:00:00 2001 From: Jonathon Anderson Date: Wed, 6 Nov 2024 15:42:46 -0700 Subject: [PATCH] Mark two upgrade arguments as required Based on TSC feedback. Signed-off-by: Jonathon Anderson --- internal/app/wwctl/upgrade/nodes/cobra.go | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/internal/app/wwctl/upgrade/nodes/cobra.go b/internal/app/wwctl/upgrade/nodes/cobra.go index d138130a..b652562b 100644 --- a/internal/app/wwctl/upgrade/nodes/cobra.go +++ b/internal/app/wwctl/upgrade/nodes/cobra.go @@ -32,6 +32,12 @@ func init() { Command.Flags().BoolVar(&replaceOverlays, "replace-overlays", false, "Replace 'wwinit' and 'generic' overlays with their split replacements") Command.Flags().StringVarP(&inputPath, "input-path", "i", node.ConfigFile, "Path to a legacy nodes.conf") Command.Flags().StringVarP(&outputPath, "output-path", "o", node.ConfigFile, "Path to write the upgraded nodes.conf to") + if err := Command.MarkFlagRequired("add-defaults"); err != nil { + panic(err) + } + if err := Command.MarkFlagRequired("replace-overlays"); err != nil { + panic(err) + } } func UpgradeNodesConf(cmd *cobra.Command, args []string) error {