feat(ci): migrate staging EEST to alpen-client with dummy OL#1509
Open
feat(ci): migrate staging EEST to alpen-client with dummy OL#1509
Conversation
Replace the old strata-client + alpen-reth + strata-sequencer-client stack with alpen-client running in sequencer mode with --dummy-ol-client. - Add alpen-client to CD build matrix so it gets pushed to ECR - Add docker-compose-eest.yml: minimal alpen-client + regtest bitcoind - Rewrite staging-eest.yml: pull alpen-client from ECR, generate keys via init-alpen-client-keys.sh, run EEST against port 8545 - Drop prover build and proof generation steps (separate concern) - Add ECR tag validation to prevent docker pull failures on None tags - Replace sleep-based readiness check with active RPC polling
The prover step is essential — the point of the workflow is to verify that EE blocks produced from EF spec tests can be proven. - Add back prover-client Docker build step (native mode) - Add prover-client service to docker-compose-eest.yml - Restore "Generate a proof of EE blocks execution" step - Restore continue-on-error on test step so proofs run even if some EF tests fail - Bump timeout back to 120min to account for prover build
Contributor
|
Commit: 83d24dd SP1 Execution Results
|
Codecov Report✅ All modified and coverable lines are covered by tests.
@@ Coverage Diff @@
## main #1509 +/- ##
==========================================
- Coverage 74.33% 65.03% -9.31%
==========================================
Files 783 798 +15
Lines 73713 74794 +1081
==========================================
- Hits 54795 48639 -6156
- Misses 18918 26155 +7237
Flags with carried forward coverage won't be shown. Click here to find out more. 🚀 New features to boost your workflow:
|
The old prover-client binary is incompatible with the new alpen-client setup — it requires --sequencer-rpc pointing to a real OL, which we don't have with --dummy-ol-client. EE proving is being redesigned to be internal to alpen-client via AlpenProverHandle (currently NoopProver). Once that's wired up, EE proof generation can be re-added to this workflow without a separate prover-client container.
storopoli
requested changes
Mar 17, 2026
| - --http.api | ||
| - eth,net,web3,txpool,admin,debug | ||
| - --ee-da-magic-bytes | ||
| - ${EE_DA_MAGIC_BYTES:-ALPT} |
Member
There was a problem hiding this comment.
Not again
Suggested change
| - ${EE_DA_MAGIC_BYTES:-ALPT} | |
| - ${EE_DA_MAGIC_BYTES:-ALPN} |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Fixes STR-2603
staging-eest.ymlis broken. it pulledstrata-clientfrom ECR which had no images, causingdocker pull ...:None→invalid reference formatalpen-client --dummy-ol-client --sequencer+ a regtest bitcoind (needed for DA args that--sequencerrequires via claprequires_all)alpen-clientto the CD build matrix so it gets pushed to ECRChanges
cd.yml: addalpen-clientto the build matrixdocker/docker-compose-eest.yml(new): minimal compose — alpen-client sequencer + regtest bitcoind with healthcheckstaging-eest.yml: rewritten to pull alpen-client from ECR, generate keys viainit-alpen-client-keys.sh, use new composeDeployment note
After merge, the CD workflow needs to run once to push the
alpen-clientimage to ECR before the staging EEST daily run will work.Test plan
docker-compose-eest.ymlstarts alpen-client, RPC responds on :8545, blocks progress