mirrors/talos
1
0
Fork 0
mirror of https://github.com/siderolabs/talos.git synced 2025-10-09 14:41:31 +02:00
Code Issues Packages Projects Releases Wiki Activity

fix: do not print out help string if the parameters are correct

Browse Source 
There was an issue that `talosctl apply config` version was printing out
the help even if arguments are correct.

Signed-off-by: Artem Chernyshev <artem.0xD2@gmail.com>
This commit is contained in:
Artem Chernyshev 2021-03-09 20:58:51 +03:00 committed by talos-bot
parent 56c95eace3
commit c2e353d6af
1 changed files with 4 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
cmd/talosctl/cmd/talos/apply-config.go
View File

@ -42,11 +42,13 @@ var applyConfigCmd = &cobra.Command{
) )
if len(args) > 0 { if len(args) > 0 {
cmd.Help() //nolint:errcheck
if args[0] != "config" && !strings.EqualFold(args[0], "machineconfig") { if args[0] != "config" && !strings.EqualFold(args[0], "machineconfig") {
cmd.Help() //nolint:errcheck
return fmt.Errorf("unknown positional argument %s", args[0]) return fmt.Errorf("unknown positional argument %s", args[0])
} else if cmd.CalledAs() == "apply-config" { } else if cmd.CalledAs() == "apply-config" {
cmd.Help() //nolint:errcheck
return fmt.Errorf("expected no positional arguments") return fmt.Errorf("expected no positional arguments")
} }
} }