Skip to content

Commit

Permalink
fix: use zookeper instead of clickhouse keeper as keeper is currently…
Browse files Browse the repository at this point in the history
… broken in the bitnami chart (#20)

* fix: use zookeper instead of clickhouse keeper as keeper is currently broken in the bitnami chart

* docs: document switch to zookeeper
  • Loading branch information
echozio authored May 3, 2024
1 parent 979681e commit c3e7f14
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 2 deletions.
5 changes: 5 additions & 0 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -177,3 +177,8 @@ separate parameters for HTTP and TCP, as different components require different
were using these parameters, simply set `config.clickhouseHttpHost` and `config.clickhouseTcpHost`
(or the equivalent `...From` variants) making sure to set the correct port. If you were setting this
on a per-component basis or letting the chart configure it for you no action is needed.

ClickHouse is now also set up to use Zookeeper instead of ClickHouse Keeper as it is currently
broken in Bitnami's Helm chart for ClickHouse: https://github.com/bitnami/charts/issues/15935. If
you had a working configuration using ClickHouse Keeper, you will need to explicitly enable it and
disable Zookeeper to avoid switching over to a fresh Zookeeper deployment.
4 changes: 2 additions & 2 deletions values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -861,6 +861,6 @@ clickhouse:
username: "jitsu"
password: "jitsu"
keeper:
enabled: true
zookeeper:
enabled: false
zookeeper:
enabled: true

0 comments on commit c3e7f14

Please sign in to comment.