Skip to content

feat(stovepipe): publish admitted requests to build - #364

Merged
mnoah1 merged 2 commits into
mainfrom
mnoah1/stovepipe-process-build-handoff
Jul 14, 2026
Merged

mnoah1 merged 2 commits into
mainfrom
mnoah1/stovepipe-process-build-handoff

Conversation

@mnoah1

@mnoah1 mnoah1 commented Jul 14, 2026 •

Copy link
Copy Markdown
Contributor

What

  • Add the internal id-only BuildRequest contract and build topic.
  • Publish admitted requests from process and re-publish processing requests to recover missed handoffs.
  • Register the publish-only build topic and cover handoff/recovery failures.

Why

The upcoming build stage needs an idempotent handoff after process commits the immutable strategy and baseline.

Test plan

  • ./tool/bazel test //stovepipe/controller/process:go_default_test //stovepipe/core/messagequeue:go_default_test //service/stovepipe/server:go_default_library
  • make proto && make gazelle

@mnoah1
mnoah1 requested review from a team, behinddwalls and sbalabanov as code owners July 14, 2026 19:55
Base automatically changed from mnoah1/stovepipe-process-build-strategy to main July 14, 2026 22:27
mnoah1 added 2 commits July 14, 2026 22:29
Publish id-only build requests after process persists the immutable build strategy, and recover missed handoffs through processing-state redelivery.
Keep the Stovepipe queue contract inputs in Gazelle's required lexical order so CI metadata checks remain clean.
@mnoah1
mnoah1 force-pushed the mnoah1/stovepipe-process-build-handoff branch from 15d3942 to 34fb8ea Compare July 14, 2026 22:30
@mnoah1
mnoah1 enabled auto-merge July 14, 2026 22:31
@mnoah1
mnoah1 added this pull request to the merge queue Jul 14, 2026
Merged via the queue into main with commit 82c88dc Jul 14, 2026
15 checks passed
@behinddwalls
behinddwalls deleted the mnoah1/stovepipe-process-build-handoff branch July 14, 2026 22:35

This branch was previously deployed

1 inactive deployment
stack-rebase — 34fb8ea1 Deployed Jul 14, 2026 by mnoah1 via Rebase Stack #216
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants