chore(chart): appVersion 0.2.4, chart 0.3.4 (elasticsql 0.20.3 train) - #18
Merged
Conversation
Last of the six-repo train: core -> extensions -> arrow -> jdbc -> repl -> THIS.
The chart deploys the federation server and its per-ES-version sidecars,
so it inherits the whole 0.20.3 train through the images:
elasticsql 0.20.3 local-file COPY INTO without Hadoop (#183), ES client
exception status preserved (#184), single-table
materialized views (#185), and a DDL statement that
can succeed WITH A WARNING
extensions 0.2.3 the refresh watcher is OPTIONAL — a JOIN materialized
view is created with a warning on a basic licence
instead of rolling back
arrow 0.2.4 statement-kind routing allowlist, so a DDL statement
carrying a JOIN reaches the materialized-view
extension and not the federation JOIN planner (#130)
All seven goldens regenerated. The diff is only what appVersion drives:
image tags, `helm.sh/chart`, `app.kubernetes.io/version`, and the
ConfigMap checksums that follow from them.
🔴 DO NOT MERGE UNTIL THE IMAGES EXIST. Checked at authoring time, all five
tags answer 404 on DockerHub:
softnetwork/softclient4es-federation:0.2.4
softnetwork/softclient4es{6,7,8,9}-arrow-flight-sql:0.2.4
appVersion is the DEFAULT image tag, so merging early ships a chart whose
pods cannot pull. Re-check all five after the arrow 0.2.4 images are built.
Verified locally: `helm lint` clean; all seven goldens re-render
byte-identical; example-three-region and example-heterogeneous-ready stay
byte-identical to each other, as the workflow's anti-copy check requires.
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
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.
Last of the six-repo release train: core → extensions → arrow → jdbc → repl → helm. This chart depends on arrow's docker images, not on the repl bundle, so it is unblocked as soon as arrow is released.
The merge blocker is now cleared
The commit body says do not merge until the images exist — when the branch was authored, all five
0.2.4tags answered 404.appVersionis the default image tag, so merging early would have shipped a chart whose pods cannot pull.Re-checked just now, all five are published:
That is the federation image and all four sidecars, which is what the
Chart.yamlcomment aboveappVersionrequires — a chart that renders a sidecar tag nobody built fails at pull time, not at template time, so no CI gate here would have caught it.What the chart inherits
0.20.3COPY INTOwithout Hadoop (#183), ES client exception status preserved (#184), single-table materialized views (#185), and a DDL statement that can succeed with a warning0.2.30.2.4Changes
Chart.yaml: chart0.3.3→0.3.4,appVersion0.2.3→0.2.4, plus the changelog entry the file keeps inline.appVersiondrives: image tags,helm.sh/chart,app.kubernetes.io/version, and the ConfigMap checksums that follow from them — no template change is hiding in there.Verification (the workflow's own gates, run locally)
helm lint— clean.--set image.tag, since the committed goldens are appVersion renders).example-three-region.yamlandexample-heterogeneous-ready.yamlremain byte-identical to each other, which is the workflow's deliberate anti-copy check — a regression that copies one example over the other would otherwise pass both the golden diff and the smoke undetected.