Skip to content
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

v7.0.0-beta.1 (Holesky Rescue) #7041

Draft
wants to merge 17 commits into
base: unstable
Choose a base branch
from
Draft

v7.0.0-beta.1 (Holesky Rescue) #7041

wants to merge 17 commits into from

Conversation

paulhauner
Copy link
Member

@paulhauner paulhauner commented Feb 26, 2025

Issue Addressed

Related to #7040

Proposed Changes

Various hacks and tweaks to ensure LH syncs a valid Holesky chain.

Most notably, we explicitly reject the invalid payload that was justified (0x2db899881ed8546476d0b92c6aa9110bea9a4cd0dbeb5519eb0ea69575f1f359).

Additional Info

  • Use this branch if you are running a node on Holesky

Copy link

mergify bot commented Feb 26, 2025

⚠️ The sha of the head commit of this PR conflicts with #7035. Mergify cannot evaluate rules on this PR. ⚠️

@@ -325,7 +325,7 @@ impl<E: EthSpec> ProductionValidatorClient<E> {
get_validator_block: slot_duration / HTTP_GET_VALIDATOR_BLOCK_TIMEOUT_QUOTIENT,
}
} else {
Timeouts::set_all(slot_duration)
Timeouts::set_all(25 * slot_duration)
Copy link
Member

Choose a reason for hiding this comment

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

@macladson could you make a clean PR for a CLI flag accomplishing the same effect against the release-v7.0.0 branch? 🙏

mergify bot pushed a commit that referenced this pull request Feb 26, 2025
Cleaned up and isolated version of the `--disable-attesting` flag for the VC, from the `holesky-rescue` branch:

- #7041

I figured we don't need the `--disable-attesting` flag on the BN for now, and it was a much more invasive impl.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants