Skip to content

docs(rfc): Fold the queue pause decision into the RFC - #871

Merged
manjari25 merged 2 commits into
mainfrom
manjari/queue-pause-6-docs
Oct 15, 2026
Merged

manjari25 merged 2 commits into
mainfrom
manjari/queue-pause-6-docs

Conversation

@manjari25

Copy link
Copy Markdown
Contributor

Why?

The pause RFC still presented two options as an open comparison, but the stack in #862 to #865 and the e2e tests implement one of them. An RFC describes what is on main, so it now records the design that shipped and only the alternative it rejected.

What?

  • Rewrites the pause RFC around the shipped design: a separate queuepause extension, reject (FailedPrecondition, carrying the operator's reason) and freeze, both failing closed on a read error. A cancelled context is passed through.
  • Documents the noop, file and fake stores, QUEUE_PAUSE_CONFIG_PATH, and the shared wiring helper.
  • Adds the requirements an implementation of queuepause.Store must meet, such as failing closed on read errors, strict parsing, and failing at construction on bad configuration.
  • States that the gateway's own request-log consumer is not frozen, so status and history keep materializing during a freeze.
  • Keeps the queue-config option only as a short rejected alternative, and drops the resolved fail-open question.
  • Corrects the consumer gate RFC: orchestrator stages share one consumer group, and production pause is a separate mechanism that wraps the gate. Updates the RFC index entry.

Test Plan

Docs only, no runnable change. make lint passes.

Issue

@manjari25
manjari25 added this pull request to stack #868 October 14, 2026 21:00
@manjari25
manjari25 marked this pull request as ready for review October 14, 2026 22:04
@manjari25
manjari25 requested review from a team, behinddwalls and sbalabanov as code owners October 14, 2026 22:04
@manjari25
manjari25 force-pushed the manjari/queue-pause-6-docs branch from 36bd42f to e547343 Compare October 15, 2026 00:04
@behinddwalls
behinddwalls added this pull request to the merge queue Oct 15, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue because a pull request earlier in the stack was removed Oct 15, 2026
@manjari25
manjari25 added this pull request to the merge queue Oct 15, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue because a pull request earlier in the stack was removed Oct 15, 2026
@manjari25
manjari25 force-pushed the manjari/queue-pause-6-docs branch from e547343 to 461e255 Compare October 15, 2026 22:15
@manjari25
manjari25 added this pull request to the merge queue Oct 15, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Oct 15, 2026
Base automatically changed from manjari/queue-pause-5-e2e to main October 15, 2026 22:57
@manjari25
manjari25 force-pushed the manjari/queue-pause-6-docs branch from 461e255 to 503c039 Compare October 15, 2026 22:57
@manjari25
manjari25 added this pull request to the merge queue Oct 15, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Oct 15, 2026
@manjari25
manjari25 added this pull request to the merge queue Oct 15, 2026
Merged via the queue into main with commit c41f602 Oct 15, 2026
21 checks passed
@manjari25
manjari25 deleted the manjari/queue-pause-6-docs branch October 15, 2026 23:28
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