Skip to content
This repository was archived by the owner on Nov 15, 2023. It is now read-only.

Disable Kademlia random walk when --reserved-nodes is passed #7999

Merged
2 commits merged into from
Feb 2, 2021

Conversation

tomaka
Copy link
Contributor

@tomaka tomaka commented Jan 28, 2021

Fix #6031

Disables automatically finding new peers when --reserved-nodes is passed.
Note that this behaviour is specific to the CLI. The network itself can still be configured to randomly walk the DHT even in reserved-only mode.

Verified

This commit was signed with the committer’s verified signature.
tomaka Pierre Krieger
@tomaka tomaka added A0-please_review Pull request needs code review. B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders. labels Jan 28, 2021
@tomaka tomaka requested review from romanb and mxinden January 28, 2021 11:09
Copy link
Contributor

@mxinden mxinden left a comment

Choose a reason for hiding this comment

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

Looks good to me.

I find the reserved_only flag misleading. Something along the lines of sync_with_reserved_only would be easier for me to grasp. I don't think this should be the concern of this pull request though.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
Co-authored-by: Roman Borschel <romanb@users.noreply.github.com>
@tomaka
Copy link
Contributor Author

tomaka commented Jan 28, 2021

Something along the lines of sync_with_reserved_only would be easier for me to grasp. I don't think this should be the concern of this pull request though.

That would be an annoying breaking change unfortunately, as I imagine that a lot of people use this option, but I guess we could introduce a second CLI option and deprecate the first one.

@tomaka
Copy link
Contributor Author

tomaka commented Feb 2, 2021

bot merge

@ghost
Copy link

ghost commented Feb 2, 2021

Trying merge.

@ghost ghost merged commit d5b0856 into paritytech:master Feb 2, 2021
@tomaka tomaka deleted the fix-6031 branch February 2, 2021 13:52
This pull request was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A0-please_review Pull request needs code review. B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Disable the Kademlia discovery when --reserved-only is passed
3 participants