Add hpa and env to helm create

Signed-off-by: Naseem Ullah <naseemkullah@gmail.com>

cmd/helm/create_test.go

@@ -143,7 +143,7 @@ func TestCreateStarterCmd(t *testing.T) {
 		t.Errorf("Wrong API version: %q", c.Metadata.ApiVersion)
 	}
 
-	expectedTemplateCount := 7
+	expectedTemplateCount := 8
 	if l := len(c.Templates); l != expectedTemplateCount {
 		t.Errorf("Expected %d templates, got %d", expectedTemplateCount, l)
 	}

pkg/chartutil/create.go

@@ -40,6 +40,8 @@ const (
 	IngressFileName = "ingress.yaml"
 	// DeploymentName is the name of the example deployment file.
 	DeploymentName = "deployment.yaml"
+	// HorizontalPodAutoscalerName is the name of the example hpa file.
+	HorizontalPodAutoscalerName = "hpa.yaml"
 	// ServiceName is the name of the example service file.
 	ServiceName = "service.yaml"
 	// NotesName is the name of the example NOTES.txt file.
@@ -56,32 +58,23 @@ const defaultValues = `# Default values for %s.
 # This is a YAML-formatted file.
 # Declare variables to be passed into your templates.
 
-replicaCount: 1
+nameOverride: ""
+fullnameOverride: ""
 
+deployment:
   image:
     repository: nginx
     tag: stable
     pullPolicy: IfNotPresent
 
-nameOverride: ""
+  replicaCount: 1
-fullnameOverride: ""
-
-service:
-  type: ClusterIP
-  port: 80
 
-ingress:
+  hpa:
     enabled: false
-  annotations: {}
+    minReplicas: 1
-    # kubernetes.io/ingress.class: nginx
+    maxReplicas: 10
-    # kubernetes.io/tls-acme: "true"
+    targetCPUUtilizationPercentage: 80
-  paths: []
+    targetMemoryUtilizationPercentage: 80
-  hosts:
-    - chart-example.local
-  tls: []
-  #  - secretName: chart-example-tls
-  #    hosts:
-  #      - chart-example.local
 
   resources: {}
     # We usually recommend not to specify default resources and to leave this as a conscious
@@ -95,11 +88,34 @@ resources: {}
     #   cpu: 100m
     #   memory: 128Mi
 
+  env: {}
+    # - name: FOO
+    #   value: bar
+    # - name: BAZ
+    #   value: qux
+
   nodeSelector: {}
 
   tolerations: []
 
   affinity: {}
+
+service:
+  type: ClusterIP
+  port: 80
+
+ingress:
+  enabled: false
+  annotations: {}
+    # kubernetes.io/ingress.class: nginx
+    # kubernetes.io/tls-acme: "true"
+  paths: []
+  hosts:
+    - chart-example.local
+  tls: []
+  #  - secretName: chart-example-tls
+  #    hosts:
+  #      - chart-example.local
 `
 
 const defaultIgnore = `# Patterns to ignore when building packages.
@@ -178,7 +194,7 @@ metadata:
     app.kubernetes.io/instance: {{ .Release.Name }}
     app.kubernetes.io/managed-by: {{ .Release.Service }}
 spec:
-  replicas: {{ .Values.replicaCount }}
+  replicas: {{ .Values.deployment.replicaCount }}
   selector:
     matchLabels:
       app.kubernetes.io/name: {{ include "<CHARTNAME>.name" . }}
@@ -191,8 +207,8 @@ spec:
     spec:
       containers:
         - name: {{ .Chart.Name }}
-          image: "{{ .Values.image.repository }}:{{ .Values.image.tag }}"
+          image: "{{ .Values.deployment.image.repository }}:{{ .Values.deployment.image.tag }}"
-          imagePullPolicy: {{ .Values.image.pullPolicy }}
+          imagePullPolicy: {{ .Values.deployment.image.pullPolicy }}
           ports:
             - name: http
               containerPort: 80
@@ -206,20 +222,54 @@ spec:
               path: /
               port: http
           resources:
-            {{- toYaml .Values.resources | nindent 12 }}
+            {{- toYaml .Values.deployment.resources | nindent 12 }}
-      {{- with .Values.nodeSelector }}
+          env:
+            {{- toYaml .Values.deployment.env | nindent 12 }}
+    {{- with .Values.deployment.nodeSelector }}
       nodeSelector:
         {{- toYaml . | nindent 8 }}
     {{- end }}
-    {{- with .Values.affinity }}
+    {{- with .Values.deployment.affinity }}
       affinity:
         {{- toYaml . | nindent 8 }}
     {{- end }}
-    {{- with .Values.tolerations }}
+    {{- with .Values.deployment.tolerations }}
       tolerations:
         {{- toYaml . | nindent 8 }}
     {{- end }}
 `
+const defaultHorizontalPodAutoscaler = `{{- if .Values.deployment.hpa.enabled }}
+apiVersion: autoscaling/v2beta2
+kind: HorizontalPodAutoscaler
+metadata:
+  labels:
+    app.kubernetes.io/name: {{ include "<CHARTNAME>.name" . }}
+    helm.sh/chart: {{ include "<CHARTNAME>.chart" . }}
+    app.kubernetes.io/instance: {{ .Release.Name }}
+    app.kubernetes.io/managed-by: {{ .Release.Service }}
+  name: {{ template "<CHARTNAME>.fullname" . }}
+spec:
+  scaleTargetRef:
+    apiVersion: apps/v1
+    kind: Deployment
+    name: {{ template "<CHARTNAME>.fullname" . }}
+  minReplicas: {{ .Values.hpa.minReplicas }}
+  maxReplicas: {{ .Values.hpa.maxReplicas }}
+  metrics:
+  {{- if .Values.autoscaling.targetCPUUtilizationPercentage }}
+    - type: Resource
+      resource:
+        name: cpu
+        targetAverageUtilization: {{ .Values.hpa.targetCPUUtilizationPercentage }}
+  {{- end }}
+  {{- if .Values.autoscaling.targetMemoryUtilizationPercentage }}
+    - type: Resource
+      resource:
+        name: memory
+        targetAverageUtilization: {{ .Values.hpa.targetMemoryUtilizationPercentage }}
+  {{- end }}
+{{- end }}
+`
 
 const defaultService = `apiVersion: v1
 kind: Service
@@ -413,6 +463,11 @@ func Create(chartfile *chart.Metadata, dir string) (string, error) {
 			path:    filepath.Join(cdir, TemplatesDir, DeploymentName),
 			content: Transform(defaultDeployment, "<CHARTNAME>", chartfile.Name),
 		},
+		{
+			// hpa.yaml
+			path:    filepath.Join(cdir, TemplatesDir, HorizontalPodAutoscalerName),
+			content: Transform(defaultHorizontalPodAutoscaler, "<CHARTNAME>", chartfile.Name),
+		},
 		{
 			// service.yaml
 			path:    filepath.Join(cdir, TemplatesDir, ServiceName),