Skip to content

feat(controller) Conclude controller updates request state to reflect batch outcome#120

Merged
manjari25 merged 3 commits into
mainfrom
manjari/conclude-controller
Mar 6, 2026
Merged

feat(controller) Conclude controller updates request state to reflect batch outcome#120
manjari25 merged 3 commits into
mainfrom
manjari/conclude-controller

Conversation

@manjari25
Copy link
Copy Markdown
Contributor

@manjari25 manjari25 commented Mar 4, 2026

Summary

feat(controller) Conclude controller updates request state to reflect batch outcome

What?

  • Inspect the request IDs within a batch
  • Map batch state to equivalent request state
  • Update request state in the request store
  • Use newly created metrics/core pkg to track metrics

Why?

These changes mark the request "concluded" within the SQ system.

Test Plan

Issues

@manjari25 manjari25 marked this pull request as ready for review March 4, 2026 23:49
@manjari25 manjari25 requested review from a team, behinddwalls and sbalabanov as code owners March 4, 2026 23:49
Comment thread orchestrator/controller/conclude/conclude.go
@manjari25 manjari25 requested a review from behinddwalls March 6, 2026 18:30
@manjari25 manjari25 added this pull request to the merge queue Mar 6, 2026
Merged via the queue into main with commit 47a7b40 Mar 6, 2026
8 checks passed
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