Skip to content

perf(ci): run the PR docker-image cache gate on a GitHub-hosted runner#3185

Open
danceratopz wants to merge 1 commit into
ethereum:forks/amsterdamfrom
danceratopz:perf-ci-better-hive-runner
Open

perf(ci): run the PR docker-image cache gate on a GitHub-hosted runner#3185
danceratopz wants to merge 1 commit into
ethereum:forks/amsterdamfrom
danceratopz:perf-ci-better-hive-runner

Conversation

@danceratopz

Copy link
Copy Markdown
Member

Description

On pull requests the cache-docker-images job is normally a pure weekly-cache restore (~12s), but it gates all test-hive jobs and paid ~2.5 min of self-hosted runner provisioning before starting, roughly a quarter of the workflow's wall time.

Select the runner by event: PRs use ubuntu-latest (picked up in seconds), while push and dispatch runs stay on self-hosted so the weekly cache is populated from GHR egress IPs, avoiding Docker Hub's per-IP rate limits on the shared GitHub-hosted IPs.

Related Issues or PRs

N/A.

Checklist

  • Ran fast static checks to avoid CI fails, see Code Standards & Verifying Changes: just static
  • PR title has the form <type>(<area>): <title>, where <type> and <area> come from an appropriate C-<type>, respectively A-<area>, label. The title should match the target squash commit message.

Cute Animal Picture

429 Too Many Requests

On pull requests the `cache-docker-images` job is normally a pure
weekly-cache restore (~12s), but it gates all `test-hive` jobs and
paid ~2.5 min of self-hosted runner provisioning before starting,
roughly a quarter of the workflow's wall time.

Select the runner by event: PRs use `ubuntu-latest` (picked up in
seconds), while push and dispatch runs stay on self-hosted so the
weekly cache is populated from GHR egress IPs, avoiding Docker Hub's
per-IP rate limits on the shared GitHub-hosted IPs.
@danceratopz danceratopz added A-ci Area: Continuous Integration C-perf Scope: Performance - an optimization or speed-up. labels Jul 16, 2026
@codecov

codecov Bot commented Jul 16, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 93.13%. Comparing base (a45205c) to head (c0e6b0e).

Additional details and impacted files
@@                 Coverage Diff                 @@
##           forks/amsterdam    #3185      +/-   ##
===================================================
- Coverage            93.37%   93.13%   -0.25%     
===================================================
  Files                  624      624              
  Lines                36990    36990              
  Branches              3386     3386              
===================================================
- Hits                 34539    34449      -90     
- Misses                1677     1741      +64     
- Partials               774      800      +26     
Flag Coverage Δ
unittests 93.13% <ø> (-0.25%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@danceratopz
danceratopz marked this pull request as ready for review July 16, 2026 20:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-ci Area: Continuous Integration C-perf Scope: Performance - an optimization or speed-up.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant