Repository navigation
feat(chart): krateo-sse-proxy 0.1.15 — image 1.1.9, the sargable poll cursor and windowed /events - #69
Merged
Conversation
… cursor and windowed /events The poll cursor no longer wraps Timestamp in a function. Per poll, EXPLAIN goes from 53 parts / 731 granules to 1 part / 1 granule, which frees ~3.25 of ClickHouse's 4 cores on krateo-057. /events tries the newest window first and returns results identical to the unbounded query (sse-proxy#25/#26; reviewer-small PASS). values.yaml image.tag is bumped together with appVersion, as in #65 and #67. The pinned tag takes precedence over the .Chart.AppVersion default, so an appVersion-only bump would keep rendering 1.1.8. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01EzZ1gXDHbHVhPh2bbE8yfi
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Bumps the sse-proxy image to 1.1.9 (krateo-platformops/sse-proxy#25, PR #26 merged as f1d87504; tag
1.1.9peels to that commit; the imageghcr.io/krateo-platformops/sse-proxy:1.1.9exists). The templates are unchanged.What 1.1.9 fixes (measured on krateo-057, read-only)
toUnixTimestamp(Timestamp) > NbecomesTimestamp >= toDateTime64(N+1, 9), which selects the same rows.EXPLAIN indexes=1per poll: 53 parts / 731 granules → 1 part / 1 granule. Before the fix, the poller used ~3.25 of ClickHouse's 4-core limit (~828 polls an hour, ~4.25M rows each), and ClickHouse sat at its CPU limit./events. Newest window first (1h, 24h, then unbounded). Results are identical to the unbounded query; granules read: unfiltered 730 → 4, devday-milan 528 → 86. Equivalence was checked on 057 with pinned samples (count + cityHash64 match).Diff
Chart.yaml: version 0.1.14 → 0.1.15, appVersion "1.1.8" → "1.1.9", plus a changelog comment in the existing house style.values.yaml:image.tag"1.1.8" → "1.1.9". This is required:deployment.yamlrenderskrateo.imagewith.Chart.AppVersiononly as the default tag, and the pinnedimage.tagwins. With an appVersion-only bump,helm templatestill renderssse-proxy:1.1.8. feat(charts): collector 0.3.8 (image 1.0.4) + sse-proxy 0.1.13 (image 1.1.7) #65 and feat(chart): krateo-sse-proxy 0.1.14 — image 1.1.8, the map-only events predicate #67 bumped both fields for the same reason.Verified locally:
helm lintpasses, andhelm templaterendersimage: "ghcr.io/krateo-platformops/sse-proxy:1.1.9". gitleaks: no leaks.Publishing (do not reuse tag
0.1.15)release-oci.yamlfires on anyX.Y.Ztag. Each run packages every first-class chart undercharts/at the literal version in its own Chart.yaml, then pushes all of them. None of these charts uses theCHART_VERSION/APP_VERSIONplaceholders, so the tag name only triggers the run; it does not version anything. Tag0.1.15already exists (2026-09-02, on 4679ee5), so it cannot be pushed again. Publish with aworkflow_dispatchofrelease-ocionmainafter this merges (inputs can stay empty), or by pushing a new unused tag on the merge commit.No roll is requested here.
🤖 Generated with Claude Code
https://claude.ai/code/session_01EzZ1gXDHbHVhPh2bbE8yfi