Skip to content

Commit c9357a6

Browse files
author
Ellie Huxtable
authored
Update default Clickhouse version to 22.3.13.80 (#601)
* Update default Clickhouse version to 22.3.13.80 22.3.6.5 has a slow memory leak, eventually causing OOM. This has been fixed in later patch versions. * Bump chart version * More * Format * Update snapshots * Add snapshot update command to README
1 parent e015ae5 commit c9357a6

File tree

8 files changed

+16
-10
lines changed

8 files changed

+16
-10
lines changed

charts/posthog/ALL_VALUES.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -372,7 +372,7 @@ The following table lists the configurable parameters of the PostHog chart and t
372372
| clickhouse.secure | bool | `false` | Whether to use TLS connection connecting to ClickHouse |
373373
| clickhouse.verify | bool | `false` | Whether to verify TLS certificate on connection to ClickHouse |
374374
| clickhouse.image.repository | string | `"clickhouse/clickhouse-server"` | ClickHouse image repository. |
375-
| clickhouse.image.tag | string | `"22.3.6.5"` | ClickHouse image tag. Note: PostHog does not support all versions of ClickHouse. Please override the default only if you know what you are doing. |
375+
| clickhouse.image.tag | string | `"22.3.13.80"` | ClickHouse image tag. Note: PostHog does not support all versions of ClickHouse. Please override the default only if you know what you are doing. |
376376
| clickhouse.image.pullPolicy | string | `"IfNotPresent"` | Image pull policy |
377377
| clickhouse.tolerations | list | `[]` | Toleration labels for clickhouse pod assignment |
378378
| clickhouse.affinity | object | `{}` | Affinity settings for clickhouse pod |
@@ -398,7 +398,7 @@ The following table lists the configurable parameters of the PostHog chart and t
398398
| clickhouse.podAnnotations | string | `nil` | |
399399
| clickhouse.podDistribution | string | `nil` | |
400400
| clickhouse.client.image.repository | string | `"clickhouse/clickhouse-server"` | ClickHouse image repository. |
401-
| clickhouse.client.image.tag | string | `"22.3.6.5"` | ClickHouse image tag. Note: PostHog does not support all versions of ClickHouse. Please override the default only if you know what you are doing. |
401+
| clickhouse.client.image.tag | string | `"22.3.13.80"` | ClickHouse image tag. Note: PostHog does not support all versions of ClickHouse. Please override the default only if you know what you are doing. |
402402
| clickhouse.client.image.pullPolicy | string | `"IfNotPresent"` | Image pull policy |
403403
| clickhouse.backup.enabled | bool | `false` | |
404404
| clickhouse.backup.image.repository | string | `"altinity/clickhouse-backup"` | Clickhouse backup image repository. |

charts/posthog/Chart.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ type: application
1111

1212
# This is the chart version. This version number should be incremented each time you make changes
1313
# to the chart and its templates, including the app version.
14-
version: 29.0.1
14+
version: 29.0.2
1515

1616
# This is the version number of the application being deployed. This version number should be
1717
# incremented each time you make changes to the application.

charts/posthog/README.md

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -49,6 +49,12 @@ For more information about how it works and how to write test cases, please look
4949

5050
To run the test suite you can execute: `helm unittest --helm3 --strict --file 'tests/*.yaml' --file 'tests/clickhouse-operator/*.yaml' charts/posthog`
5151

52+
If you need to update the snapshots, execute:
53+
54+
```
55+
helm unittest --helm3 --strict --file 'tests/*.yaml' --file 'tests/**/*.yaml' charts/posthog -u
56+
```
57+
5258
#### Integration tests
5359
- [kubetest](https://github.com/PostHog/charts-clickhouse/tree/main/ci/kubetest): to verify if applying the rendered Helm templates against a Kubernetes target cluster gives us the stack we expect (example: are the disks encrypted? Can this pod communicate with this service?)
5460
- [k6](https://github.com/PostHog/charts-clickhouse/tree/main/ci/k6): HTTP test used to verify the reliability, performance and compliance of the PostHog installation (example: is the PostHog ingestion working correctly?)

charts/posthog/tests/__snapshot__/clickhouse-backup-cronjob.yaml.snap

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,7 @@ should match the snapshot when backup is true:
3030
value: backup
3131
- name: BACKUP_PASSWORD
3232
value: backup_password
33-
image: clickhouse/clickhouse-server:22.3.6.5
33+
image: clickhouse/clickhouse-server:22.3.13.80
3434
imagePullPolicy: IfNotPresent
3535
name: run-backup-cron
3636
volumeMounts:

charts/posthog/tests/__snapshot__/clickhouse-instance.yaml.snap

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -75,7 +75,7 @@ the manifest should match the snapshot when using default values:
7575
- /bin/bash
7676
- -c
7777
- /usr/bin/clickhouse-server --config-file=/etc/clickhouse-server/config.xml
78-
image: clickhouse/clickhouse-server:22.3.6.5
78+
image: clickhouse/clickhouse-server:22.3.13.80
7979
name: clickhouse
8080
ports:
8181
- containerPort: 8123

charts/posthog/tests/clickhouse-instance.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -344,7 +344,7 @@ tests:
344344
count: 1
345345
- equal:
346346
path: spec.templates.podTemplates[0].spec.containers[0].image
347-
value: "clickhouse/clickhouse-server:22.3.6.5"
347+
value: "clickhouse/clickhouse-server:22.3.13.80"
348348

349349
- it: allows modifying clickhouse-server version
350350
set:

charts/posthog/values.yaml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1193,7 +1193,7 @@ clickhouse:
11931193
# -- ClickHouse image repository.
11941194
repository: clickhouse/clickhouse-server
11951195
# -- ClickHouse image tag. Note: PostHog does not support all versions of ClickHouse. Please override the default only if you know what you are doing.
1196-
tag: "22.3.6.5"
1196+
tag: "22.3.13.80"
11971197
# -- Image pull policy
11981198
pullPolicy: IfNotPresent
11991199

@@ -1305,7 +1305,7 @@ clickhouse:
13051305
# -- ClickHouse image repository.
13061306
repository: clickhouse/clickhouse-server
13071307
# -- ClickHouse image tag. Note: PostHog does not support all versions of ClickHouse. Please override the default only if you know what you are doing.
1308-
tag: "22.3.6.5"
1308+
tag: "22.3.13.80"
13091309
# -- Image pull policy
13101310
pullPolicy: IfNotPresent
13111311

ci/kubetest/test_clickhouse_different_image.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@
1414
1515
clickhouse:
1616
image:
17-
tag: 22.3.6.5-alpine
17+
tag: 22.3.13.80-alpine
1818
"""
1919

2020

@@ -24,4 +24,4 @@ def test_clickhouse_pod_image(kube):
2424

2525
is_posthog_healthy(kube)
2626
pod_spec = get_clickhouse_pod_spec(kube)
27-
assert pod_spec.containers[0].image == "clickhouse/clickhouse-server:22.3.6.5-alpine"
27+
assert pod_spec.containers[0].image == "clickhouse/clickhouse-server:22.3.13.80-alpine"

0 commit comments

Comments
 (0)