From 20d47208df2e5f4730d0bb99ea1b26d29170aac4 Mon Sep 17 00:00:00 2001 From: Thomas Miller Date: Thu, 1 Aug 2019 00:16:24 +1000 Subject: [PATCH] [stable/opa] Checksum logic fix (#15985) * Updated deployment to restart on new certs - OPA does not use new certs when helm upgrade changes them - Updated chart version Signed-off-by: Thomas Miller * Bumped chart version to 1.6.0 Signed-off-by: Thomas Miller * Fixed bug in checksum annotation - Checksum annotation should be using 'or' and not 'and' - Updated chart version to 1.6.1 Signed-off-by: Thomas Miller --- stable/opa/Chart.yaml | 2 +- stable/opa/templates/deployment.yaml | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/stable/opa/Chart.yaml b/stable/opa/Chart.yaml index ac34dcfcbc..5e7b90744a 100644 --- a/stable/opa/Chart.yaml +++ b/stable/opa/Chart.yaml @@ -6,7 +6,7 @@ keywords: - opa - admission control - policy -version: 1.6.0 +version: 1.6.1 home: https://www.openpolicyagent.org icon: https://raw.githubusercontent.com/open-policy-agent/opa/master/logo/logo.png sources: diff --git a/stable/opa/templates/deployment.yaml b/stable/opa/templates/deployment.yaml index 3ed0ce145b..a68f5acbfd 100644 --- a/stable/opa/templates/deployment.yaml +++ b/stable/opa/templates/deployment.yaml @@ -11,7 +11,7 @@ spec: app: {{ template "opa.fullname" . }} template: metadata: -{{- if and .Values.generateAdmissionControllerCerts .Values.opa }} +{{- if or .Values.generateAdmissionControllerCerts .Values.opa }} annotations: {{- if .Values.generateAdmissionControllerCerts }} checksum/certs: {{ include (print $.Template.BasePath "/webhookconfiguration.yaml") . | sha256sum }}