helm-charts icon indicating copy to clipboard operation
helm-charts copied to clipboard

fix: Use visibility server when defining visibility config

Open bakjos opened this issue 2 years ago • 1 comments

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?

bakjos avatar Oct 24 '23 23:10 bakjos

CLA assistant check
All committers have signed the CLA.

CLAassistant avatar Oct 24 '23 23:10 CLAassistant

Thanks :)

robholland avatar Jun 08 '24 15:06 robholland