Update common 3.0.1 (#112)

* 2.5.0

* 3.0.1

* Remove upstream ingress unit test

ingress is unused by us and the unit test proves to be unstable

library/common-test/Chart.yaml

@@ -1,7 +1,8 @@
 apiVersion: v2
 name: common-test
 description: Helper chart to test different use cases of the common library
-version: 1.2.0
+version: 1.2.1
+kubeVersion: ">=1.16.0-0"
 keywords:
   - truecharts
   - common

library/common-test/ci/basic-values.yaml

@@ -19,26 +19,6 @@ probes:
     enabled: true
 
 
-appIngressEnabled: true
-appIngress:
-  main:
-    enabled: true
-  mainhttp:
-    enabled: true
-    entrypoint: "web"
-    certType: ""
-    existingcert: ""
-    authForwardURL: ""
-    annotations: {}
-    labels: {}
-    hosts:
-      - host: app.truecharts.placeholder
-        paths:
-          - path: /
-            # Ignored if not kubeVersion >= 1.14-0
-            pathType: Prefix
-    
-appVolumesEnabled: true
 appVolumeMounts:
   data:
     enabled: true

library/common-test/ci/codeserver-values.yaml

@@ -11,7 +11,7 @@ ingress:
   enabled: true
 
 persistence:
-    config:
+  config:
     enabled: true
     emptyDir: true
     mountPath: /www/assets
@@ -25,27 +25,8 @@ addons:
     volumeMounts:
     - name: config
       mountPath: /www/assets
-
-appIngressEnabled: true
-appIngress:
-  main:
-    enabled: true
-  mainhttp:
-    enabled: true
-    entrypoint: "web"
-    certType: ""
-    existingcert: ""
-    authForwardURL: ""
-    annotations: {}
-    labels: {}
-    hosts:
-      - host: app.truecharts.placeholder
-        paths:
-          - path: /
-            # Ignored if not kubeVersion >= 1.14-0
-            pathType: Prefix
     
-appVolumesEnabled: true
+
 appVolumeMounts:
   data:
     enabled: true

library/common-test/values.yaml

@@ -10,26 +10,6 @@ service:
 ingress:
   enabled: true
 
-appIngressEnabled: true
-appIngress:
-  main:
-    enabled: true
-  mainhttp:
-    enabled: true
-    entrypoint: "web"
-    certType: ""
-    existingcert: ""
-    authForwardURL: ""
-    annotations: {}
-    labels: {}
-    hosts:
-      - host: app.truecharts.placeholder
-        paths:
-          - path: /
-            # Ignored if not kubeVersion >= 1.14-0
-            pathType: Prefix
-    
-appVolumesEnabled: true
 appVolumeMounts:
   data:
     enabled: true

library/common/CHANGELOG.md

@@ -4,6 +4,38 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [3.0.1]
+
+### Fixed
+
+- Fixed `kubeVersion` in `Chart.yaml` not working with people running k8s pre-releases
+- Added `ingressClassName` to `values.yaml`
+
+## [3.0.0]
+
+### Changed
+
+- `probes.startupProbe` is now enabled by default. This will allow for quicker `Ready` states on Pods. It will precede the liveness and readiness Probes. *Note:* This requires that you are running [Kubernetes 1.16](https://kubernetes.io/docs/reference/command-line-tools-reference/feature-gates/)
+- Aligned our default values for liveness and readiness Probes with the Kubernetes defaults:
+  - `initialDelaySeconds` is now `0`
+  - `timeoutSeconds` is now `1`
+  - `failureThreshold` is now `3`
+
+## [2.5.1]
+
+### Added
+
+- Fixed a small typo in `values.yaml`: `ingress.tls.hostTpl` -> `ingress.tls.hostsTpl`
+
+## [2.5.0]
+
+### Added
+
+- Allow setting ingress values through Helm templates:
+  - `ingress.hosts.hostTpl`
+  - `ingress.tls.secretNameTpl`
+  - `ingress.tls.hostsTpl`
+
 ## [2.4.0]
 
 ### Added
@@ -84,6 +116,14 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 This is the last version before starting this changelog. All sorts of cool stuff was changed, but only `git log` remembers what that was :slightly_frowning_face:
 
+[3.0.1]: https://github.com/k8s-at-home/charts/tree/common-3.0.1/charts/common
+
+[3.0.0]: https://github.com/k8s-at-home/charts/tree/common-3.0.0/charts/common
+
+[2.5.1]: https://github.com/k8s-at-home/charts/tree/common-2.5.1/charts/common
+
+[2.5.0]: https://github.com/k8s-at-home/charts/tree/common-2.5.0/charts/common
+
 [2.4.0]: https://github.com/k8s-at-home/charts/tree/common-2.4.0/charts/common
 
 [2.3.0]: https://github.com/k8s-at-home/charts/tree/common-2.3.0/charts/common

library/common/Chart.yaml

@@ -2,8 +2,9 @@ apiVersion: v2
 name: common
 description: Function library for TrueCharts
 type: library
-version: 0.12.2
+version: 1.0.0
-# upstream_version: 2.4.0
+kubeVersion: ">=1.16.0-0"
+# upstream_version: 3.0.1
 keywords:
   - truecharts
   - library-chart

library/common/templates/_notes.tpl

@@ -6,7 +6,7 @@ Default NOTES.txt content.
 1. Get the application URL by running these commands:
 {{- if .Values.ingress.enabled }}
 {{- range .Values.ingress.hosts }}
-  http{{ if $.Values.ingress.tls }}s{{ end }}://{{ .host }}{{ (first .paths).path }}
+  http{{ if $.Values.ingress.tls }}s{{ end }}://{{- if .hostTpl }}{{ tpl .hostTpl $ }}{{ else }}{{ .host }}{{ end }}{{ (first .paths).path }}
 {{- end }}
 {{- else if contains "NodePort" .Values.service.type }}
   export NODE_PORT=$(kubectl get --namespace {{ .Release.Namespace }} -o jsonpath="{.spec.ports[0].nodePort}" services {{ include "common.names.fullname" . }})

library/common/templates/_service.tpl

@@ -6,6 +6,7 @@ of the main Service and any additionalServices.
   {{- if .Values.service.enabled -}}
     {{- /* Generate primary service */ -}}
     {{- include "common.classes.service" . }}
+
     {{- /* Generate additional services as required */ -}}
     {{- range $index, $extraService := .Values.service.additionalServices }}
       {{- if $extraService.enabled -}}

library/common/templates/classes/_ingress.tpl

@@ -38,12 +38,23 @@ spec:
         {{- range .hosts }}
         - {{ . | quote }}
         {{- end }}
+        {{- range .hostsTpl }}
+        - {{ tpl . $ | quote }}
+        {{- end }}
+      {{- if .secretNameTpl }}
+      secretName: {{ tpl .secretNameTpl $ | quote}}
+      {{- else }}
       secretName: {{ .secretName }}
+      {{- end }}
     {{- end }}
   {{- end }}
   rules:
   {{- range $values.hosts }}
+  {{- if .hostTpl }}
+    - host: {{ tpl .hostTpl $ | quote }} 
+  {{- else }}
     - host: {{ .host | quote }}
+  {{- end }}
       http:
         paths:
           {{- range .paths }}

library/common/values.yaml

@@ -24,7 +24,7 @@ fullnameOverride: ""
 podAnnotations: {}
 
 serviceAccount:
-    # Specifies whether a service account should be created
+  # Specifies whether a service account should be created
   create: false
   # Annotations to add to the service account
   annotations: {}
@@ -33,7 +33,7 @@ serviceAccount:
   name: ""
 
 env: {}
-# TZ: UTC
+  # TZ: UTC
 
 ## Variables with values set from templates, example
 ## With a release name of: demo, the example env value will be: demo-admin
@@ -98,10 +98,10 @@ probes:
     ## The spec field contains the values for the default livenessProbe.
     ## If you selected custom: true, this field holds the definition of the livenessProbe.
     spec:
-      initialDelaySeconds: 30
+      initialDelaySeconds: 0
-      failureThreshold: 5
       periodSeconds: 10
       timeoutSeconds: 10
+      failureThreshold: 5
 
   readiness:
     enabled: true
@@ -110,22 +110,23 @@ probes:
     ## The spec field contains the values for the default readinessProbe.
     ## If you selected custom: true, this field holds the definition of the readinessProbe.
     spec:
-      initialDelaySeconds: 30
+      initialDelaySeconds: 0
-      failureThreshold: 5
       periodSeconds: 10
       timeoutSeconds: 10
+      failureThreshold: 5
 
   startup:
-    enabled: false
+    enabled: true
     ## Set this to true if you wish to specify your own startupProbe
     custom: false
     ## The spec field contains the values for the default startupProbe.
     ## If you selected custom: true, this field holds the definition of the startupProbe.
     spec:
-      initialDelaySeconds: 5
+      initialDelaySeconds: 0
-      failureThreshold: 30
-      periodSeconds: 10
       timeoutSeconds: 10
+      ## This means it has a maximum of 5*30=150 seconds to start up before it fails
+      periodSeconds: 5
+      failureThreshold: 120
 
 service:
   enabled: true
@@ -178,14 +179,21 @@ ingress:
   labels: {}
   hosts:
     - host: chart-example.local
+      ## Or a tpl that is evaluated
+      # hostTpl: '{{ include "common.names.fullname" . }}.{{ .Release.Namespace }}.{{ .Values.ingress.domainname }}'
       paths:
         - path: /
           # Ignored if not kubeVersion >= 1.14-0
           pathType: Prefix
   tls: []
   #  - secretName: chart-example-tls
+  ## Or if you need a dynamic secretname
+  #  - secretNameTpl: '{{ include "common.names.fullname" . }}-ingress'
   #    hosts:
   #      - chart-example.local
+  ## Or a tpl that is evaluated
+  #    hostsTpl:
+  #      - '{{ include "common.names.fullname" . }}.{{ .Release.Namespace }}.{{ .Values.ingress.domainname }}'
   additionalIngresses: []
   # - enabled: false
   #   nameSuffix: "api"
@@ -469,7 +477,6 @@ appAdditionalServicesEnabled: false
 #         port: 3001
 #         targetPort: 3001
 
-appVolumesEnabled: false
 # appVolumeMounts:
 #   config:
 #     enabled: false
@@ -488,42 +495,39 @@ appVolumesEnabled: false
 #     hostPathEnabled: false
 #     hostPath: ""
 
-emptyDirVolumes: true
+# appIngress:
-
+#  main:
-appIngressEnabled: false
+#    enabled: false
-appIngress:
+#    entrypoint: "websecure"
-  main:
+#    certType: "letsencrypt-prod"
-    enabled: false
+#    existingcert: ""
-    entrypoint: "websecure"
+#    authForwardURL: ""
-    certType: "letsencrypt-prod"
+#    annotations: {}
-    existingcert: ""
+#    labels: {}
-    authForwardURL: ""
+#    hosts:
-    annotations: {}
+#      - host: app.truecharts.placeholder
-    labels: {}
+#        paths:
-    hosts:
+#          - path: /
-      - host: app.truecharts.placeholder
+#            # Ignored if not kubeVersion >= 1.14-0
-        paths:
+#            pathType: Prefix
-          - path: /
+#
-            # Ignored if not kubeVersion >= 1.14-0
+#  maintcp:
-            pathType: Prefix
+#    enabled: false
-
+#    type: "TCP"
-  maintcp:
+#    entrypoint: "kms"
-    enabled: false
+#    certType: "none"
-    type: "TCP"
+#    existingcert: ""
-    entrypoint: "kms"
+#    annotations: {}
-    certType: "none"
+#    labels: {}
-    existingcert: ""
+#    hosts:
-    annotations: {}
+#      - host: app.truecharts.placeholder
-    labels: {}
+#        paths:
-    hosts:
+#          - path: /
-      - host: app.truecharts.placeholder
+#            # Ignored if not kubeVersion >= 1.14-0
-        paths:
+#            pathType: Prefix
-          - path: /
+#  mainudp:
-            # Ignored if not kubeVersion >= 1.14-0
+#    enabled: false
-            pathType: Prefix
+#    type: "UDP"
-  mainudp:
+#    entrypoint: "DNSUDP"
-    enabled: false
+#    annotations: {}
-    type: "UDP"
+#    labels: {}
-    entrypoint: "DNSUDP"
-    annotations: {}
-    labels: {}