Rename package-version parameter to just version (#517)

kudobuilder · Jul 9, 2019 · beca886 · beca886
1 parent 4cf6a31
commit beca886
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/pkg/kudoctl/cmd/install.go b/pkg/kudoctl/cmd/install.go
@@ -99,7 +99,7 @@ func newInstallCmd() *cobra.Command {
 	installCmd.Flags().StringVar(&options.InstanceName, "instance", "", "The instance name. (default to Operator name)")
 	installCmd.Flags().StringVar(&options.Namespace, "namespace", "default", "The namespace used for the package installation. (default \"default\"")
 	installCmd.Flags().StringArrayVarP(&parameters, "parameter", "p", nil, "The parameter name and value separated by '='")
-	installCmd.Flags().StringVar(&options.PackageVersion, "package-version", "", "A specific package version on the official GitHub repo. (default to the most recent)")
+	installCmd.Flags().StringVar(&options.PackageVersion, "version", "v", "A specific package version on the official GitHub repo. (default to the most recent)")
 	installCmd.Flags().BoolVar(&options.SkipInstance, "skip-instance", false, "If set, install will install the Operator and OperatorVersion, but not an instance. (default \"false\")")
 
 	const usageFmt = "Usage:\n  %s\n\nFlags:\n%s"

diff --git a/pkg/kudoctl/util/repo/repo_index.go b/pkg/kudoctl/util/repo/repo_index.go
@@ -83,7 +83,7 @@ func parseIndexFile(data []byte) (*IndexFile, error) {
 func (i IndexFile) GetByNameAndVersion(name, version string) (*BundleVersion, error) {
 	vs, ok := i.Entries[name]
 	if !ok || len(vs) == 0 {
-		return nil, fmt.Errorf("no operator of given name %s", name)
+		return nil, fmt.Errorf("no operator of given name %s found", name)
 	}
 
 	for _, ver := range vs {