search

temporalio / helm-charts

Temporal Helm charts
MIT License
282 stars 316 forks source link

Fix rendering for elasticsearch visibility storage #451

Closed askoriy closed 1 week ago

askoriy commented 6 months ago

What was changed

To specify Elasticsearch as only visibility storage, the value server.config.persistence.visibility.driver should be set to elasticsearch. This omits configuring Cassandra as visibility storage coming from default values.

Why?

When Cassandra was disabled and external Postgres was used as default storage and external Elasticsearch for visibility storage, helm failed to install because created inconsistent resources and configuration.

Checklist

  1. Closes

  2. How was this tested:

  3. Any docs updates needed?

robholland commented 1 week ago

Fixed by #522