Fix lego issue (#388)

* Fixes 385

* Address PR comments and add monitoring service

* Address PR comments

stable/nginx-lego/Chart.yaml

@@ -1,5 +1,5 @@
 name: nginx-lego
-version: 0.1.0
+version: 0.2.0
 description: Chart for nginx-ingress-controller and kube-lego
 keywords:
 - kube-lego

stable/nginx-lego/templates/default-service.yaml

@@ -10,4 +10,4 @@ spec:
   - port: 80
     targetPort: 8080
   selector:
-    app: {{ template "fullname" . }}-default-backend
+    app: {{ template "fullname" . }}-default-backend

stable/nginx-lego/templates/lego-configmap.yaml

@@ -3,7 +3,9 @@ apiVersion: v1
 metadata:
   name: {{ template "fullname" . }}-lego
 data:
+  # modify this to specify your address
   lego.email: {{ .Values.lego.configmap.email | quote }}
+  # configure letencrypt's production api
   lego.url: {{ .Values.lego.configmap.url | quote }}
 kind: ConfigMap
-{{- end }}
+{{- end }}

stable/nginx-lego/templates/lego-deployment.yaml

@@ -8,7 +8,7 @@ spec:
   template:
     metadata:
       labels:
-        app: {{ template "fullname" . }}-lego
+        app: kube-lego
     spec:
       containers:
       - name: {{ .Chart.Name }}-lego
@@ -37,4 +37,4 @@ spec:
               fieldPath: status.podIP
         resources:
 {{ toYaml .Values.nginx.resources | indent 10 }}
-{{- end }}
+{{- end }}

stable/nginx-lego/templates/nginx-deployment.yaml

@@ -27,7 +27,9 @@ spec:
         ports:
         - containerPort: 80
         - containerPort: 443
+{{- if .Values.nginx.monitoring }}
         - containerPort: 8080
+{{- end }}
         resources:
 {{ toYaml .Values.nginx.resources | indent 10 }}
         args:

stable/nginx-lego/templates/nginx-monitoring.yaml

@@ -1,15 +1,15 @@
-{{- if .Values.lego.enabled }}
+{{- if .Values.nginx.monitoring }}
 apiVersion: v1
 kind: Service
 metadata:
-  name: {{ template "fullname" . }}-lego
+  name: {{ template "fullname" . }}-monitoring
   labels:
     chart: "{{ .Chart.Name }}-{{ .Chart.Version }}"
 spec:
   type: ClusterIP
   ports:
-  - port: 80
-    name: "lego"
+  - port: 18080
+    name: vts-stats
   selector:
-    app: {{ template "fullname" . }}-lego
+    app: {{ template "fullname" . }}
 {{- end }}

stable/nginx-lego/values.yaml

@@ -12,6 +12,7 @@ nginx:
     pullPolicy: IfNotPresent
   service:
     type: LoadBalancer
+  monitoring: false
   resources:
     limits:
       cpu: 1
@@ -69,4 +70,4 @@ lego:
       memory: 2Gi
     requests:
       cpu: 1
-      memory: 128Mi
+      memory: 128Mi