kepctl: add support to query all SIGs

Signed-off-by: Nabarun Pal <[email protected]>

Author: palnabarun

cmd/kepctl/query.go

@@ -40,7 +40,7 @@ func buildQueryCommand(k *kepctl.Client) *cobra.Command {
 	}
 
 	f := cmd.Flags()
-	f.StringSliceVar(&opts.SIG, "sig", nil, "SIG")
+	f.StringSliceVar(&opts.SIG, "sig", nil, "SIG. If not specified, KEPs from all SIGs are shown.")
 	f.StringSliceVar(&opts.Status, "status", nil, "Status")
 	f.StringSliceVar(&opts.Stage, "stage", nil, "Stage")
 	f.StringSliceVar(&opts.PRRApprover, "prr", nil, "Prod Readiness Approver")

pkg/kepctl/query.go

@@ -55,17 +55,21 @@ type QueryOpts struct {
 	Output      string
 }
 
-
+// Validate checks the query options and cleans them up if needed
 func (c *QueryOpts) Validate() error {
+	groups := util.Groups()
 	if len(c.SIG) > 0 {
-		sigs, err := selectByRegexp(util.Groups(), c.SIG)
+		sigs, err := selectByRegexp(groups, c.SIG)
 		if err != nil {
 			return err
 		}
 		if len(sigs) == 0 {
 			return fmt.Errorf("No SIG matches any of the passed regular expressions")
 		}
 		c.SIG = sigs
+	} else {
+		// if no SIGs are passed, list KEPs from all SIGs
+		c.SIG = groups
 	}
 
 	// check if the Output specified is one of "", "json" or "yaml"