Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Bug] Setting existingSecret not respected by schema setup/update jobs #261

Closed
perplexa opened this issue Jan 26, 2022 · 0 comments · Fixed by #262
Closed

[Bug] Setting existingSecret not respected by schema setup/update jobs #261

perplexa opened this issue Jan 26, 2022 · 0 comments · Fixed by #262
Labels
bug Something isn't working

Comments

@perplexa
Copy link
Contributor

What are you really trying to do?

I am trying to set up temporal using the helm chart on an existing cassandra cluster using the following settings:

helm template temporal . \
  --values values/values.cassandra.yaml \
  --set server.config.persistence.default.cassandra.existingSecret=someExistingSecret \
  --set server.config.persistence.visibility.cassandra.password=aPasswordInstead
  --set schema.setup.enabled=true \
  --set schema.update.enabled=true 

Describe the bug

The rendered chart template should include the blocks as follows:

default

            - name: CASSANDRA_PASSWORD
              valueFrom:
                secretKeyRef:
                  name: someExistingSecret
                  key: password

but it has:

            - name: CASSANDRA_PASSWORD
              value: password

visibility

            - name: CASSANDRA_PASSWORD
              value: aPasswordInstead

This works as expected.

Minimal Reproduction

See helm template command at beginning of issue

Environment/Versions

  • Latest helm chart master branch commit 683c53a
  • Helm v3.7.2
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant