Skip to content

feat(stovepipe): publish build repair requests to the fix queue - #936

Open
mnoah1 wants to merge 1 commit into
mnoah1/stovepipe-fix-controllerfrom
mnoah1/stovepipe-fix-queue
Open

mnoah1 wants to merge 1 commit into
mnoah1/stovepipe-fix-controllerfrom
mnoah1/stovepipe-fix-queue

Conversation

@mnoah1

@mnoah1 mnoah1 commented Oct 23, 2026 •

Copy link
Copy Markdown
Contributor

Why?

Let issue tracking hand a build to the fix controller after persisting its issue records.

What?

Add the FixRequest protobuf and topic, a publisher carrying queue and build IDs, and the fix controller's standard Process delivery adapter. Issue tracking can call the publisher after its writes succeed. The adapter validates the message and delegates to the controller introduced by the previous PR.

Test Plan

Run ./tool/bazel test //stovepipe/controller/fix:go_default_test //stovepipe/core/fixrequest:go_default_test //stovepipe/core/messagequeue:go_default_test. Run make lint check-tidy check-gazelle check-mocks.

Revert Plan

Revert this commit and remove any host wiring that calls the publisher.

Issue Links

None — this is the message-wiring layer of the planned repair workflow.

Stack

  1. feat(stovepipe): associate build issues with repairs using entity CAS #934
  2. feat(stovepipe): select and submit one repair per build #935
  3. @ feat(stovepipe): publish build repair requests to the fix queue #936

Define FixRequest and its topic, publish a queue-scoped build identity, and connect queue delivery validation to the fix controller.

Test Plan: Run the fix, fixrequest, messagequeue, storage, and buildanalysis unit targets. Run make lint check-tidy check-gazelle check-mocks.

Revert Plan: Revert this commit.
@mnoah1
mnoah1 added this pull request to stack #937 October 23, 2026 21:04
@mnoah1
mnoah1 marked this pull request as ready for review October 23, 2026 21:10
@mnoah1
mnoah1 requested review from a team, behinddwalls and sbalabanov as code owners October 23, 2026 21:10

This branch has not been deployed

No deployments
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.

1 participant