From 3a39ca8f4e03df68de02bab9f4f80a80b3f6c259 Mon Sep 17 00:00:00 2001 From: rjshrjndrn Date: Mon, 18 Nov 2024 12:31:46 +0100 Subject: [PATCH] fix(helm): version change check Signed-off-by: rjshrjndrn --- scripts/helmcharts/openreplay/templates/job.yaml | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/scripts/helmcharts/openreplay/templates/job.yaml b/scripts/helmcharts/openreplay/templates/job.yaml index 4a9722b1e..d3882d9b1 100644 --- a/scripts/helmcharts/openreplay/templates/job.yaml +++ b/scripts/helmcharts/openreplay/templates/job.yaml @@ -2,8 +2,14 @@ Don't have to trigger migration if there is no version change Don't have to trigger migration if skipMigration is set Have to trigger migration if forceMigration is set + +versionChange is true when: + Release.IsUpgrade is false. + Or .Values.deployment.argo is set. + Or Release.IsUpgrade is true and .Values.fromVersion is not equal to .Chart.AppVersion. */}} -{{- $versionChange := (or .Values.deployment.argo (and (not (eq .Values.fromVersion .Chart.AppVersion)) .Release.IsUpgrade)) }} + +{{- $versionChange := (or (not .Release.IsUpgrade) .Values.deployment.argo (and .Release.IsUpgrade (not (eq .Values.fromVersion .Chart.AppVersion)))) }} {{- if or .Values.forceMigration (and (not .Values.skipMigration) $versionChange) }} --- apiVersion: v1