temporalio / helm-charts

Temporal Helm charts
MIT License
305 stars 335 forks source link

Use a shared config map for all services. #514

Closed robholland closed 3 months ago

robholland commented 3 months ago

What was changed

Use a shared config map for all services, rather than one per service. Remove helpers in favour of defaults via values file. Remove unused Elasticsearch environment variables. Stop switching between es-visibility and visibility for store names. Various templating improvements.

Why?

Simplifies the deployment and removes some helper code. Makes it clearer that if Elasticsearch is enabled we use that rather than SQL.

Checklist

  1. Closes

  2. How was this tested: Manual review of templates and test installations.

  3. Any docs updates needed?