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

consensus node stop sync block. #841

Closed
ZhangTao1596 opened this issue Jun 18, 2019 · 8 comments · Fixed by #899
Closed

consensus node stop sync block. #841

ZhangTao1596 opened this issue Jun 18, 2019 · 8 comments · Fixed by #899
Labels
bug Used to tag confirmed bugs critical Issues (bugs) that need to be fixed ASAP

Comments

@ZhangTao1596
Copy link
Contributor

Consensus node stop sync data, unless restart.
image
After restart.
image

This usually happens when a cn ran for a long term.

@shargon shargon added the bug Used to tag confirmed bugs label Jun 18, 2019
@igormcoelho
Copy link
Contributor

NF=6 means it considers all other nodes as faulty...meaning its completely isolated from p2p.
This looks like a p2p issue, perhaps the same issue that eventually yields empty blocks... but this is the worst case, when its actually disconnected.
We need to investigate this.

@ZhangTao1596
Copy link
Contributor Author

I think maybe this cn far behind block height , it refused all normal consensus messages. So for it, it didn't receive any message from other cn. It thinks nf=6.
Why it stop sync from non-consensus nodes?

@vncoelho
Copy link
Member

PR #620 mentions this, it is related to network P2P problems as IGOR said.
Perhaps solving this will also solve the problem of nodes with empty mempools.

@vncoelho vncoelho added the critical Issues (bugs) that need to be fixed ASAP label Jun 20, 2019
@vncoelho
Copy link
Member

@shargon, I would prefer to remove the bug flag, because in general the consensus comes back to live again, if you wait.
In this sense, the manually reestart mentioned by @KickSeason is solved with the time.

However, it is still some logic that we need to improve on the P2P and also in the way tasksare being cached.

@yongjiema
Copy link
Contributor

I have the same issue even for non-consensus nodes.

@ZhangTao1596
Copy link
Contributor Author

ZhangTao1596 commented Jul 8, 2019

NGD is testing #862 and #865 .

@shargon
Copy link
Member

shargon commented Jul 9, 2019

Fix here: #904

@vncoelho
Copy link
Member

vncoelho commented Jul 9, 2019

image

We need that eveyone testing (@KickSeason) run on the last Master-2.x.
In addition, that Testnet and MainNet nodes, as soon as possible, updates to up-to-date version (which will be 2.9.3.

Since 2.9.2 we had important fixes on Priorities Ques of Akka Mailboxes and also Memory Leak on TaskManager.

I believe that these new fixes should enter in another version, let's say 2.9.4.

@lock9 lock9 added the bug Used to tag confirmed bugs label Jul 11, 2019
Thacryba pushed a commit to simplitech/neo that referenced this issue Feb 17, 2020
Thacryba pushed a commit to simplitech/neo that referenced this issue Feb 17, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Used to tag confirmed bugs critical Issues (bugs) that need to be fixed ASAP
Projects
None yet
6 participants