-
Notifications
You must be signed in to change notification settings - Fork 291
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
Mainnet Release v2025.0.0 #4831
Merged
Conversation
This file contains 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
Co-authored-by: Soph <[email protected]>
* remove error logs if stream already exist * improve pickAvailableStream to continue looking for other available streams if any stream fails * add thread safe map to request manager in p2p stream layer * improve stream sync switching from long range to short range * improve context for doSync loop in stream sync * add error handling for context deadline exceed issue * fix request manager test * improve computeLongestHashChain and its whitelist * improve countHashMaxVote * introduce new configs to StagedStreamSync struct * initialize new configs in stream sync and add them to logs * pass node config to stream sync downloader, increase stream sync ticker time, init sync if not epoch chain * add two new metrics to stream sync * pass node configs to service and downloaders to have access to node configurations in stream sync * using BeaconValidator and BecaonShard instead of BeaconNode, using EpochChain config to detect epochcian in stages, add acceptPartially to GetBlockHash to be able to get a part of hashes * pass nodeconfig to downloaders * remove BeaconNode from protocol, fix shortrange tests * improve lock in request manager test * update Failures option names, remove beaconNode from stream protocol * fix shard id detection * improve getBlockByMaxVote * add one more checking for prestaking * improve epoch and beacon node detection for stream sync * fix passing roles to sync protocol * remove nil error from logs * fix switch between long range and short range in stream sync * fix stream sync slowness * improve short range helpers concurrency * stop readMsgLoop once stream is closed * stop discovery loop once stream manager is closing * fix peer id in stream manager logs * separate stream set and stream manager
…treams if any stream fails
…4772) Enhance Concurrency and Lock Management in Staged Stream Sync Module
…treams if any stream fails
Add Support for Trusted Nodes in P2P Host
…nsus Resets (#4787) * improve consensus finalCommit * not trigger BlocksSynchronized if node is fully synced * stream sync unsets consensus sync flags in long range * dns sync triggers BlocksSynchronized only if there is some blocks added * not send BlocksSynchronized signal for epoch chain
* Removed `UpdateConsensusInformation` * Moved to tests.
becase we gave |
…branch with exact same name does not exist (#4832)
GheisMohammadi
approved these changes
Jan 20, 2025
Frozen
reviewed
Jan 22, 2025
Frozen
reviewed
Jan 22, 2025
Deadlock major, if we are not using streamsync it's fine |
major, if we are not using streamsync it's fine |
@Frozen thanks yeah we are not using streamsync in mainnet nor testnet right now. Appreciate your approval. |
Frozen
reviewed
Jan 22, 2025
* update harmony and bootnode version * add support for commitAt
Frozen
reviewed
Jan 22, 2025
Frozen
approved these changes
Jan 22, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
#4762 - Improve P2P Stream Handling & Stream Sync Enhancements
#4772 - Enhance Concurrency and Lock Management in Staged Stream Sync Module
#4777 - Add Support for Trusted Nodes in P2P Host
#4786 - Enhance Final Commit Function with Improved Error Handling
#4791 - Improve logging for consensus leader setup
#4792 - [fix] - CI - update macos from 12 to 13, sync the CI steps for macos
#4787 - Optimize Syncing-Consensus Communication to Prevent Unnecessary Consensus Resets
#4790 - Moved the
UpdateConsensusInformation
method to the test suite#4801 - Add Logging to ReadySignal Method for New Block Proposals
#4785 - Additional logs for consensus
#4793 - localnet with multi bls
#4798 - Enhance Leader Rotation Logic to Address Edge Cases in Leader Selection
#4805 - Add Check for Faucet Key Existence Before Building External Validator
#4807 - Add Localnet Config to Adjust Blocks per Epoch
#4803 - Minor: Moved method
WaitForConsensusReadyV2
fromConsensus
struct.#4799 - Improve Consensus logic for Multi-BLS Validators with quorum
#4811 - Fix Multi-BLS Key Validator Leader Rotation to Prevent Multiple Block Proposals
#4804 - Prevents BlocksSynchronized being called before finalCommit
#4815 - Add Resource Manager to P2P Host
#4818 - Add New Command to Makefile for Enhanced Localnet Debugging
#4816 - Fix for excess propose signals.
#4819 - multi bls multi ext node
#4827 - Add Resource Manager to P2P Configs When Enabled
#4832 - refactor: CI - fallback to the master branch in harmony-test if same branch name doesn't exitst
#4767 - fix: fix slice init length
#4759 - Effective gas price
#4834 - update harmony and bootnode version