Skip to content

feat(stovepipe): Announce validation results - #646

Open
prathshenoy wants to merge 1 commit into
mainfrom
prath.shenoy/publish-event
Open

feat(stovepipe): Announce validation results#646
prathshenoy wants to merge 1 commit into
mainfrom
prath.shenoy/publish-event

Conversation

@prathshenoy

@prathshenoy prathshenoy commented Aug 27, 2026

Copy link
Copy Markdown
Collaborator

Summary

What:

  • Publish an event whenever a commit's validation result becomes durable, carrying whether the commit is healthy or broken and what it was measured against.
  • Publish a distinct event when validation ends without a verdict, and fail the work when either event cannot be sent.

Why:

  • Let subscriber systems act on trunk validation results as they land, instead of polling for them.
  • Keep the event vendor-neutral so each deployment wires its own subscriber to forward results onward, instead of building any single environment's delivery path into the pipeline.

Test Plan

  • Add unit tests.

Revert Plan

  • Revert this PR. The events are additive and nothing consumes them yet.

Issues

@prathshenoy
prathshenoy force-pushed the prath.shenoy/publish-event branch from 7ac490d to 4e0cd19 Compare August 27, 2026 18:37
@prathshenoy
prathshenoy marked this pull request as ready for review August 27, 2026 18:47
@prathshenoy
prathshenoy enabled auto-merge August 27, 2026 18:51
@prathshenoy
prathshenoy force-pushed the prath.shenoy/publish-event branch 3 times, most recently from 53c7352 to 1d918dc Compare August 27, 2026 19:10
Comment thread stovepipe/controller/record/record.go Outdated
Comment thread stovepipe/controller/record/record.go Outdated
Comment thread stovepipe/controller/record/record.go Outdated
Comment thread stovepipe/controller/record/record.go Outdated
@prathshenoy
prathshenoy force-pushed the prath.shenoy/publish-event branch from 1d918dc to 0a475b2 Compare August 27, 2026 22:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants