Skip to content

Commit

Permalink
Remove hardcoded version of chaoscenter in pre-hook job in litmus-age…
Browse files Browse the repository at this point in the history
…nt 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 <[email protected]>

* Regenerate the documentation for litmus-agent with helm-docs

Signed-off-by: Calvin Audier <[email protected]>

* Resolve lint issues found in pipeline

Signed-off-by: Calvin Audier <[email protected]>

---------

Signed-off-by: Calvin Audier <[email protected]>
Signed-off-by: Calvinaud <[email protected]>
Co-authored-by: Calvin Audier <[email protected]>
Co-authored-by: Maria Kotliarevskaia <[email protected]>
  • Loading branch information
3 people authored Jan 11, 2024
1 parent 603f4c9 commit 31a0a30
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 1 deletion.
1 change: 1 addition & 0 deletions charts/litmus-agent/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -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"` | |
Expand Down
2 changes: 1 addition & 1 deletion charts/litmus-agent/templates/hook-pre-install-job.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -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" . }}
Expand Down
2 changes: 2 additions & 0 deletions charts/litmus-agent/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down

0 comments on commit 31a0a30

Please sign in to comment.