Skip to content
This repository has been archived by the owner on Jun 11, 2024. It is now read-only.

Network module should ignore peers for consensus while sycing to network #3786

Closed
ManuGowda opened this issue Jun 6, 2019 · 1 comment
Closed

Comments

@ManuGowda
Copy link
Contributor

Expected behavior

Network module should consider <= 100 peers with matching broadhash for consensus calculation.

Actual behavior

When a group of peers are syncing to network and not reached the network height, when a forging node takes these peers for consensus calculation it results in low consensus due to sycing and miss match braodhash, this results in delegate missing a block.

Steps to reproduce

  1. Deploy a network of 10 nodes
  2. Enable 101 delegates forging on 10 nodes
  3. After reaching certain height, Deploy 100 nodes, you can observe the delegates starts to miss forging blocks due to low consensus

Which version(s) does this affect? (Environment, OS, etc...)

2.0.0

@shuse2
Copy link
Collaborator

shuse2 commented Jun 6, 2019

Consensus is calculated in the chain module, and also consensus calculation does support this scenario.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants