Skip to content

Conversation

spencer-tb
Copy link
Contributor

@spencer-tb spencer-tb commented Sep 11, 2025

🗒️ Description

Aims to fix the random fails during the tests-develop fill stage in the tox verify workflow.

cc https://github.com/ethereum/execution-spec-tests/actions/runs/17620738824/job/50115936119

The workflow is still running so the assumption is that the runner is running out of memory.

🔗 Related Issues or PRs

N/A.

✅ Checklist

  • All: Ran fast tox checks to avoid unnecessary CI fails, see also Code Standards and Enabling Pre-commit Checks:
    uvx --with=tox-uv tox -e lint,typecheck,spellcheck,markdownlint
  • 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).

@spencer-tb spencer-tb added scope:ci Scope: Continuous Integration type:chore Type: Chore labels Sep 11, 2025
Copy link
Member

@marioevz marioevz left a comment

Choose a reason for hiding this comment

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

Smol comment :)

@@ -154,8 +154,8 @@ jobs:
strategy:
matrix:
include:
- os: [self-hosted-ghr, size-xl-x64]
name: self-hosted-ghr-xl-x64
- os: [self-hosted-ghr, size-gigachungus-x64]
Copy link
Member

Choose a reason for hiding this comment

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

Could we try the smaller chungus first? I feel like we should only request the really big one for fixture building since that's the most critical workflow we have that requires it. Might be just me overthinking though :P

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ahh I thought this was the next one.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Switched :) thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
scope:ci Scope: Continuous Integration type:chore Type: Chore
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants