Sync develop → main for v1.5.1 chart release#230
Merged
Conversation
chore(#227): bump greenfield ingestor baseline to v0.3.6
Contributor
|
👋 Heads-up — Code review queue is at 10 / 8 Above the WIP limit. The team convention is to review existing PRs before opening new work. Open PRs currently in Code review (oldest first):
Pull from review before opening new work. (This is a nudge from the kanban WIP check, not a block.) |
LukasWodka
approved these changes
Jun 8, 2026
5 tasks
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.
Promote develop → main for the v1.5.1 chart release
Ships the ingestor greenfield-baseline bump to customers. On merge, create a GitHub Release
v1.5.1to triggerrelease-helm-chart.yaml(packages + publishes to the gh-pages helm repo).What this carries
Exactly one commit — the baseline bump (#228, toward #227):
client/values.yamlimages.ingestor.digest0.3.2(d361fa77…) →0.3.6(dfdaa7e6…, multi-arch)client/Chart.yamlversion+appVersion1.5.0 → 1.5.1Why
A customer (Opentarget) is stuck on ingestor v0.3.2, hitting
data-ingestors#150("empty cells flagged as non-numeric", fixed in 0.3.3). main currently publishes chart 1.5.0, whose pinned baseline is v0.3.2 — so fresh installs andhelm upgradekeep landing there. Bumping the baseline to v0.3.6 means fresh installs and upgrades land on the fixed image.Precedent: the previous baseline bump (v0.3.1→v0.3.2) shipped as chart v1.4.4 the same way.
After merge
v1.5.1(tag on main) → publishes chart tohttps://tracebloc.github.io/client.helm repo update && helm upgrade … tracebloc/client→ lands on v0.3.6.Toward #227. Release-gated promotion — fr-gate + second approver (no self-approve).
🤖 Generated with Claude Code
Note
Low Risk
Helm values and chart version only; no template or runtime logic changes, with a targeted image digest bump for a known customer regression.
Overview
Releases client chart v1.5.1 by promoting the ingestor greenfield baseline from v0.3.2 to v0.3.6 (
images.ingestor.digestinclient/values.yaml), with matchingversion/appVersionbumps inclient/Chart.yaml.Fresh installs and
helm upgradewill seed jobs-manager’sINGESTOR_IMAGE_DIGESTwith the v0.3.6 multi-arch image so ingestion Jobs no longer land on the stale 0.3.2 pin (including the empty-cell / NULL handling fixes called out in the values comments). The floatingimages.ingestor.tag(0.3) is unchanged; ongoing drift is still handled by image-refresh.Reviewed by Cursor Bugbot for commit 6f2a70a. Bugbot is set up for automated code reviews on this repo. Configure here.