From 2c54f56cde54c1e82f72de674fe0b14a87fae852 Mon Sep 17 00:00:00 2001 From: Tom Dohrmann Date: Wed, 3 Jul 2024 13:28:54 +0200 Subject: [PATCH] generate: rename --workload-owner-key to --add-workload-owner-key This makes it more obvious to the user that this flag only ever adds keys and never removes existing keys. --- cli/cmd/generate.go | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/cli/cmd/generate.go b/cli/cmd/generate.go index aa8c891b87..a8f8620a05 100644 --- a/cli/cmd/generate.go +++ b/cli/cmd/generate.go @@ -63,7 +63,7 @@ subcommands.`, cmd.Flags().StringP("genpolicy-cache-path", "c", layersCacheFilename, "path to cache for the cache (.json) file containing the image layers") cmd.Flags().StringP("manifest", "m", manifestFilename, "path to manifest (.json) file") cmd.Flags().String("reference-values", "", "set the default reference values used for attestation (one of: aks)") - cmd.Flags().StringArrayP("workload-owner-key", "w", []string{workloadOwnerPEM}, + cmd.Flags().StringArrayP("add-workload-owner-key", "w", []string{workloadOwnerPEM}, "path to workload owner key (.pem) file (can be passed more than once)") cmd.Flags().StringArray("seedshare-owner-key", []string{seedshareOwnerPEM}, "path to seedshare owner key (.pem) file (can be passed more than once)") @@ -74,7 +74,7 @@ subcommands.`, must(cmd.MarkFlagFilename("policy", "rego")) must(cmd.MarkFlagFilename("settings", "json")) must(cmd.MarkFlagFilename("manifest", "json")) - cmd.MarkFlagsMutuallyExclusive("workload-owner-key", "disable-updates") + cmd.MarkFlagsMutuallyExclusive("add-workload-owner-key", "disable-updates") return cmd } @@ -557,7 +557,7 @@ func parseGenerateFlags(cmd *cobra.Command) (*generateFlags, error) { if !slices.Contains([]string{"", "aks"}, referenceValues) { return nil, fmt.Errorf("unknown reference values") } - workloadOwnerKeys, err := cmd.Flags().GetStringArray("workload-owner-key") + workloadOwnerKeys, err := cmd.Flags().GetStringArray("add-workload-owner-key") if err != nil { return nil, err } @@ -587,7 +587,7 @@ func parseGenerateFlags(cmd *cobra.Command) (*generateFlags, error) { if !cmd.Flags().Changed("manifest") { manifestPath = filepath.Join(workspaceDir, manifestFilename) } - if !cmd.Flags().Changed("workload-owner-key") { + if !cmd.Flags().Changed("add-workload-owner-key") { workloadOwnerKeys = []string{filepath.Join(workspaceDir, workloadOwnerKeys[0])} } if !cmd.Flags().Changed("seedshare-owner-key") {