temporalio / helm-charts

Temporal Helm charts
MIT License
305 stars 337 forks source link

fix: Use visibility server when defining visibility config #436

Closed bakjos closed 3 months ago

bakjos commented 11 months ago

What was changed

Why?

When the default store uses cassandra and the visibility store sql the config was not including the sql part

Checklist

  1. Closes

  2. How was this tested:

    Using the following values

server:
  replicaCount: 1
  config:
    persistence:
      default:
        cassandra:
          hosts:
            - cassandra
          port: 9042
      visibility:
        driver: "sql"
        sql:
          driver: "postgres"
          host: "postgres"
          port: 5432
          database: "temporal_visibility"
          user: "postgres"
          password: "postgres"
          maxConns: 20
          maxConnLifetime: "1h"
  1. Any docs updates needed?
CLAassistant commented 11 months ago

CLA assistant check
All committers have signed the CLA.

robholland commented 3 months ago

Thanks :)