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

Add RadiumBlock bootnode to Moonbeam spec #2996

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

radiumb
Copy link

@radiumb radiumb commented Oct 8, 2024

What does it do?

Add RadiumBlock bootnode to Moonbeam spec

Is there something left for follow-up PRs?

No

What alternative implementations were considered?

None

Are there relevant PRs or issues in other repositories (Substrate, Polkadot, Frontier, Cumulus)?

None

What value does it bring to the blockchain users?

Have another bootnode to discover peers

@pLabarta
Copy link
Contributor

Node is able to sync with both endpoints ✔️

@pLabarta pLabarta added B5-clientnoteworthy Changes should be mentioned in any downstream projects' release notes D3-trivial PR contains trivial changes in a runtime directory that do not require an audit not-breaking Does not need to be mentioned in breaking changes D2-notlive PR doesn't change runtime code (so can't be audited) and removed D3-trivial PR contains trivial changes in a runtime directory that do not require an audit labels Oct 14, 2024
@noandrea noandrea added the A8-mergeoncegreen Pull request is reviewed well. label Nov 15, 2024
@RomarQ
Copy link
Contributor

RomarQ commented Nov 19, 2024

The provided bootnodes seem to be offline.

Tested with the following command:

./moonbeam --chain=moonbeam --reserved-only --reserved-nodes /dns/moonbeam-bootnode.radiumblock.com/tcp/30333/p2p/12D3KooWMrR6TV7q1gQxMJ6B9jfU6V8tgxkzUZExYK9jfyZLa6U7 /dns/moonbeam-bootnode.radiumblock.com/tcp/30336/wss/p2p/12D3KooWMrR6TV7q1gQxMJ6B9jfU6V8tgxkzUZExYK9jfyZLa6U7

@radiumb
Copy link
Author

radiumb commented Nov 20, 2024

We have fixed the issue with the bootnode and tested it.

@RomarQ RomarQ removed the A8-mergeoncegreen Pull request is reviewed well. label Nov 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
B5-clientnoteworthy Changes should be mentioned in any downstream projects' release notes D2-notlive PR doesn't change runtime code (so can't be audited) not-breaking Does not need to be mentioned in breaking changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants