From 31a0a30a370c64de6227bfb0ff10035c4b2dcd8c Mon Sep 17 00:00:00 2001 From: Calvinaud Date: Thu, 11 Jan 2024 15:30:17 +0100 Subject: [PATCH] Remove hardcoded version of chaoscenter in pre-hook job in litmus-agent chart #329 (#330) * Remove hardcoded version in litmus-agent charts - Use a variable for the version value in the hook-pre-install-job templates - Add variable to the values.yaml with a default value of 3.0.0-beta8 - Bump chart to version 0.2.1 Signed-off-by: Calvin Audier * Regenerate the documentation for litmus-agent with helm-docs Signed-off-by: Calvin Audier * Resolve lint issues found in pipeline Signed-off-by: Calvin Audier --------- Signed-off-by: Calvin Audier Signed-off-by: Calvinaud Co-authored-by: Calvin Audier Co-authored-by: Maria Kotliarevskaia --- charts/litmus-agent/README.md | 1 + charts/litmus-agent/templates/hook-pre-install-job.yaml | 2 +- charts/litmus-agent/values.yaml | 2 ++ 3 files changed, 4 insertions(+), 1 deletion(-) diff --git a/charts/litmus-agent/README.md b/charts/litmus-agent/README.md index 41dac0f9..9c1f0422 100644 --- a/charts/litmus-agent/README.md +++ b/charts/litmus-agent/README.md @@ -55,6 +55,7 @@ $ helm install litmus-agent litmuschaos/litmus-agent \ | AGENT_DESCRIPTION | string | `"chaos agent deployed with helm"` | | | AGENT_NAME | string | `"helm-agent"` | | | AGENT_NODE_SELECTOR | string | `""` | | +| APP_VERSION | string | `"3.0.0-beta8"` | | | CLUSTER_TYPE | string | `"external"` | | | LITMUS_BACKEND_URL | string | `""` | | | LITMUS_PASSWORD | string | `"litmus"` | | diff --git a/charts/litmus-agent/templates/hook-pre-install-job.yaml b/charts/litmus-agent/templates/hook-pre-install-job.yaml index 8702a397..4b83cd51 100644 --- a/charts/litmus-agent/templates/hook-pre-install-job.yaml +++ b/charts/litmus-agent/templates/hook-pre-install-job.yaml @@ -64,7 +64,7 @@ spec: value: {{ .Release.Namespace }} - name: APP_VERSION - value: "3.0.0-beta8" #For compatibility with 3.0.0-beta8 ChaosCenter, Version has to be same + value: {{ .Values.APP_VERSION }} - name: SERVICE_ACCOUNT_NAME value: {{ include "litmus-agent.serviceAccountName" . }} diff --git a/charts/litmus-agent/values.yaml b/charts/litmus-agent/values.yaml index 0b277122..fd4af9fc 100644 --- a/charts/litmus-agent/values.yaml +++ b/charts/litmus-agent/values.yaml @@ -16,6 +16,8 @@ AGENT_NODE_SELECTOR: "" SA_EXISTS: true NS_EXISTS: true CLUSTER_TYPE: "external" +# For compatibility with 3.0.0-beta8 ChaosCenter, Version has to be same +APP_VERSION: "3.0.0-beta8" SKIP_SSL: "false" # PLATFORM_NAME: AWS, GKE, Openshift, Rancher, Others