[fm] Guard support bundle creation with SitrepGuardedInsert.#10535
Open
mergeconflict wants to merge 1 commit into
Open
Conversation
mergeconflict
added
the
fault-management
mergeconflict
force-pushed
the
mergeconflict/fm-sitrepguardedinsert-support-bundles
branch
from
e15c2c1 to
e5d67e2
Compare
mergeconflict
force-pushed
the
mergeconflict/fm-sitrepguardedinsert-alerts
branch
from
1bc2f60 to
62cc3dd
Compare
mergeconflict
force-pushed
the
mergeconflict/fm-sitrepguardedinsert-support-bundles
branch
from
e5d67e2 to
809d7d3
Compare
mergeconflict
force-pushed
the
mergeconflict/fm-sitrepguardedinsert-alerts
branch
from
62cc3dd to
3d5af68
Compare
The support-bundle mirror of the preceding alert change: `impl SitrepGuardedResource for SupportBundle`, the `support_bundle_generation` column and `rendezvous_support_bundle_created` marker table (migration fm-bundle-resource-deletion), `support_bundle_create`'s FM path routed through the combinator inside its transaction, support-bundle generation tracking and carry-forward, the fm_rendezvous bundle loop, the fm_analysis bundle-marker lookup, and the omdb display. `support_bundle_generation` and `alert_generation` are tracked and guarded independently, so a stale generation for one resource aborts only that resource's rendezvous loop.
mergeconflict
force-pushed
the
mergeconflict/fm-sitrepguardedinsert-support-bundles
branch
from
809d7d3 to
7128ff2
Compare
mergeconflict
force-pushed
the
mergeconflict/fm-sitrepguardedinsert-alerts
branch
from
3d5af68 to
a1beef9
Compare
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
2 participants
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.
Wire the support bundle resource through
SitrepGuardedInsert:impl SitrepGuardedResource for SupportBundle;support_bundle_generationonfm_sitrepand therendezvous_support_bundle_createdmarker table (migration fm-bundle-resource-deletion);support_bundle_create's FM path routes through the combinator, surfacing a stale sitrep asError::Conflict;SitrepBuildertrackssupport_bundle_generation, bumping it when the outstanding support-bundle-request set changes; the closed-case carry-forward filter drops fully-satisfied closed cases and keeps those with unsatisfied support bundle requests;support_bundle_generationandalert_generationare tracked and guarded independently, so a stale generation for one resource aborts only that resource's rendezvous loop.Context: #10248, builds on #10532 and #10533. Note, garbage collection for
rendezvous_support_bundle_createdrows is not in this PR, that should be coming later this week if all goes well.