fix: preserve sticky scheduler runtime and event lineage - #572
Draft
zhuanyongxigua wants to merge 4 commits into
Draft
fix: preserve sticky scheduler runtime and event lineage#572zhuanyongxigua wants to merge 4 commits into
zhuanyongxigua wants to merge 4 commits into
Conversation
|
评审任务执行失败。 |
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.
Summary
.git/index.lockcleanup before resuming a stopped sticky sandboxThis is a draft integration PR for the platform support needed by per-project sticky schedulers. It fixes the runtime facade mismatch reproduced after session reload, unrelated-revision sticky churn, missing pre-command run linkage, late cell writes reviving stopped metadata, and routed workloads being absent from the root Event trace.
Testing
Completed on the current branch:
GOTOOLCHAIN=go1.26.2 go test ./pkg/agentcompose/adapters ./pkg/events/webhooks ./pkg/llms/runtimefacade ./pkg/model ./pkg/schedulers ./pkg/storage/configstore ./pkg/storage/sandboxstore -count=1-race -count=1passed for adapters, webhooks, runtimefacade, model, schedulers, and configstore-race -count=1; the complete sandboxstore race package produced no result after about eight minutes and was stoppedtask --force docs:buildgit diff --check origin/main...HEADKnown validation boundary:
newsandbox0b66f661HEAD still needs the complete dev180 sticky E2E: two runs reuse one Sandbox, facade traffic has no 401, pre-command linkage is visible, force-stop remains STOPPED after late command persistence, resume cleanup occurs, and the root trace includes the workloadThe full repository suite currently also contains baseline failures on macOS outside this branch's changed runtime code (timezone and
/varversus/private/varassertions). This Draft is not ready to merge until the final E2E and CI review are complete.Checklist
Draft follow-up