Skip to content

feat(stovepipe): per-queue config yaml - #714

Open
mnoah1 wants to merge 2 commits into
mnoah1/stovepipe-admission-throttlefrom
mnoah1/stovepipe-admission-config-v2
Open

mnoah1 wants to merge 2 commits into
mnoah1/stovepipe-admission-throttlefrom
mnoah1/stovepipe-admission-config-v2

Conversation

@mnoah1

@mnoah1 mnoah1 commented Sep 15, 2026

Copy link
Copy Markdown
Contributor

Summary

Intent:

  • Make logical admission throttling configurable per Stovepipe queue. We have internal yaml config for this already, but here we were only using the default value.

Changes:

  • Add a strict, validated YAML-backed queue configuration store.
  • Wire optional QUEUE_CONFIG_PATH loading into the Stovepipe server and local stack.
  • Validate configured queues against message-queue tenants and document duration settings.

Revert Plan

Revert this PR to restore the built-in queue configuration.

Summary:
Intent:
- Make logical admission throttling configurable per Stovepipe queue.

Changes:
- Add a strict, validated YAML-backed queue configuration store.
- Wire optional QUEUE_CONFIG_PATH loading into the Stovepipe server and local stack.
- Validate configured queues against message-queue tenants and document duration settings.

This change builds on the general admission throttle introduced by the parent PR.
@mnoah1
mnoah1 force-pushed the mnoah1/stovepipe-admission-config-v2 branch from aa3aa8b to c6cb600 Compare September 15, 2026 21:10
@mnoah1
mnoah1 marked this pull request as draft September 15, 2026 21:10
@mnoah1 mnoah1 changed the title feat(stovepipe): configure admission policies per queue feat(stovepipe): per-queue config yaml Sep 15, 2026
@mnoah1
mnoah1 marked this pull request as ready for review September 15, 2026 21:15
@mnoah1
mnoah1 removed this pull request from stack #715 September 15, 2026 21:20
@mnoah1
mnoah1 added this pull request to stack #716 September 15, 2026 21:21
@mnoah1
mnoah1 marked this pull request as draft September 15, 2026 21:26
@mnoah1
mnoah1 marked this pull request as ready for review September 15, 2026 21:45
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