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

Deprecate --Xp2p-peer-lower-bound #6501

Merged
merged 11 commits into from Feb 2, 2024

Conversation

macfarla
Copy link
Contributor

Deprecate CLI option --Xp2p-peer-lower-bound and remove usage

Fixes #6376

Signed-off-by: Sally MacFarlane <macfarla.github@gmail.com>
Copy link

github-actions bot commented Jan 31, 2024

  • I thought about documentation and added the doc-change-required label to this PR if updates are required.
  • I thought about the changelog and included a changelog update if required.
  • If my PR includes database changes (e.g. KeyValueSegmentIdentifier) I have thought about compatibility and performed forwards and backwards compatibility tests

Signed-off-by: Sally MacFarlane <macfarla.github@gmail.com>
@macfarla macfarla added the doc-change-required Indicates an issue or PR that requires doc to be updated label Jan 31, 2024
@macfarla
Copy link
Contributor Author

although it's early access, this option is mentioned in the docs so that needs to be updated eg https://besu.hyperledger.org/public-networks/how-to/connect/manage-peers#limit-peers

macfarla and others added 5 commits January 31, 2024 13:57
Signed-off-by: Sally MacFarlane <macfarla.github@gmail.com>
Signed-off-by: Sally MacFarlane <macfarla.github@gmail.com>
Signed-off-by: Sally MacFarlane <macfarla.github@gmail.com>
Copy link
Contributor

@pinges pinges left a comment

Choose a reason for hiding this comment

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

LGTM

@macfarla macfarla enabled auto-merge (squash) February 1, 2024 07:24
Signed-off-by: Sally MacFarlane <macfarla.github@gmail.com>
@macfarla macfarla merged commit 2077bb7 into hyperledger:main Feb 2, 2024
10 checks passed
@bgravenorst bgravenorst removed the doc-change-required Indicates an issue or PR that requires doc to be updated label Feb 6, 2024
@macfarla macfarla deleted the remove-p2p-lower-bound branch February 27, 2024 05:22
@arnili
Copy link

arnili commented Mar 16, 2024

I'm using this option per the documentation at https://besu.hyperledger.org/24.1.0/public-networks/how-to/connect/manage-peers#limit-peers to balance my incoming/outgoing connections to 50:50 after significantly reducing my number of peers.

If this option is removed most/all my connections will be outgoing and I will be a net-burden to the network.

@macfarla
Copy link
Contributor Author

Hi @arnili - I recommend you look at https://besu.hyperledger.org/public-networks/how-to/connect/manage-peers#limit-remote-connections and also static nodes configuration might be a good option for you if you want to maintain connections to specific nodes. Additionally, whether connections are initiated inbound or outbound makes no difference, once the nodes are connected as peers they are treated equally. If you have questions still - reach out in discord!

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.

delete -Xp2p-peer-lower-bound
4 participants