Skip to content

fix(ci): fix api rate limiting issue in consume flow#1818

Merged
danceratopz merged 2 commits intoethereum:forks/osakafrom
danceratopz:fix-consume-ci-rate-limiting
Nov 27, 2025
Merged

fix(ci): fix api rate limiting issue in consume flow#1818
danceratopz merged 2 commits intoethereum:forks/osakafrom
danceratopz:fix-consume-ci-rate-limiting

Conversation

@danceratopz
Copy link
Member

🗒️ Description

This PR:

  • Fixes the issue here by specifying an explicit release URL instead of using a release specifier. In the case of a release specifier consume cache pages all releases which can lead to rate limiting.
  • Adds a master.yaml file, analagous to latest.yaml, but for client nightly builds, instead of creating the clients file on the fly.

🔗 Related Issues or PRs

Follow-up to:

✅ Checklist

  • All: Ran fast tox checks to avoid unnecessary CI fails, see also Code Standards and Enabling Pre-commit Checks:
    uvx tox -e static
  • All: PR title adheres to the repo standard - it will be used as the squash commit message and should start type(scope):.
  • All: Considered adding an entry to CHANGELOG.md.
  • All: Considered updating the online docs in the ./docs/ directory.
  • All: Set appropriate labels for the changes (only maintainers can apply labels).

Cute Animal Picture

image

@danceratopz danceratopz added A-test-consume Area: execution_testing.cli.pytest_commands.plugins.consume A-ci Area: Continuous Integration labels Nov 27, 2025
Copy link
Contributor

@spencer-tb spencer-tb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Thanks. One comment.

@danceratopz danceratopz merged commit b105c23 into ethereum:forks/osaka Nov 27, 2025
12 of 13 checks passed
mirgee pushed a commit to mirgee/execution-specs that referenced this pull request Dec 5, 2025
Co-authored-by: danceratopz <danceratopz@gmail.com>
@danceratopz danceratopz deleted the fix-consume-ci-rate-limiting branch December 30, 2025 07:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-ci Area: Continuous Integration A-test-consume Area: execution_testing.cli.pytest_commands.plugins.consume

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants