Skip to content

docs(stovepipe): propose request history API - #634

Merged
mnoah1 merged 1 commit into
mnoah1/stovepipe-event-historyfrom
mnoah1/stovepipe-history-api
Sep 1, 2026
Merged

docs(stovepipe): propose request history API#634
mnoah1 merged 1 commit into
mnoah1/stovepipe-event-historyfrom
mnoah1/stovepipe-history-api

Conversation

@mnoah1

@mnoah1 mnoah1 commented Aug 26, 2026

Copy link
Copy Markdown
Contributor

Summary

Intent:

  • Expose the retained Stovepipe request log as public request history using SubmitQueue-aligned API terminology and response shapes.
  • Keep current-state reads owned by operational entities instead of introducing a competing materialized projection.

Changes:

  • Return ordered HistoryEvent values directly from request-ID lookup.
  • Return repeated RequestHistory values from URI lookup; each contains only the request ID and ordered events, while the current mapping yields one and the contract can represent future retained attempts.
  • Resolve URI, build strategy, and base URI from the Request instead of duplicating them in history responses.
  • Define the direct projection from stored RequestLog records and explain why Stovepipe does not need SubmitQueue separate history materialization.
  • Keep the raw request-log metadata map and validation degree out of the initial public API; a typed projection can be added when metadata readers and enforcement land.

This PR builds on the request-log RFC in #633.

Test Plan

  • ./tool/bazel test //api/stovepipe/... //service/stovepipe/... //stovepipe/...
  • make lint check-tidy check-gazelle

Revert Plan

Revert this documentation-only PR; #633 remains independently reviewable.

Jira Issues

None.

@mnoah1
mnoah1 force-pushed the mnoah1/stovepipe-history-api branch 3 times, most recently from 77acc2c to c811781 Compare August 26, 2026 17:33
@mnoah1
mnoah1 marked this pull request as ready for review August 26, 2026 17:35
@mnoah1
mnoah1 requested review from a team, behinddwalls and sbalabanov as code owners August 26, 2026 17:35
@mnoah1
mnoah1 force-pushed the mnoah1/stovepipe-history-api branch from c811781 to 3eec207 Compare August 26, 2026 17:36
@mnoah1
mnoah1 force-pushed the mnoah1/stovepipe-history-api branch from 3eec207 to a992335 Compare August 31, 2026 16:28
@mnoah1
mnoah1 force-pushed the mnoah1/stovepipe-history-api branch 2 times, most recently from 3b773c2 to 18228bd Compare August 31, 2026 17:19
@mnoah1
mnoah1 force-pushed the mnoah1/stovepipe-history-api branch 2 times, most recently from aee0f33 to c327e9b Compare August 31, 2026 22:38
@mnoah1
mnoah1 force-pushed the mnoah1/stovepipe-history-api branch from c327e9b to 9b7d685 Compare August 31, 2026 22:52
@mnoah1
mnoah1 force-pushed the mnoah1/stovepipe-history-api branch from 9b7d685 to 10007b7 Compare August 31, 2026 23:03
@mnoah1
mnoah1 added this pull request to the merge queue Sep 1, 2026
Merged via the queue into main with commit d0bb708 Sep 1, 2026
15 checks passed
@mnoah1
mnoah1 deployed to stack-rebase September 1, 2026 15:11 — with GitHub Actions Active
@behinddwalls
behinddwalls deleted the mnoah1/stovepipe-history-api branch September 1, 2026 15:13
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