[Docs] Evaluate walgit for repository bootstrap - #1635
Draft
roomote-roomote[bot] wants to merge 1 commit into
Draft
[Docs] Evaluate walgit for repository bootstrap#1635roomote-roomote[bot] wants to merge 1 commit into
roomote-roomote[bot] wants to merge 1 commit into
Conversation
Contributor
| resumed filesystem snapshot. There is no cross-sandbox Git mirror, bundle | ||
| cache, alternates store, LFS bootstrap, or submodule bootstrap. | ||
|
|
||
| Repository clone, fetch, and checkout/reset stages already emit durations via |
Contributor
There was a problem hiding this comment.
Existing telemetry does not emit these stage-level durations. recordPhase reaches setup, which records initializeRepositories as one phase, but initializeRepositories and createWorkspaceManager do not pass that recorder into their nested timedStep calls; WorkspaceManager.timed therefore only writes debug logs for clone, fetch, and checkout. The stated entry gate cannot be calculated from existing setup events, so make stage-level event instrumentation a prerequisite rather than claiming this experiment needs no code change.
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.
What changed
Added a technical evaluation of walgit as a possible repository bootstrap accelerator for disposable Roomote sandboxes. It documents the current clone/startup path, fit and non-fit, security and tenant-isolation constraints, and a gated A/B experiment with statistical success criteria.
The recommendation is a no-go for production adoption and only a conditional go for an isolated blobless
bundle-uriexperiment after existing setup telemetry proves clone transfer is a material bottleneck.Why this change was made
The radar signal identified walgit's object-store-backed Git architecture as potentially useful for large repositories. A complete audit of 128 open PRs found no exact walgit, bundle-URI, mirror, or shared Git object-cache work; PR #934 is adjacent clone-timeout work and remains separate.
The evaluation turns that signal into a durable, source-pinned decision while avoiding a speculative provider or clone-path implementation. It also captures walgit's documented filtered-bundle caveat and current repository-authorization gap before either can become a production risk.
Impact
There is no runtime behavior change. Reviewers gain a concrete decision gate and experiment design that preserves the existing source-control provider as
origin, fetch target, credential authority, and push destination.