Skip to content

Conversation

@spencer-tb
Copy link
Contributor

@spencer-tb spencer-tb commented Oct 27, 2025

Description

Adds a CI workflow to validate hive integration with consume commands. Previously, we lacked CI for consume, which has led to breaking changes going undetected until they impacted downstream EL clients.

This workflow provides early detection of consume-related regressions, preventing disruption to client teams who depend on our test fixtures.

What's Tested

The workflow runs four parallel test jobs:

  • ethereum/eels/consume-engine
  • ethereum/eels/consume-rlp
  • ethereum/eels/consume-sync
  • ethereum/eels/consume-engine Dev mode validation

All tests use go-ethereum as the reference client and filter to a subset of Osaka fork tests to keep CI runtime reasonable while still providing meaningful coverage. We pull the latest go-ethereum image from ethpandaops.

We run a single test just to check for no fails. It assumes go-ethereum will pass this test always. It takes no longer that 5 minutes to run the workflow.

We may consider expanding the filter for when we run this workflow, to include any changes to packages/testing.

@spencer-tb spencer-tb added C-chore Category: chore A-ci Area: Continuous Integration labels Oct 27, 2025
@spencer-tb spencer-tb changed the title chore(ci): add hive consume tests workflow feat(ci): add hive consume tests workflow Oct 27, 2025
@spencer-tb spencer-tb removed the C-chore Category: chore label Oct 27, 2025
@spencer-tb spencer-tb force-pushed the add-hive-ci-check branch 8 times, most recently from 27793d7 to 7d6b9fd Compare October 27, 2025 13:39
@spencer-tb
Copy link
Contributor Author

Passing them here: https://github.com/ethereum/execution-specs/actions/runs/18876034305/job/53865955924?pr=1692

I added the filtering back now so this is ready for review!

Copy link
Member

@danceratopz danceratopz left a comment

Choose a reason for hiding this comment

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

Looks like we need a rebase - I will do this and address some issues directly :)

Copy link
Member

@danceratopz danceratopz left a comment

Choose a reason for hiding this comment

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

@spencer-tb this is awesome, we've been missing this for a while! 😅
I rebased so I could fix-up the paths, added a couple more paths to trigger the workflow and updated to hte astral-sh/update-uv action.

One other suggestion below.

@spencer-tb spencer-tb changed the title feat(ci): add hive consume tests workflow enhance(ci): add hive consume tests workflow Oct 29, 2025
@spencer-tb spencer-tb added the C-enhance Category: an improvement or new feature label Oct 29, 2025
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-enhance Category: an improvement or new feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants