fix: make zapi rest CLI consistent for max-records arg

NetApp · Oct 27, 2023 · fc22034 · fc22034
1 parent 086ab01
commit fc22034
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/cmd/tools/zapi/zapi.go b/cmd/tools/zapi/zapi.go
@@ -343,7 +343,9 @@ func init() {
 	flags.StringVarP(&args.Attr, "attr", "t", "", "ZAPI attribute to show")
 	flags.StringVarP(&args.Object, "object", "o", "", "ZapiPerf object to show")
 	flags.StringSliceVarP(&args.Counters, "counter", "c", []string{}, "ZapiPerf counter(s) to show. Can be specified multiple times")
-	flags.IntVarP(&args.MaxRecords, "max", "m", 100, "max-records: max instances per API request")
+	flags.IntVarP(&args.MaxRecords, "max-records", "m", 100, "max instances per API request")
+	flags.IntVar(&args.MaxRecords, "max", 100, "max instances per API request (Deprecated: Use --max-records instead)")
+	_ = flags.MarkDeprecated("max", "Please use --max-records instead")
 	flags.StringSliceVarP(&args.Parameters, "parameters", "r", []string{}, "parameter to add to the ZAPI query")
 	flags.StringVar(&args.Config, "config", configPath, "harvest config file path")