Skip to content

ci: split engine_v2_tests out of the c[n-o] test job - #2902

Merged
gzliudan merged 1 commit into
dev-upgradefrom
ci/split-cno-tests
Oct 30, 2026
Merged

gzliudan merged 1 commit into
dev-upgradefrom
ci/split-cno-tests

Conversation

@benjamin202410

Copy link
Copy Markdown
Collaborator

Summary

The `C-[n-o] tests` matrix job is the slowest CI job (~8 min vs ~2–4 min for the others). `build/ci.go test` runs packages sequentially (`-p 1`), and `consensus/tests/engine_v2_tests` alone accounts for ~235s of it.

This moves `engine_v2_tests` into its own `Engine v2 tests` matrix entry and excludes it from `C-[n-o] tests`, so the two run in parallel. No package is dropped or duplicated.

Test plan

  • Checked that the new `C-[n-o]` package list no longer contains `engine_v2_tests`
  • CI: both jobs pass, and the longest test job is shorter

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Oct 26, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: a98f1eee-a040-4e2d-930f-79463da08e9c

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@gzliudan

Copy link
Copy Markdown
Collaborator

@gzliudan
gzliudan merged commit 016b9a4 into dev-upgrade Oct 30, 2026
17 checks passed
@gzliudan
gzliudan deleted the ci/split-cno-tests branch October 30, 2026 01:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants