Rebalance test buckets (CopterTests1A, 1B etc etc) into 15 minute chunks#33755
Open
peterbarker wants to merge 5 commits into
Open
Rebalance test buckets (CopterTests1A, 1B etc etc) into 15 minute chunks#33755peterbarker wants to merge 5 commits into
peterbarker wants to merge 5 commits into
Conversation
peterbarker
force-pushed
the
pr-claude/ci-rebalance-test-buckets
branch
from
July 17, 2026 04:13
76f9063 to
f4ce958
Compare
github-actions
Bot
added
CI
Copter
github_actions
peterbarker
force-pushed
the
pr-claude/ci-rebalance-test-buckets
branch
from
July 17, 2026 07:45
f4ce958 to
d419afd
Compare
CopterTests2b had grown to ~50 minutes of test wall time per CI run while the other buckets ran 5 to 14 minutes. Redistribute it as contiguous, order-preserving slices, sized using per-test wall times extracted from --show-test-timings output of the last ten green master CI runs: 1a 14.6m 1b 14.6m 1c 15.0m 1d 12.9m 1e 13.9m 2a 14.2m 2b 14.2m The FixedYawCalibration/SITLCompassCalibration pair stays at the front of CopterTests2a, in that order; tests may run after the pair, so 2a now takes a share. The comment there now states the actual constraint rather than saying the test must run alone. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
PlaneTests1b had grown to ~32 minutes of test wall time per CI run while PlaneTests1a ran ~11 minutes and PlaneTests1c under one. Move MAVFTP through RudderArmedTakeoffRequiresNeutralThrottle to 1a and the tail from MODE_SWITCH_RESET onwards to 1c, leaving ~14 minutes of test time in each bucket, sized using per-test wall times extracted from --show-test-timings output of the last ten green master CI runs. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
The quadplane suite ran as a single ~23-minute CI job while the Copter and Plane suites are balanced to ~15-minute buckets. Split the test list into QuadPlaneTests1a (common tests through VTOLQuicktune_CPP, ~12m) and QuadPlaneTests1b (PrecisionLanding onwards, ~11m), sized using per-test wall times extracted from --show-test-timings output of a green master CI run. test.QuadPlane still runs the full list. The build_ci.sh target and workflow matrix entry are replaced in the same commit so every commit runs green in CI. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
The quadplane suite ran as a single ~23-minute CI job while the Copter and Plane suites are balanced to ~15-minute buckets. Split the test list into QuadPlaneTests1a (common tests through VTOLQuicktune_CPP, ~12m) and QuadPlaneTests1b (PrecisionLanding onwards, ~11m), sized using per-test wall times extracted from --show-test-timings output of a green master CI run. test.QuadPlane still runs the full list. The build_ci.sh target and workflow matrix entry are replaced in the same commit so every commit runs green in CI. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
The quadplane suite ran as a single ~23-minute CI job while the Copter and Plane suites are balanced to ~15-minute buckets. Split the test list into QuadPlaneTests1a (common tests through VTOLQuicktune_CPP, ~12m) and QuadPlaneTests1b (PrecisionLanding onwards, ~11m), sized using per-test wall times extracted from --show-test-timings output of a green master CI run. test.QuadPlane still runs the full list. The build_ci.sh target and workflow matrix entry are replaced in the same commit so every commit runs green in CI. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
peterbarker
force-pushed
the
pr-claude/ci-rebalance-test-buckets
branch
from
July 17, 2026 11:22
d419afd to
d6b662b
Compare
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
1 participant
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
Rebalances the time required for each bucket to be somewhat uniform across buckets
Classification & Testing (check all that apply and add your own)
Description
One of the Copter buckets was 50 minutes long, another just 3 or 4.
Bring up is about 3-4 minutes for each machine.
Also splits quadplane into 2 15 minute buckets.