Repository navigation
test(cli): cover stop and offline untrack recovery - #1172
Merged
Merged
Conversation
8 tasks
* commit '5e223d667e9eca8d23e07f51787ddf21883362f1': test(indexer): use promoted batch writer in publication failure control test(daemon): bind repair failure injection to the tracked restart configuration test(daemon): use resolved index defaults for restart failure fixture test(daemon): use the registered Go parser package in restart fixture fix(indexer): resume canceled test projections on unchanged startup test(indexer): cover successful cancellable projection repair across groups fix(indexer): stop canceled warmup test projection between complete groups Keep sidecar shutdown from starting implicit SQLite checkpoints Share SQLite physical close-checkpoint policy between stores test(mcp): retire collision worker before missing-owner retry
…erage * commit 'eff4c97a69b84bf8956b62827d3bf7921ca6e871': test(daemon): join restarted relay before closing host transport test(indexer): document checked persistence outside delta staging
…tion * commit 'ffbec2b21bde2c3ca991aed6e2847b2870424bfc': test: await catalog daemon serve admission before returning
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.
An unwanted cold warmup has documented recovery steps and a repeatable opt-in test: stop the daemon, untrack the absolute repository path while offline, then restart. The test observes active indexing, measures complete stop through child exit and socket removal, verifies the store lock is released, and checks that restart purges the removed repository while keeping the other repository ready.
Closes #1029. Prerequisite #1022 is merged. The CLI guide, onboarding troubleshooting and v0.64.12 upgrade note use the same recovery procedure. Offline unit tests cover removal, unknown paths and unresolvable bare prefixes.
At final head
e6a8b4b79957769262006e75bb0ae214104f8654, affected race controls and all pinned lint, formatting, vet, CGO/workflow and module checks passed before push. Native evidence at517d4d146remains applicable: the final integration changes only an upstream test fixture; all six issue-specific files and production code are unchanged.These runs cover 80 normal daemon exits. Process, socket, fixture and handle cleanup was verified. Linux used the retained product built at
5e223d667, whose non-test Go source matches the integration, and a fresh race test binary from517d4d146. An initial owned-cache permission failure occurred before product execution and is retained separately; no native case failed or was retried. Detailed evidence.Final-head CI passed (24 successful, two skipped checks), and the PR merged as
12625b0ca18976e8b3f2351c2f668fdd439f81f0. #1029 is closed and marked Done on the project board. Acceptance retains each native result’s original source attribution.