Repository navigation
feat(stovepipe): wire queue enablement control into ingest RPC - #959
Merged
Merged
Conversation
mnoah1
force-pushed
the
mnoah1/stovepipe-ingest-enablement
branch
from
October 27, 2026 21:15
55aeba0 to
1d03c11
Compare
mnoah1
marked this pull request as ready for review
October 27, 2026 21:28
mnoah1
requested review from
a team,
behinddwalls and
sbalabanov
as code owners
October 27, 2026 21:28
prathshenoy
reviewed
Oct 27, 2026
prathshenoy
approved these changes
Oct 27, 2026
Summary: Wire the existing queue enablement state and history into Ingest so callers can record the commit boundaries consumed by GetQueueStatus. Accept ENABLED or DISABLED and return applied or observed enablement status with the request ID. Omitted/UNKNOWN preserves state; repeated settings preserve the boundary and version. Changed settings use the same captured commit URI as the validation request. Enablement remains independent of pause and does not block ingestion or processing. Recording is prospective and uses the existing queue state, history, and summary stores. Test Plan: Bazel tests for Stovepipe, server handlers/mappers, MySQL integration, end-to-end behavior, and generated web API bindings. Repository lint, mock generation, dependency, and build-file checks.
mnoah1
force-pushed
the
mnoah1/stovepipe-ingest-enablement
branch
from
October 28, 2026 02:20
56534ae to
8cc5563
Compare
mnoah1
enabled auto-merge
October 28, 2026 02:20
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Why?
Ingestrpc - when included, the queue will be updated with a transition for the enablement/disablement status. Including it here insures we have a durable transition, as opposed to a more feature flag like behavior that we expect for pauses/other forms of gating.What?
ENABLEDorDISABLEDand return the applied or observed enablement status with the request ID. Omitted/UNKNOWNpreserves state; repeated settings preserve the original boundary and version.Test Plan