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(op-node): add opBNB bootnodes #89

Merged
merged 4 commits into from
Dec 12, 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 bootnodes by chainId including testnet and mainnet. If p2p.bootnodes is configured, the configured bootnodes will be used. The configured p2p.bootnodes have the highest priority.

Changes

Add opBNB testnet and mainnet bootnodes in code.

@redhdx redhdx requested a review from krish-nr November 17, 2023 09:44
op-node/p2p/config.go Outdated Show resolved Hide resolved
redhdx and others added 2 commits November 21, 2023 14:04
Co-authored-by: welkin22 <136572398+welkin22@users.noreply.github.com>
Copy link
Contributor

@welkin22 welkin22 left a comment

Choose a reason for hiding this comment

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

LGTM

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

Copy link
Contributor

This PR is stale because it has been open 14 days with no activity. Remove stale label or comment or this will be closed in 5 days.

@github-actions github-actions bot added the Stale label Dec 12, 2023
@owen-reorg owen-reorg merged commit 429e20b into bnb-chain:develop Dec 12, 2023
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants