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

iip 6: exclude delegate nodes with lower than expected version #10

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

CoderZhi
Copy link
Contributor

No description provided.

@@ -0,0 +1,29 @@
```
IIP: 6
Title: Exclude delegate nodes with lower-than-expected chain version
Copy link
Member

Choose a reason for hiding this comment

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

How about "Exclude Nodes With Unexpected Binary Version from Consensus Delegates "

The problem can be concurred by enforcing all delegate nodes to be up-to-date, however, the communication cost could be very high for a decentralized system like us. To prevent similar problems, we are proposing a new delegate election strategy to exclude delegate nodes who fail to adopt a mandatory upgrade. With this new strategy, these delegate nodes will be replaced by qualified backup delegate nodes until they adopt all mandatory upgrades.

## Specification
A system level contract will be deployed onto IoTeX blockchain to register the chain versions of all delegate nodes. When starting a blockchain node, a version checking will be performed to update the version registered in the contract if necessary. In poll protocol, a delegate node will be filtered if its chain version registered in the contract is lower than expected, which means it has not adopted the latest version of mandatory upgrade.
Copy link
Member

Choose a reason for hiding this comment

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

who sets the expected version in this contract?

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.

2 participants