Skip to content

fix: kurtosis in electra support branch #1421

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 15 commits into from
Apr 9, 2025

Conversation

rodrigo-o
Copy link
Collaborator

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

Motivation

Kurtosis was faling to start

Description

This PR added a new branch to our ethereum-package fork, lecc-integration-electra which syncs with upstream/main.

It updates the network_params to match the new changes and fix the start of the node in the CI

Closes #1418 #1412 & #1422

Verified

This commit was signed with the committer’s verified signature.
rodrigo-o Rodrigo Oliveri

Verified

This commit was signed with the committer’s verified signature.
rodrigo-o Rodrigo Oliveri

Verified

This commit was signed with the committer’s verified signature.
rodrigo-o Rodrigo Oliveri

Verified

This commit was signed with the committer’s verified signature.
rodrigo-o Rodrigo Oliveri

Verified

This commit was signed with the committer’s verified signature.
rodrigo-o Rodrigo Oliveri
@rodrigo-o rodrigo-o requested a review from a team as a code owner April 8, 2025 18:41
@rodrigo-o rodrigo-o changed the base branch from main to electra-support April 8, 2025 18:41
@rodrigo-o rodrigo-o changed the title fix:kurtosis in electra support branch fix: kurtosis in electra support branch Apr 8, 2025
@rodrigo-o rodrigo-o marked this pull request as draft April 8, 2025 18:43

Verified

This commit was signed with the committer’s verified signature.
rodrigo-o Rodrigo Oliveri

Verified

This commit was signed with the committer’s verified signature.
rodrigo-o Rodrigo Oliveri

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.

Verified

This commit was signed with the committer’s verified signature.
rodrigo-o Rodrigo Oliveri
…angea

Verified

This commit was signed with the committer’s verified signature.
rodrigo-o Rodrigo Oliveri

Verified

This commit was signed with the committer’s verified signature.
rodrigo-o Rodrigo Oliveri
@rodrigo-o rodrigo-o marked this pull request as ready for review April 9, 2025 19:05
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.

Nice!

Verified

This commit was signed with the committer’s verified signature.
rodrigo-o Rodrigo Oliveri

Verified

This commit was signed with the committer’s verified signature.
rodrigo-o Rodrigo Oliveri

Verified

This commit was signed with the committer’s verified signature.
rodrigo-o Rodrigo Oliveri

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
@rodrigo-o
Copy link
Collaborator Author

rodrigo-o commented Apr 9, 2025

Now Kurtosis starts and assertoor fail in the initial test, which was expected and tracked in #1413

2025-04-09 22:02:05  Test Status:
  cl-stability-check               failure 	Check Execution Stability

Failed Test Task Status:
 1   	complete	failure	check_clients_are_healthy                          	Check if all clients are ready
 2   	pending 	none   	run_tasks_concurrent                               	Check if all EL & CL clients are synced and the tx spammer is working
 3   	pending 	none   	run_task_matrix                                    	Check block proposals from all client pairs
 4   	pending 	none   	run_tasks_concurrent                               	Check chain stability (reorgs and forks)

@rodrigo-o rodrigo-o merged commit 439b652 into electra-support Apr 9, 2025
10 of 14 checks passed
@rodrigo-o rodrigo-o deleted the electra-support-fix-kurtosis branch April 9, 2025 22:05
rodrigo-o added a commit that referenced this pull request Apr 25, 2025

Verified

This commit was signed with the committer’s verified signature.
rodrigo-o Rodrigo Oliveri
rodrigo-o added a commit that referenced this pull request Apr 25, 2025

Verified

This commit was signed with the committer’s verified signature.
rodrigo-o Rodrigo Oliveri
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.

SSZ beacon state decoding error on node startup
2 participants