feat(ingestion): add control plane to rust deploy matrix - #71014
Merged
Conversation
The image builds via rust-images.yml but had no deploy-matrix entry, so master merges never dispatched a commit_state_update to charts and the pinned digest went stale.
|
Note 🤖 stamphog reviewed Gates denied (CI/deploy workflow file touched, tier T2-never) and this is CI/deploy risk territory; author is not on the owning team and there's no independent review yet — plus two bot reviews are actively in flight (👀), so approval must wait for those to complete.
Gate mechanics and policy version
|
Contributor
|
Reviews (1): Last reviewed commit: "feat(ingestion): add control plane to ru..." | Re-trigger Greptile |
k33lan
approved these changes
Jul 15, 2026
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
2 participants
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.
Problem
ingestion-control-planebuilds on master viarust-images.yml(e.g. the run for 6df5920 built and pushed it), but thedeployjob inrust-docker-build.ymlhas no matrix entry for it, so nocommit_state_updatedispatch reaches PostHog/charts andstate/ingestion-control-plane.yamlstays pinned to a stale digest.Changes
ingestion-control-planerelease to the deploy matrix (digest_key/values_keyfollowing the ingestion-consumer entry). The entry inherits the existing job-level gate (CD_DEPLOY_ENABLED+ PostHog org + master), per the deploy-gating conventions.How did you test this code?
hogli lint:workflowspasses (all 6 checks across 101 workflows). Matrix-only change to an already-gated job; the deploy step itself is unchanged and skips gracefully when a build didn't produce this image's digest.Automatic notifications
Docs update
None needed.
🤖 Agent context
Autonomy: Human-driven (agent-assisted)
Built with Claude Code, directed by @jose-sequeira, after tracing why the charts state digest wasn't updated by the #70828 merge: the build job ran but there was no
deploy ingestion-control-planejob in the run. Skills invoked: /gating-production-deploys (confirmed the matrix entry inherits the existing job-level gate rather than needing its own).