dev-inf: use targeted tests and increase timeout in issue-autosolve#164307
Open
ajstorm wants to merge 1 commit intocockroachdb:masterfrom
Open
dev-inf: use targeted tests and increase timeout in issue-autosolve#164307ajstorm wants to merge 1 commit intocockroachdb:masterfrom
ajstorm wants to merge 1 commit intocockroachdb:masterfrom
Conversation
The autosolver's single Claude attempt was consuming the entire 120-minute job timeout running broad test suites (e.g. ./dev test pkg/sql or ./dev testlogic without --files). Update the prompt to explicitly instruct Claude to run only targeted tests for the specific packages and files it changed, and bump the job timeout to 180 minutes as a safety net for initial Bazel compilation overhead. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Contributor
Member
|
This change is |
rickystewart
approved these changes
Feb 24, 2026
Collaborator
Author
|
/trunk merge TFTR! |
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
3 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.
Summary
tests for the specific packages/files it changed (e.g.
./dev test <pkg> -f=<Test>,./dev testlogic --files=<file>), rather than broad suites like./dev test pkg/sqlor./dev testlogicwithout--filesthat can run forhours.
net, since even targeted tests require initial Bazel compilation on a fresh
runner.
Release note: None
Epic: None
🤖 Generated with Claude Code