Skip to content

fix: run codecov on buildjet#6407

Merged
LesnyRumcajs merged 1 commit intomainfrom
codecov-on-buildjet
Jan 12, 2026
Merged

fix: run codecov on buildjet#6407
LesnyRumcajs merged 1 commit intomainfrom
codecov-on-buildjet

Conversation

@LesnyRumcajs
Copy link
Copy Markdown
Member

@LesnyRumcajs LesnyRumcajs commented Jan 12, 2026

Summary of changes

Changes introduced in this pull request:

  • codecov runs on GH workers are constantly timing out, let's switch back to buildjet.

Reference issue to close (if applicable)

Closes

Other information and links

Change checklist

  • I have performed a self-review of my own code,
  • I have made corresponding changes to the documentation. All new code adheres to the team's documentation standards,
  • I have added tests that prove my fix is effective or that my feature works (if possible),
  • I have made sure the CHANGELOG is up-to-date. All user-facing changes should be reflected in this document.

Summary by CodeRabbit

  • Chores
    • Optimized CI/CD infrastructure for improved build efficiency.

✏️ Tip: You can customize this high-level summary in your review settings.

@LesnyRumcajs LesnyRumcajs requested a review from a team as a code owner January 12, 2026 13:46
@LesnyRumcajs LesnyRumcajs requested review from hanabi1224 and sudo-shashank and removed request for a team January 12, 2026 13:46
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Jan 12, 2026

Walkthrough

Updated the GitHub Actions workflow configuration to use a different runner image (buildjet-4vcpu-ubuntu-2204 instead of ubuntu-24.04-arm) and reduced the coverage job timeout from 60 to 45 minutes.

Changes

Cohort / File(s) Summary
GitHub Actions Workflow Configuration
.github/workflows/coverage.yml
Changed runner label from ubuntu-24.04-arm to buildjet-4vcpu-ubuntu-2204 and reduced job timeout from 60 to 45 minutes

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Suggested labels

github_actions

Suggested reviewers

  • sudo-shashank
  • hanabi1224
🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'fix: run codecov on buildjet' directly and clearly describes the primary change: switching the Codecov runner from GitHub Actions to BuildJet.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


📜 Recent review details

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 7eb7569 and 8e60acf.

📒 Files selected for processing (1)
  • .github/workflows/coverage.yml
🧰 Additional context used
🧠 Learnings (1)
📓 Common learnings
Learnt from: LesnyRumcajs
Repo: ChainSafe/forest PR: 5907
File: src/rpc/methods/state.rs:523-570
Timestamp: 2025-08-06T15:44:33.467Z
Learning: LesnyRumcajs prefers to rely on BufWriter's Drop implementation for automatic flushing rather than explicit flush() calls in Forest codebase.
🪛 actionlint (1.7.10)
.github/workflows/coverage.yml

46-46: label "buildjet-4vcpu-ubuntu-2204" is unknown. available labels are "windows-latest", "windows-latest-8-cores", "windows-2025", "windows-2022", "windows-11-arm", "ubuntu-slim", "ubuntu-latest", "ubuntu-latest-4-cores", "ubuntu-latest-8-cores", "ubuntu-latest-16-cores", "ubuntu-24.04", "ubuntu-24.04-arm", "ubuntu-22.04", "ubuntu-22.04-arm", "macos-latest", "macos-latest-xlarge", "macos-latest-large", "macos-26-xlarge", "macos-26", "macos-15-intel", "macos-15-xlarge", "macos-15-large", "macos-15", "macos-14-xlarge", "macos-14-large", "macos-14", "self-hosted", "x64", "arm", "arm64", "linux", "macos", "windows". if it is a custom label for self-hosted runner, set list of labels in actionlint.yaml config file

(runner-label)

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (8)
  • GitHub Check: Build MacOS
  • GitHub Check: cargo-publish-dry-run
  • GitHub Check: Build Ubuntu
  • GitHub Check: Build forest binaries on Linux AMD64
  • GitHub Check: Coverage
  • GitHub Check: All lint checks
  • GitHub Check: tests-release
  • GitHub Check: rubocop
🔇 Additional comments (1)
.github/workflows/coverage.yml (1)

46-47: Architecture change from ARM to x86_64 — verify compatibility.

The runner switch from ubuntu-24.04-arm (ARM64) to buildjet-4vcpu-ubuntu-2204 (x86_64) changes the build architecture. While this should generally work, ensure that:

  1. The sccache bucket has cached artifacts for x86_64 builds
  2. Any architecture-specific dependencies or proof parameters are compatible

The actionlint warning about the unknown label is a false positive—BuildJet runner labels are valid third-party self-hosted runners.

Reducing the timeout from 60 to 45 minutes while moving to BuildJet seems reasonable given their typically faster infrastructure.


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

@LesnyRumcajs LesnyRumcajs added this pull request to the merge queue Jan 12, 2026
Merged via the queue into main with commit 2f39a40 Jan 12, 2026
42 checks passed
@LesnyRumcajs LesnyRumcajs deleted the codecov-on-buildjet branch January 12, 2026 15:06
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