Skip to content

feat(stovepipe): Persist terminal build identity - #746

Draft
prathshenoy wants to merge 1 commit into
mainfrom
prath.shenoy/stovepipe-project-result-build
Draft

prathshenoy wants to merge 1 commit into
mainfrom
prath.shenoy/stovepipe-project-result-build

Conversation

@prathshenoy

@prathshenoy prathshenoy commented Sep 24, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

Persist the winning terminal build identity in the terminal request-history entry, atomically with the request outcome, so record can later resolve final artifacts without a Build reverse index.

Test Plan

./tool/bazel test //stovepipe/controller/buildsignal:go_default_test //stovepipe/core/messagequeue:go_default_test //stovepipe/extension/storage/mysql:go_default_test --test_output=errors

Revert Plan

Revert this commit to stop persisting terminal build identity.

Issues

@prathshenoy
prathshenoy force-pushed the prath.shenoy/stovepipe-project-result-build branch 3 times, most recently from 208831a to 66cef3f Compare September 25, 2026 03:19
@prathshenoy prathshenoy changed the title feat(stovepipe): Pass terminal build to project result resolver feat(stovepipe): Persist terminal build identity Sep 25, 2026
@prathshenoy
prathshenoy force-pushed the prath.shenoy/stovepipe-project-result-build branch 5 times, most recently from 0236e2b to 8aa2492 Compare September 25, 2026 14:59
Summary:
Persist the winning terminal build identity in the terminal request-history entry, atomically with the request outcome, so record can later resolve final artifacts without a Build reverse index.

Test Plan:
./tool/bazel test //stovepipe/controller/buildsignal:go_default_test //stovepipe/core/messagequeue:go_default_test //stovepipe/extension/storage/mysql:go_default_test --test_output=errors

Revert Plan:
Revert this commit to stop persisting terminal build identity.

Jira Issues: None
@prathshenoy
prathshenoy force-pushed the prath.shenoy/stovepipe-project-result-build branch from 8aa2492 to a51f9d4 Compare September 25, 2026 15:01

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