Skip to content

fix(helm): bump clickhouse subchart 9.3.7 -> 9.4.4 (clickhouse 25.7.5)#3524

Merged
nicktrn merged 1 commit intomainfrom
fix/helm-clickhouse-subchart-bump
May 5, 2026
Merged

fix(helm): bump clickhouse subchart 9.3.7 -> 9.4.4 (clickhouse 25.7.5)#3524
nicktrn merged 1 commit intomainfrom
fix/helm-clickhouse-subchart-bump

Conversation

@nicktrn
Copy link
Copy Markdown
Collaborator

@nicktrn nicktrn commented May 5, 2026

Fixes #3520. The bundled bitnami clickhouse subchart was pinned at 9.3.7 (clickhouse 25.6.1-debian-12-r0), which hits a memory-tracker accounting bug under sustained ingest - the global counter overflows to ~7 EiB and every query gets rejected by OvercommitTracker until the pod is restarted. Self-hosters running 4.0.5 through 4.4.5 are exposed regardless of chart version since the subchart pin hadn't moved.

Bumping to 9.4.4 (clickhouse 25.7.5-debian-12-r0) pulls in the 25.7.x memory-tracker fixes. This is also the latest publicly packaged release at oci://registry-1.docker.io/bitnamicharts - that registry has been frozen since 2025-08-28 (Bitnami catalog changes), but the chart source remains under Apache 2 on bitnami/charts. The image continues to resolve via bitnamilegacy/clickhouse per the existing values.yaml override, since bitnami/clickhouse itself moved to paid-only.

Verified locally: helm dependency update + helm lint + helm template + kubeconform across all 57 rendered manifests. Rendered statefulset image is docker.io/bitnamilegacy/clickhouse:25.7.5-debian-12-r0.

Pulls in upstream memory-tracker fixes shipped in clickhouse 25.7.x.
The previous pin (25.6.1) hits a memory-tracker accounting bug under
sustained ingest where the global counter overflows to ~7 EiB, after
which every query is rejected by OvercommitTracker until the pod is
restarted - reported in #3520 with full repro and pod logs.

9.4.4 is the latest packaged release at oci://registry-1.docker.io/
bitnamicharts/clickhouse; that registry has been frozen since
2025-08-28 but the chart source (Apache 2) is still maintained at
bitnami/charts. The image already resolves via bitnamilegacy/clickhouse
(per existing values.yaml override) since bitnami/clickhouse is now
paid-only.

refs #3520
@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented May 5, 2026

⚠️ No Changeset found

Latest commit: 494d797

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 5, 2026

🧭 Helm Chart Prerelease Published

Version: 4.4.5-pr3524.494d797

Install:

helm upgrade --install trigger \
  oci://ghcr.io/triggerdotdev/charts/trigger \
  --version "4.4.5-pr3524.494d797"

⚠️ This is a prerelease for testing. Do not use in production.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 5, 2026

Walkthrough

This pull request updates ClickHouse dependencies in the Kubernetes Helm chart configuration. The Helm chart dependency specification in Chart.yaml was updated from version 9.3.7 to 9.4.4, and the commented ClickHouse image version reference in values.yaml was updated from 25.6.1-debian-12-r0 to 25.7.5-debian-12-r0. Both changes are version updates with no logic or configuration modifications.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning PR description provides detailed context about the bug fix, version changes, and verification steps, but is missing the required template sections (checklist, testing section, changelog section). Add the required PR template sections: checklist items, explicit testing steps description, and a changelog entry summarizing the changes.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and specifically describes the main change: bumping the ClickHouse subchart version with the old and new versions and resulting image tag.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/helm-clickhouse-subchart-bump

Tip

💬 Introducing Slack Agent: The best way for teams to turn conversations into code.

Slack Agent is built on CodeRabbit's deep understanding of your code, so your team can collaborate across the entire SDLC without losing context.

  • Generate code and open pull requests
  • Plan features and break down work
  • Investigate incidents and troubleshoot customer tickets together
  • Automate recurring tasks and respond to alerts with triggers
  • Summarize progress and report instantly

Built for teams:

  • Shared memory across your entire org—no repeating context
  • Per-thread sandboxes to safely plan and execute work
  • Governance built-in—scoped access, auditability, and budget controls

One agent for your entire SDLC. Right inside Slack.

👉 Get started


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown
Contributor

@devin-ai-integration devin-ai-integration Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Devin Review found 0 potential issues.

Open in Devin Review

@nicktrn nicktrn added the ready label May 5, 2026
@nicktrn
Copy link
Copy Markdown
Collaborator Author

nicktrn commented May 5, 2026

ready

@nicktrn nicktrn merged commit a8966a4 into main May 5, 2026
14 checks passed
@nicktrn nicktrn deleted the fix/helm-clickhouse-subchart-bump branch May 5, 2026 09:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Helm chart pins ClickHouse 25.6.1, which has a memory-tracker overflow that breaks the dashboard until pod restart

2 participants