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

feat(stovepipe): publish admitted requests to build#364
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
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