search

litmuschaos / litmus-helm

Helm Charts for the Litmus Chaos Operator & CRDs
Apache License 2.0
46 stars 88 forks source link

[litmus] mongodb rootPassword is converted to an integer #339

Closed wrdls closed 7 months ago

wrdls commented 1 year ago

The default rootPassword gets automatically converted to an int by YAML parser libraries.

This makes the env of the post-upgrade-hook Job look like this:

          env:
            - name: DB_PASSWORD
              value: 1234
            - name: DB_USER
              value: root

Which is not a valid Job because env values need to be strings.

Change the default to something that will always be a string (e.g. abc123) or make sure to apply proper quoting with the | quote function.

Haven't checked the whole chart but it's at least an issue here: https://github.com/litmuschaos/litmus-helm/blob/master/charts/litmus/templates/post-upgrade-hook.yaml#L43

Jonsy13 commented 7 months ago

The upgrade job has been removed in 3.x for now. So this issue shouldn't be happening. Closing this issue, feel free to open if the issue still persists. Thanks!