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.

Sign up for GitHub

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

Jump to bottom

Confirm frontiers for bootstrap with multiple failures #2349

Merged
merged 44 commits into from
Oct 24, 2019
Merged

Confirm frontiers for bootstrap with multiple failures #2349

merged 44 commits into from
Oct 24, 2019

Conversation

SergiySW
Copy link
Contributor

No description provided.

@SergiySW
Network function to send confirm_req with vector of hashes+roots
afe3d47
@SergiySW
Add confirmed status to gap cache, separate inactive votes cache from…
9684ff7 
… gap_cache
@SergiySW
Check frontiers for bootstrap attempt with a lot of failed pulls
8bb5d20
@SergiySW
Confirm bootstrap frontiers if there are multiple pulls failures
7ed6ea2
@SergiySW
Merge remote-tracking branch 'upstream/master' into bootstrap/confirm…
9e19458 
…_frontiers
@SergiySW
Adjust attempt_restart_check
ac12344
@SergiySW
Adjust failed pulls limit
3a2b8cd
@SergiySW
Improve variables initialization
2ae8d5a
@SergiySW
Use restarted pulls limit
d6b796f
@SergiySW
Close clients sockets on attempt restart
fc0a329
@SergiySW
Clear idle peers list
67b3151
@SergiySW
Merge remote-tracking branch 'upstream/master' into bootstrap/confirm…
9df8069 
…_frontiers
@SergiySW
Add recet recent_pulls_head deque, fix confirm_req deserialization wi…
ca0f757 
…th empty root
@SergiySW
Save 500 ms in last loop
6a53943
@SergiySW
Update bootstrap.cpp
2bd6542
@SergiySW
Merge remote-tracking branch 'upstream/master' into bootstrap/confirm…
fb62086 
…_frontiers
@SergiySW
Adjust frontiers confirmation settings
2369eaf
@SergiySW
App confirmed_frontiers status to bootstrap_status RPC
7c199a5
@SergiySW
Merge remote-tracking branch 'upstream/master' into bootstrap/confirm…
f13c281 
…_frontiers
@SergiySW
Add "confirmed_frontiers" option to bootstrap RPC
d9be218
@SergiySW
Rename requeued_pulls
9e7e640
@SergiySW
Add basic blacklist for peers with unconfirmed frontiers
0920a1b
@SergiySW
Check blacklisted items
04f1b34
@SergiySW
Update frontiers_unconfirmed test
cf50d42
@SergiySW
Zero heads check
6d5044a
@SergiySW SergiySW added quality improvements This item indicates the need for or supplies changes that improve maintainability rpc nonbreaking change beta testing wanted labels Oct 17, 2019
@SergiySW SergiySW added this to the V20.0 milestone Oct 17, 2019
@SergiySW SergiySW requested a review from wezrule October 17, 2019 00:25
@zhyatt zhyatt requested a review from clemahieu October 18, 2019 16:16
@guilhermelawless guilhermelawless added the documentation This item indicates the need for or supplies updated or expanded documentation label Oct 18, 2019
@guilhermelawless
Copy link
Contributor

Added documentation label so the corresponding RPC changes are tracked.

@guilhermelawless guilhermelawless mentioned this pull request Oct 24, 2019
Closed
@zhyatt zhyatt removed the request for review from clemahieu October 24, 2019 13:52
@SergiySW SergiySW merged commit 2c914b8 into nanocurrency:master Oct 24, 2019
@zhyatt zhyatt added rpc Changes related to Remote Procedure Calls and removed rpc nonbreaking change labels Mar 17, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@wezrule wezrule wezrule approved these changes

@argakiig argakiig argakiig approved these changes

@guilhermelawless guilhermelawless guilhermelawless approved these changes

@cryptocode cryptocode cryptocode approved these changes

Assignees

@SergiySW SergiySW

Labels
beta testing wanted documentation This item indicates the need for or supplies updated or expanded documentation quality improvements This item indicates the need for or supplies changes that improve maintainability rpc Changes related to Remote Procedure Calls
Projects
None yet
Milestone
V20.0
Development

Successfully merging this pull request may close these issues.
6 participants
@SergiySW @guilhermelawless @wezrule @argakiig @cryptocode @zhyatt