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

feat(l1): add Mekong presets #1862

Merged
merged 5 commits into from
Jan 31, 2025
Merged

feat(l1): add Mekong presets #1862

merged 5 commits into from
Jan 31, 2025

Conversation

fmoletta
Copy link
Contributor

Motivation
This testnet is much newer than holesky, so It should allow us to test snap sync much faster

Description

Closes #issue_number

Copy link

github-actions bot commented Jan 31, 2025

| File                                                   | Lines | Diff |
+--------------------------------------------------------+-------+------+
| /home/runner/work/ethrex/ethrex/cmd/ethrex/ethrex.rs   | 390   | +5   |
+--------------------------------------------------------+-------+------+
| /home/runner/work/ethrex/ethrex/cmd/ethrex/networks.rs | 22    | +6   |
+--------------------------------------------------------+-------+------+

Total lines added: +11
Total lines removed: 0
Total lines changed: 11

@mpaulucci mpaulucci changed the title feat(l1): Add Mekong presets feat(l1): add Mekong presets Jan 31, 2025
@@ -114,6 +114,13 @@ async fn main() {
bootnodes = networks::SEPOLIA_BOOTNODES.to_vec();
}

if network == "mekong" {
warn!("Using mekong presets, bootnodes field will be ignored");
Copy link
Collaborator

Choose a reason for hiding this comment

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

instead of ignoring the bootnodes field, shouldn't we append it? Thoughts?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Good idea, we should add that logic to the other networks too

Copy link
Contributor Author

Choose a reason for hiding this comment

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


pub const HOLESKY_BOOTNODES_PATH: &str = "cmd/ethrex/networks/holesky/bootnodes.json";
pub const SEPOLIA_BOOTNODES_PATH: &str = "cmd/ethrex/networks/sepolia/bootnodes.json";
pub const MEKONG_BOOTNODES_PATH: &str = "cmd/ethrex/networks/mekong/bootnodes.json";
Copy link
Collaborator

Choose a reason for hiding this comment

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

where are these files?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I forgot to push them, should be up now

@fmoletta fmoletta marked this pull request as ready for review January 31, 2025 19:20
@fmoletta fmoletta requested a review from a team as a code owner January 31, 2025 19:20
@fmoletta fmoletta added this pull request to the merge queue Jan 31, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jan 31, 2025
@fmoletta fmoletta enabled auto-merge January 31, 2025 19:30
@fmoletta fmoletta added this pull request to the merge queue Jan 31, 2025
Merged via the queue into main with commit 3892796 Jan 31, 2025
19 checks passed
@fmoletta fmoletta deleted the mekong branch January 31, 2025 20:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants