From ba8c35da9fa3d6cc40f24af641c77c1431be0599 Mon Sep 17 00:00:00 2001 From: Michel Bathe Date: Fri, 18 Aug 2023 14:44:24 +0200 Subject: [PATCH 1/4] feat: add probes to helm chart --- .../templates/deployment.yaml | 11 +++++++++++ charts/bitwarden-crd-operator/values.yaml | 14 ++++++++++++++ 2 files changed, 25 insertions(+) diff --git a/charts/bitwarden-crd-operator/templates/deployment.yaml b/charts/bitwarden-crd-operator/templates/deployment.yaml index 931f3fa..0827229 100644 --- a/charts/bitwarden-crd-operator/templates/deployment.yaml +++ b/charts/bitwarden-crd-operator/templates/deployment.yaml @@ -50,10 +50,21 @@ spec: httpGet: path: /healthz port: http + failureThreshold: {{ .Values.livenessProbe.failureThreshold }} + initialDelaySeconds: {{ .Values.livenessProbe.initialDelaySeconds }} + periodSeconds: {{ .Values.livenessProbe.periodSeconds }} + successThreshold: {{ .Values.livenessProbe.successThreshold }} + timeoutSeconds: {{ .Values.livenessProbe.timeoutSeconds }} readinessProbe: httpGet: path: /healthz port: http + scheme: HTTP + failureThreshold: {{ .Values.readinessProbe.failureThreshold }} + initialDelaySeconds: {{ .Values.readinessProbe.initialDelaySeconds }} + periodSeconds: {{ .Values.readinessProbe.periodSeconds }} + successThreshold: {{ .Values.readinessProbe.successThreshold }} + timeoutSeconds: {{ .Values.readinessProbe.timeoutSeconds }} resources: {{- toYaml .Values.resources | nindent 12 }} {{- with .Values.nodeSelector }} diff --git a/charts/bitwarden-crd-operator/values.yaml b/charts/bitwarden-crd-operator/values.yaml index 5a3aa6e..a293c41 100644 --- a/charts/bitwarden-crd-operator/values.yaml +++ b/charts/bitwarden-crd-operator/values.yaml @@ -51,6 +51,20 @@ securityContext: {} # runAsNonRoot: true # runAsUser: 1000 +readinessProbe: + failureThreshold: 3 + initialDelaySeconds: 10 + periodSeconds: 10 + successThreshold: 1 + timeoutSeconds: 1 + +livenessProbe: + failureThreshold: 3 + initialDelaySeconds: 10 + periodSeconds: 10 + successThreshold: 1 + timeoutSeconds: 1 + resources: {} # We usually recommend not to specify default resources and to leave this as a conscious # choice for the user. This also increases chances charts run on environments with little From d629fa600f55192e7f60c198a8ad308e85bcde56 Mon Sep 17 00:00:00 2001 From: Michel Bathe Date: Fri, 18 Aug 2023 14:47:58 +0200 Subject: [PATCH 2/4] feat: add probes to helm chart --- charts/bitwarden-crd-operator/templates/deployment.yaml | 1 - 1 file changed, 1 deletion(-) diff --git a/charts/bitwarden-crd-operator/templates/deployment.yaml b/charts/bitwarden-crd-operator/templates/deployment.yaml index 0827229..707aec6 100644 --- a/charts/bitwarden-crd-operator/templates/deployment.yaml +++ b/charts/bitwarden-crd-operator/templates/deployment.yaml @@ -59,7 +59,6 @@ spec: httpGet: path: /healthz port: http - scheme: HTTP failureThreshold: {{ .Values.readinessProbe.failureThreshold }} initialDelaySeconds: {{ .Values.readinessProbe.initialDelaySeconds }} periodSeconds: {{ .Values.readinessProbe.periodSeconds }} From bec7476acec6dff3cefa7495d54f837e9438e4a7 Mon Sep 17 00:00:00 2001 From: Michel Bathe Date: Mon, 21 Aug 2023 11:36:52 +0200 Subject: [PATCH 3/4] fix: updatet chart version and added change log --- charts/bitwarden-crd-operator/Chart.yaml | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/charts/bitwarden-crd-operator/Chart.yaml b/charts/bitwarden-crd-operator/Chart.yaml index 9fee9d9..3d95ff0 100644 --- a/charts/bitwarden-crd-operator/Chart.yaml +++ b/charts/bitwarden-crd-operator/Chart.yaml @@ -4,7 +4,7 @@ description: Deploy the Bitwarden CRD Operator type: application -version: "v0.7.4" +version: "v0.7.5" appVersion: "0.6.4" @@ -95,8 +95,8 @@ annotations: artifacthub.io/operator: "true" artifacthub.io/containsSecurityUpdates: "false" artifacthub.io/changes: | - - kind: fixed - description: "Fixed bitwarden installation" + - kind: feat + description: "Added prob variables for helm" artifacthub.io/images: | - name: bitwarden-crd-operator image: ghcr.io/lerentis/bitwarden-crd-operator:0.6.4 From 62a2b488d2efdeda0e6a2c50729ddd9f013488f3 Mon Sep 17 00:00:00 2001 From: Tobias Trabelsi Date: Mon, 21 Aug 2023 11:43:39 +0200 Subject: [PATCH 4/4] Update charts/bitwarden-crd-operator/Chart.yaml --- charts/bitwarden-crd-operator/Chart.yaml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/charts/bitwarden-crd-operator/Chart.yaml b/charts/bitwarden-crd-operator/Chart.yaml index 3d95ff0..119daa2 100644 --- a/charts/bitwarden-crd-operator/Chart.yaml +++ b/charts/bitwarden-crd-operator/Chart.yaml @@ -95,7 +95,7 @@ annotations: artifacthub.io/operator: "true" artifacthub.io/containsSecurityUpdates: "false" artifacthub.io/changes: | - - kind: feat + - kind: added description: "Added prob variables for helm" artifacthub.io/images: | - name: bitwarden-crd-operator