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

feature: add opBNB bootnodes #32

Merged
merged 1 commit into from
Nov 23, 2023

Conversation

redhdx
Copy link
Contributor

@redhdx redhdx commented Nov 17, 2023

Description

Add opBNB testnet and mainnet bootnodes in code.

Rationale

Add opBNB testnet and mainnet bootnodes in code can facilitate the startup of new nodes.

Example

The startup node will use the opBNB mainnet's bootnodes by default. If the --networkid is configured and it is testnet, the testnet bootnodes will be used. If --bootnodes is configured, the configured bootnodes will be used. The configured --bootnodes have the highest priority.

Changes

Add opBNB testnet and mainnet bootnodes in code.

2129zxl

This comment was marked as abuse.

Copy link
Contributor

@krish-nr krish-nr left a comment

Choose a reason for hiding this comment

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

LGTM

@owen-reorg owen-reorg merged commit 62a7d78 into bnb-chain:develop Nov 23, 2023
2 checks passed
sunny2022da pushed a commit to sunny2022da/op-geth that referenced this pull request Aug 14, 2024
Co-authored-by: andyzhang2023 <[email protected]>
sunny2022da pushed a commit to sunny2022da/op-geth that referenced this pull request Sep 25, 2024
Co-authored-by: andyzhang2023 <[email protected]>
sunny2022da pushed a commit to sunny2022da/op-geth that referenced this pull request Oct 11, 2024
Co-authored-by: andyzhang2023 <[email protected]>
welkin22 pushed a commit that referenced this pull request Oct 22, 2024
Co-authored-by: andyzhang2023 <[email protected]>
sunny2022da pushed a commit to sunny2022da/op-geth that referenced this pull request Nov 14, 2024
Co-authored-by: andyzhang2023 <[email protected]>
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.

6 participants