Skip to content

ci: reuse debug dependencies for E2E builds - #420

Merged
tac0turtle merged 1 commit into
mainfrom
ci/e2e-build-reuse
Oct 26, 2026
Merged

tac0turtle merged 1 commit into
mainfrom
ci/e2e-build-reuse

Conversation

@tac0turtle

Copy link
Copy Markdown
Contributor

The E2E workflow builds a release node binary before nextest builds the dependency graph again in the debug/test profile. On #419, the release build alone took 21m25s.

Build the E2E node in the debug profile and select ev-reth plus ev-tests in both Cargo commands so their dependencies can be reused. The TypeScript E2E harness already supports target/debug/ev-reth. Optimized builds remain covered by the Docker workflow.

Validation:

  • Workflow YAML parses; git diff --check passes.
  • The exact revised build and nextest commands passed locally with the dependency upgrades from chore(deps): upgrade Reth to 2.8 and consolidate dependency updates #419: 16 Rust E2E tests passed, and all 5 client E2E tests passed against the debug binary.
  • With existing local artifacts, the build took 1.18s and nextest compilation took 2.14s; these are warm local timings, not a cold CI benchmark.

This PR contains only the workflow change and is based directly on main. The dependency updates remain in #419.

@tac0turtle
tac0turtle requested a review from a team as a code owner October 26, 2026 16:40
@coderabbitai

coderabbitai Bot commented Oct 26, 2026

Copy link
Copy Markdown

Warning

Review limit reached

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Next included review available in 30 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

Learn how review limits work.

Review configuration:

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: 69b9c5ac-c86d-4016-bde9-c051d0a9828f
Commits

Reviewing files that changed from the base of the PR and between 0488268 and 66c3ca8.

Files selected for processing: 1
  • .github/workflows/e2e.yml
  • 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.

@tac0turtle
tac0turtle merged commit 980faad into main Oct 26, 2026
16 checks passed
@tac0turtle
tac0turtle deleted the ci/e2e-build-reuse branch October 26, 2026 17:00
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.

2 participants