feat: PLA-1058 bound independent test case concurrency - #37
Merged
Merged
Conversation
Co-Authored-By: Codex <noreply@openai.com>
jeffatgametime
added a commit
that referenced
this pull request
Sep 15, 2026
Admin merge explicitly authorized by Jeff. Exact-head CI, fresh race tests repeated three times, vet, deterministic ordering/fallback contracts and Temporal replay validation passed. Parent #37 is merged and this PR targets master. Co-Authored-By: Codex <noreply@openai.com>
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.
The following code makes an impact.
Purpose of Change
Provide bounded opt-in concurrency for independent top-level hosted test cases without parallelizing ordered steps inside a case or losing cleanup ownership.
Ticket Link
PLA-1058
Change Overview
ConcurrentTestregistration and copied canonicalListCasesidentities.CaseExecutorper hosted worker, shared across packages/runs. Ordinary cases and package hooks use exclusive admission.Stack dependency: #36 provides explicit skip/lifecycle semantics. This PR targets
report-skipped-testsonly while #36 is open. Do not merge this branch into that feature branch; after #36 merges, retarget tomaster, verify the resulting diff and rerun required checks/review.Affected Functionality
Generic testy hosted execution and native teardown. Nil executor preserves serial-in-package behavior; size one serializes cases sharing that executor.
ConcurrentTestis an explicit independence declaration, not blanket activation of legacy nested Parallel calls.No api-tests cases are opted in by this library PR. This is a process-local bound, not a fleet-wide admission service or an exactly-once side-effect guarantee. Non-cooperative callbacks cannot be preempted.
Testing
go test -race ./... -count=3 -timeout=90spass.go vet ./...andgit diff --checkpass.Observability
TestResult.QueueDurmeasures admission delay separately from admitted lifecycle Dur. Canonical names/results and errors remain visible. Runtime peak load, queue/phase signals and exact consumer identity must be validated in the api-tests adoption workstream; no hosted speedup is claimed here.Rollout
Merge #36 first, retarget and verify this PR on master, then normal required review/check gates. No tag or deployment from this PR-management task. api-tests must consume the approved immutable library identity separately. Root coordinates staged worker drain and maximum replica/surge/dependency budget before enabling overlap; retain serial rollback. Hosted consumer/load acceptance remains pending on PLA-1058.