Skip to content

Conversation

@rodrigo-o
Copy link
Collaborator

@rodrigo-o rodrigo-o commented Apr 21, 2025

Motivation

PRs were failing in main due to assertoor (and specifically the update of ethereum-package)

Description

This PR updates kurtosis version as well as the clients one, it doesn't add the electra_fork_epoch param to still run in deneb.

The PR also add some minor changes, it fixed some hex values in the config matching the variables ending as FORK_VERSION, to avoid listing the new ones added in ethereum-package one by one and adds a convenient way to run assertoor locally.

@rodrigo-o rodrigo-o changed the title support electra version of kurtosis chore: update to the latest version of kurtosis Apr 21, 2025
title: "Wait for block proposal from ${validatorPairName}"
config:
minTransactionCount: 240
minTransactionCount: 80 # For some reason the tx fuzz is working different than the old spammer, we need to check it
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Here is an issue for tracking this: #1440

Comment on lines +108 to +111
#💻 kurtosis.assertoor: @ Execute the assertoor network params in .github
kurtosis.assertoor: kurtosis.clean kurtosis.setup.lambdaconsensus
kurtosis run --enclave $(KURTOSIS_ENCLAVE) $(KURTOSIS_DIR) --args-file .github/config/assertoor/network-params.yml

Copy link
Collaborator Author

@rodrigo-o rodrigo-o Apr 22, 2025

Choose a reason for hiding this comment

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

Just a convenient way to run the assertoor step locally

@rodrigo-o rodrigo-o marked this pull request as ready for review April 22, 2025 14:45
@rodrigo-o rodrigo-o requested a review from a team as a code owner April 22, 2025 14:45
Copy link
Contributor

@LeanSerra LeanSerra left a comment

Choose a reason for hiding this comment

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

LGTM

@rodrigo-o rodrigo-o merged commit 1396bdd into main Apr 22, 2025
13 of 14 checks passed
@rodrigo-o rodrigo-o deleted the fix-kurtosis-and-assertoor branch April 22, 2025 15:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

3 participants