Skip to content

Commit

Permalink
[bitnami/clickhouse] Set Zookeeper 3.8.x as subchart (#19005)
Browse files Browse the repository at this point in the history
* [bitnami/clickhouse] Set Zookeeper 3.8.x as subchart

Signed-off-by: Celia Garcia Marquez <gcelia@vmware.com>

* [bitnami/clickhouse] Increase timeout

Signed-off-by: Celia Garcia Marquez <gcelia@vmware.com>

* [bitnami/clickhouse] Update chart version

Signed-off-by: Celia Garcia Marquez <gcelia@vmware.com>

* Update README.md with readme-generator-for-helm

Signed-off-by: Bitnami Containers <bitnami-bot@vmware.com>

---------

Signed-off-by: Celia Garcia Marquez <gcelia@vmware.com>
Signed-off-by: Bitnami Containers <bitnami-bot@vmware.com>
Co-authored-by: Bitnami Containers <bitnami-bot@vmware.com>
  • Loading branch information
CeliaGMqrz and bitnami-bot authored Sep 6, 2023
1 parent db0f0cd commit bb4f400
Show file tree
Hide file tree
Showing 4 changed files with 23 additions and 8 deletions.
2 changes: 1 addition & 1 deletion .vib/clickhouse/ginkgo/clickhouse_suite_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,7 @@ func init() {
flag.StringVar(&username, "username", "", "database user")
flag.StringVar(&password, "password", "", "database password for username")
flag.IntVar(&shards, "shards", 2, "number of shards")
flag.IntVar(&timeoutSeconds, "timeout", 120, "timeout in seconds")
flag.IntVar(&timeoutSeconds, "timeout", 180, "timeout in seconds")
timeout = time.Duration(timeoutSeconds) * time.Second
}

Expand Down
2 changes: 1 addition & 1 deletion bitnami/clickhouse/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -33,4 +33,4 @@ maintainers:
name: clickhouse
sources:
- https://github.com/bitnami/charts/tree/main/bitnami/clickhouse
version: 3.8.0
version: 3.8.1
14 changes: 9 additions & 5 deletions bitnami/clickhouse/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -342,11 +342,15 @@ The command removes all the Kubernetes components associated with the chart and

### Zookeeper subchart parameters

| Name | Description | Value |
| -------------------------------- | ----------------------------- | ------ |
| `zookeeper.enabled` | Deploy Zookeeper subchart | `true` |
| `zookeeper.replicaCount` | Number of Zookeeper instances | `3` |
| `zookeeper.service.ports.client` | Zookeeper client port | `2181` |
| Name | Description | Value |
| -------------------------------- | ---------------------------------------------------- | --------------------- |
| `zookeeper.enabled` | Deploy Zookeeper subchart | `true` |
| `zookeeper.replicaCount` | Number of Zookeeper instances | `3` |
| `zookeeper.service.ports.client` | Zookeeper client port | `2181` |
| `zookeeper.image.registry` | Zookeeper image registry | `docker.io` |
| `zookeeper.image.repository` | Zookeeper image repository | `bitnami/zookeeper` |
| `zookeeper.image.tag` | Zookeeper image tag (immutable tags are recommended) | `3.8.2-debian-11-r40` |
| `zookeeper.image.pullPolicy` | Zookeeper image pull policy | `IfNotPresent` |

See <https://github.com/bitnami-labs/readme-generator-for-helm> to create the table.

Expand Down
13 changes: 12 additions & 1 deletion bitnami/clickhouse/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -1110,7 +1110,18 @@ externalZookeeper:
##
zookeeper:
enabled: true
## Override zookeeper default image as 3.9 is not supported https://github.com/ClickHouse/ClickHouse/issues/53749
## ref: https://github.com/bitnami/containers/tree/main/bitnami/zookeeper
## @param zookeeper.image.registry Zookeeper image registry
## @param zookeeper.image.repository Zookeeper image repository
## @param zookeeper.image.tag Zookeeper image tag (immutable tags are recommended)
## @param zookeeper.image.pullPolicy Zookeeper image pull policy
image:
registry: docker.io
repository: bitnami/zookeeper
tag: 3.8.2-debian-11-r40
pullPolicy: IfNotPresent
replicaCount: 3
service:
ports:
client: 2181
client: 2181

0 comments on commit bb4f400

Please sign in to comment.