Merge pull request #431 from technosophos/feat/helm-chart-create

feat(cli): add 'helm chart create'

cmd/helm/chart.go

@@ -17,7 +17,11 @@ limitations under the License.
 package main
 
 import (
+	"errors"
+	"path/filepath"
+
 	"github.com/codegangsta/cli"
+	"github.com/kubernetes/helm/pkg/chart"
 )
 
 func init() {
@@ -42,7 +46,10 @@ func chartCommands() cli.Command {
 				ArgsUsage: "CHART",
 			},
 			{
-				Name: "scaffold",
+				Name:      "create",
+				Usage:     "Create a new chart directory and set up base files and directories.",
+				ArgsUsage: "CHARTNAME",
+				Action:    func(c *cli.Context) { run(c, createChart) },
 			},
 			{
 				Name:      "list",
@@ -57,3 +64,21 @@ func chartCommands() cli.Command {
 		},
 	}
 }
+
+func createChart(c *cli.Context) error {
+	args := c.Args()
+	if len(args) < 1 {
+		return errors.New("'helm create' requires a chart name as an argument")
+	}
+
+	dir, name := filepath.Split(args[0])
+
+	cf := &chart.Chartfile{
+		Name:        name,
+		Description: "Created by Helm",
+		Version:     "0.1.0",
+	}
+
+	_, err := chart.Create(cf, dir)
+	return err
+}

cmd/helm/create.go

@@ -1,56 +0,0 @@
-/*
-Copyright 2016 The Kubernetes Authors All rights reserved.
-
-Licensed under the Apache License, Version 2.0 (the "License");
-you may not use this file except in compliance with the License.
-You may obtain a copy of the License at
-
-    http://www.apache.org/licenses/LICENSE-2.0
-
-Unless required by applicable law or agreed to in writing, software
-distributed under the License is distributed on an "AS IS" BASIS,
-WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-See the License for the specific language governing permissions and
-limitations under the License.
-*/
-
-package main
-
-import (
-	"errors"
-	"path/filepath"
-
-	"github.com/codegangsta/cli"
-	"github.com/kubernetes/helm/pkg/chart"
-)
-
-func init() {
-	addCommands(createCmd())
-}
-
-func createCmd() cli.Command {
-	return cli.Command{
-		Name:      "create",
-		Usage:     "Create a new local chart for editing.",
-		Action:    func(c *cli.Context) { run(c, create) },
-		ArgsUsage: "NAME",
-	}
-}
-
-func create(c *cli.Context) error {
-	args := c.Args()
-	if len(args) < 1 {
-		return errors.New("'helm create' requires a chart name as an argument")
-	}
-
-	dir, name := filepath.Split(args[0])
-
-	cf := &chart.Chartfile{
-		Name:        name,
-		Description: "Created by Helm",
-		Version:     "0.1.0",
-	}
-
-	_, err := chart.Create(cf, dir)
-	return err
-}

docs/workflow/developer-workflows.md

@@ -320,7 +320,7 @@ chart, it bundles the chart, and sends the entire chart, not just the
 values.
 
 ```
-$ helm create mychart
+$ helm chart create mychart
 Created mychart/Chart.yaml
 $ helm lint mychart
 OK
@@ -334,12 +334,12 @@ $ helm redeploy skinny-pigeon
 Redeployed skinny-pigeon
 ```
 
-- `helm create` and `helm lint` are client side operations
+- `helm chart create` and `helm lint` are client side operations
 - `helm deploy`, `helm status`, and `helm redeploy` are explained above.
 
 General pattern for create:
 ```
-helm create [--from NAME] CHARTNAME
+helm chart create [--from NAME] CHARTNAME
 ```
 
 Where `NAME` will result in fetching the generated values from the cluster.