Skip to content

Commit 6f29e73

Browse files
wrn14897claude
andauthored
fix: combine ClickHouse app user grants into single query (HDX-3832) (#198)
## Summary - The ClickHouse operator only applies the first `grants` entry when multiple are listed separately - The `app` user was missing `SELECT ON system.*` and `SELECT ON default.*` permissions — only `GRANT SHOW ON *.*` was applied - Combined all grants into a single query, matching the pattern that works for the `otelcollector` user ## Test plan - [x] `helm unittest charts/clickstack` — all 146 tests pass - [x] Deploy and verify with `SHOW GRANTS FOR app` in ClickHouse — should show all 3 permissions Linear: HDX-3832 --------- Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent 9f9eb5d commit 6f29e73

File tree

2 files changed

+6
-3
lines changed

2 files changed

+6
-3
lines changed

.changeset/better-otters-train.md

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
---
2+
"helm-charts": patch
3+
---
4+
5+
fix: combine ClickHouse app user grants into single query

charts/clickstack/values.yaml

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -305,9 +305,7 @@ clickhouse:
305305
password: '{{ .Values.hyperdx.secrets.CLICKHOUSE_APP_PASSWORD }}'
306306
profile: default
307307
grants:
308-
- query: "GRANT SHOW ON *.*"
309-
- query: "GRANT SELECT ON system.*"
310-
- query: "GRANT SELECT ON default.*"
308+
- query: "GRANT SHOW ON *.*, SELECT ON system.*, SELECT ON default.*"
311309
otelcollector:
312310
password: '{{ .Values.hyperdx.secrets.CLICKHOUSE_PASSWORD }}'
313311
profile: default

0 commit comments

Comments
 (0)