PR feedback

chart/keel/Chart.yaml

@@ -1,7 +1,7 @@
 apiVersion: v1
 name: keel
 description: Open source, tool for automating Kubernetes deployment updates. Keel is stateless, robust and lightweight.
-version: 0.6.0
+version: 0.6.1
 # Note that we use appVersion to get images tag, so make sure this is correct.
 appversion: 0.9.5
 keywords:

chart/keel/README.md

@@ -106,6 +106,7 @@ The following table lists has the main configurable parameters (polling, trigger
 | `hipchat.user_name`               | Hipchat username in Jabber format      |                                                           |
 | `hipchat.password`                | Hipchat password for approvals user    |                                                           |
 | `googleApplicationCredentials`    | GCP Service account key configurable   |                                                           |
+| `notification_level`              | Keel notification level                | `info`                                                    |
 
 Specify each parameter using the `--set key=value[,key=value]` argument to `helm install`.
 

chart/keel/templates/deployment.yaml

@@ -105,7 +105,7 @@ spec:
               value: "{{ .Values.hipchat.password }}"
 {{- end }}
             - name: NOTIFICATION_LEVEL
-              value: {{ default "info" .Values.notification_level | quote }}
+              value: "{{ .Values.notification_level }}"
 {{- if .Values.debug }}
             # Enable debug logging
             - name: DEBUG

chart/keel/values.yaml

@@ -28,6 +28,9 @@ gcr:
   pubSub:
     enabled: false
 
+# Notification level (debug, info, success, warn, error, fatal)
+notification_level: info
+
 # Webhook Notification
 # Remote webhook endpoint for notification delivery
 webhook:

deployment/deployment-norbac-whr-sidecar.yaml

@@ -15,7 +15,7 @@ metadata:
   namespace: keel
   labels:
     app: keel
-    chart: keel-0.2.1
+    chart: keel-0.6.1
     release: keel
     heritage: Tiller
 
@@ -29,7 +29,7 @@ metadata:
   namespace: keel
   labels:
     app: keel
-    chart: keel-0.2.1
+    chart: keel-0.6.1
     release: keel
     heritage: Tiller
 spec:
@@ -52,7 +52,7 @@ metadata:
   namespace: keel
   labels:
     app: keel
-    chart: keel-0.2.1
+    chart: keel-0.6.1
     release: keel
     heritage: Tiller
 spec:
@@ -110,6 +110,8 @@ spec:
               value: ""
             - name: HIPCHAT_APPROVALS_PASSWORT
               value: ""
+            - name: NOTIFICATION_LEVEL
+              value: "info"
             # Enable insecure registries
             - name: INSECURE_REGISTRY
               value: "true"
@@ -155,6 +157,20 @@ spec:
               memory: 64Mi
             
 
+---
+# Source: keel/templates/pod-disruption-budget.yaml
+
+apiVersion: policy/v1beta1
+kind: PodDisruptionBudget
+metadata:
+  name: keel
+  namespace: keel
+spec:
+  maxUnavailable: 1
+  selector:
+    matchLabels:
+      app: keel
+
 ---
 # Source: keel/templates/clusterrole.yaml
 

deployment/deployment-norbac.yaml

@@ -15,7 +15,7 @@ metadata:
   namespace: keel
   labels:
     app: keel
-    chart: keel-0.2.1
+    chart: keel-0.6.1
     release: keel
     heritage: Tiller
 
@@ -29,7 +29,7 @@ metadata:
   namespace: keel
   labels:
     app: keel
-    chart: keel-0.2.1
+    chart: keel-0.6.1
     release: keel
     heritage: Tiller
 spec:
@@ -52,7 +52,7 @@ metadata:
   namespace: keel
   labels:
     app: keel
-    chart: keel-0.2.1
+    chart: keel-0.6.1
     release: keel
     heritage: Tiller
 spec:
@@ -110,6 +110,8 @@ spec:
               value: ""
             - name: HIPCHAT_APPROVALS_PASSWORT
               value: ""
+            - name: NOTIFICATION_LEVEL
+              value: "info"
             # Enable insecure registries
             - name: INSECURE_REGISTRY
               value: "true"
@@ -155,6 +157,20 @@ spec:
               memory: 64Mi
             
 
+---
+# Source: keel/templates/pod-disruption-budget.yaml
+
+apiVersion: policy/v1beta1
+kind: PodDisruptionBudget
+metadata:
+  name: keel
+  namespace: keel
+spec:
+  maxUnavailable: 1
+  selector:
+    matchLabels:
+      app: keel
+
 ---
 # Source: keel/templates/clusterrole.yaml
 

deployment/deployment-rbac-whr-sidecar.yaml

@@ -15,7 +15,7 @@ metadata:
   namespace: keel
   labels:
     app: keel
-    chart: keel-0.2.1
+    chart: keel-0.6.1
     release: keel
     heritage: Tiller
 
@@ -97,7 +97,7 @@ metadata:
   namespace: keel
   labels:
     app: keel
-    chart: keel-0.2.1
+    chart: keel-0.6.1
     release: keel
     heritage: Tiller
 spec:
@@ -120,7 +120,7 @@ metadata:
   namespace: keel
   labels:
     app: keel
-    chart: keel-0.2.1
+    chart: keel-0.6.1
     release: keel
     heritage: Tiller
 spec:
@@ -178,6 +178,8 @@ spec:
               value: ""
             - name: HIPCHAT_APPROVALS_PASSWORT
               value: ""
+            - name: NOTIFICATION_LEVEL
+              value: "info"
             # Enable insecure registries
             - name: INSECURE_REGISTRY
               value: "true"
@@ -223,6 +225,20 @@ spec:
               memory: 64Mi
             
 
+---
+# Source: keel/templates/pod-disruption-budget.yaml
+
+apiVersion: policy/v1beta1
+kind: PodDisruptionBudget
+metadata:
+  name: keel
+  namespace: keel
+spec:
+  maxUnavailable: 1
+  selector:
+    matchLabels:
+      app: keel
+
 ---
 # Source: keel/templates/secrets-google.yaml
 

deployment/deployment-rbac.yaml

@@ -15,7 +15,7 @@ metadata:
   namespace: keel
   labels:
     app: keel
-    chart: keel-0.2.1
+    chart: keel-0.6.1
     release: keel
     heritage: Tiller
 
@@ -97,7 +97,7 @@ metadata:
   namespace: keel
   labels:
     app: keel
-    chart: keel-0.2.1
+    chart: keel-0.6.1
     release: keel
     heritage: Tiller
 spec:
@@ -120,7 +120,7 @@ metadata:
   namespace: keel
   labels:
     app: keel
-    chart: keel-0.2.1
+    chart: keel-0.6.1
     release: keel
     heritage: Tiller
 spec:
@@ -178,6 +178,8 @@ spec:
               value: ""
             - name: HIPCHAT_APPROVALS_PASSWORT
               value: ""
+            - name: NOTIFICATION_LEVEL
+              value: "info"
             # Enable insecure registries
             - name: INSECURE_REGISTRY
               value: "true"
@@ -223,6 +225,20 @@ spec:
               memory: 64Mi
             
 
+---
+# Source: keel/templates/pod-disruption-budget.yaml
+
+apiVersion: policy/v1beta1
+kind: PodDisruptionBudget
+metadata:
+  name: keel
+  namespace: keel
+spec:
+  maxUnavailable: 1
+  selector:
+    matchLabels:
+      app: keel
+
 ---
 # Source: keel/templates/secrets-google.yaml