Merge pull request #27 from galal-hussein/helmv3

changes for helm v3

pkg/apis/helm.cattle.io/v1/types.go

@@ -23,6 +23,7 @@ type HelmChartSpec struct {
 	Repo            string                        `json:"repo,omitempty"`
 	Set             map[string]intstr.IntOrString `json:"set,omitempty"`
 	ValuesContent   string                        `json:"valuesContent,omitempty"`
+	HelmVersion     string                        `json:"helmVersion,omitempty"`
 }
 
 type HelmChartStatus struct {

pkg/helm/controller.go

@@ -39,7 +39,7 @@ type Controller struct {
 }
 
 const (
-	image = "rancher/klipper-helm:v0.1.5"
+	image = "rancher/klipper-helm:v0.2.0"
 	label = "helmcharts.helm.cattle.io/chart"
 	name  = "helm-controller"
 )
@@ -199,6 +199,22 @@ func job(chart *helmv1.HelmChart) (*batch.Job, *core.ConfigMap) {
 									Name:  "VALUES_HASH",
 									Value: hex.EncodeToString(valuesHash[:]),
 								},
+								{
+									Name:  "HELM_DRIVER",
+									Value: "secret",
+								},
+								{
+									Name: "CHART_NAMESPACE",
+									Value: chart.Namespace,
+								},
+								{
+									Name: "CHART",
+									Value: chart.Spec.Chart,
+								},
+								{
+									Name: "HELM_VERSION",
+									Value: chart.Spec.HelmVersion,
+								},
 							},
 						},
 					},
@@ -298,15 +314,12 @@ func args(chart *helmv1.HelmChart) []string {
 	if chart.DeletionTimestamp != nil {
 		return []string{
 			"delete",
-			"--purge", chart.Name,
 		}
 	}
 
 	spec := chart.Spec
 	args := []string{
 		"install",
-		"--name", chart.Name,
-		spec.Chart,
 	}
 	if spec.TargetNamespace != "" {
 		args = append(args, "--namespace", spec.TargetNamespace)

pkg/helm/controller_test.go

@@ -32,7 +32,7 @@ func TestDeleteJob(t *testing.T) {
 func TestInstallArgs(t *testing.T) {
 	assert := assert.New(t)
 	stringArgs := strings.Join(args(NewChart()), " ")
-	assert.Equal("install --name traefik stable/traefik --set-string rbac.enabled=true --set-string ssl.enabled=true", stringArgs)
+	assert.Equal("install --set-string rbac.enabled=true --set-string ssl.enabled=true", stringArgs)
 }
 
 func TestDeleteArgs(t *testing.T) {
@@ -41,7 +41,7 @@ func TestDeleteArgs(t *testing.T) {
 	deleteTime := v12.NewTime(time.Time{})
 	chart.DeletionTimestamp = &deleteTime
 	stringArgs := strings.Join(args(chart), " ")
-	assert.Equal("delete --purge traefik", stringArgs)
+	assert.Equal("delete", stringArgs)
 }
 
 func NewChart() *v1.HelmChart {