Skip to content

[draft/wip] feat(stovepipe): admit latest head and publish to build#335

Draft
mnoah1 wants to merge 2 commits into
mnoah1/stovepipe-process-queueconfigfrom
mnoah1/stovepipe-process-admit
Draft

[draft/wip] feat(stovepipe): admit latest head and publish to build#335
mnoah1 wants to merge 2 commits into
mnoah1/stovepipe-process-queueconfigfrom
mnoah1/stovepipe-process-admit

Conversation

@mnoah1

@mnoah1 mnoah1 commented Jul 10, 2026

Copy link
Copy Markdown
Contributor

Add BuildRequest queue contract and wire process to admit the latest accepted head when a slot is open: cold-start full strategy, increment in_flight_count, transition to processing, and publish to build. Processing redeliveries republish to build. Gate-closed heads ack and wait for a follow-up defer PR.

Why?

What?

Test Plan

Issue

@mnoah1 mnoah1 changed the title feat(stovepipe): admit latest head and publish to build [draft/wip] feat(stovepipe): admit latest head and publish to build Jul 10, 2026
@mnoah1 mnoah1 force-pushed the mnoah1/stovepipe-process-admit branch from a384a94 to 86eaa35 Compare July 10, 2026 17:24
@mnoah1 mnoah1 force-pushed the mnoah1/stovepipe-process-queueconfig branch 2 times, most recently from c814927 to acdd570 Compare July 10, 2026 18:52
@mnoah1 mnoah1 force-pushed the mnoah1/stovepipe-process-admit branch from 86eaa35 to 9008831 Compare July 10, 2026 19:47
@behinddwalls behinddwalls force-pushed the mnoah1/stovepipe-process-queueconfig branch from acdd570 to 98cef3f Compare July 10, 2026 20:07
Increment in_flight_count and transition accepted→processing with a
cold-start full build strategy when the concurrency gate is open.
Build queue publish lands in a follow-up PR.
@behinddwalls behinddwalls force-pushed the mnoah1/stovepipe-process-admit branch from 9008831 to 44606ba Compare July 10, 2026 20:07
Move max-concurrent enforcement into admitRequest's retry loop so
incrementInFlightCount is a single CAS increment that reloads on version mismatch.
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