You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository has been archived by the owner on Jun 11, 2024. It is now read-only.
block should get certified if BLS keys are registered during the init phase even if useInvalidBLSKeys are enabled.
Actual behavior
block doesn't get certified forever.
Reason is
When useInvalidBLSKeys are used, BLS keys will be registered during the init rounds (~309 blocks by default)
at 309, engine will store new BFT properties with valid BLS keys as height 310. (this is because the new validators are active from height 310)
aggregateCommit.height cannot be larger than heightNextBFTParameters - 1. In this case, heightNextBFTParameters is 310, so at maximum 309 must be certified. However, BFTParameters for height 309 will not be created because there is no valid BLSkey.
Steps to reproduce
Use useINvalidBLSKeys in the PoS module and register keys during the init rounds.
after around 1000 blocks, there should be aggregateCommits.heght at 0 where it should certify earlier block
Which version(s) does this affect? (Environment, OS, etc...)
Expected behavior
block should get certified if BLS keys are registered during the init phase even if
useInvalidBLSKeys
are enabled.Actual behavior
block doesn't get certified forever.
Reason is
useInvalidBLSKeys
are used, BLS keys will be registered during the init rounds (~309 blocks by default)heightNextBFTParameters - 1
. In this case,heightNextBFTParameters
is310
, so at maximum309
must be certified. However, BFTParameters for height309
will not be created because there is no valid BLSkey.Steps to reproduce
Use
useINvalidBLSKeys
in the PoS module and register keys during the init rounds.after around 1000 blocks, there should be aggregateCommits.heght at
0
where it should certify earlier blockWhich version(s) does this affect? (Environment, OS, etc...)
6.0.0-alpha.21
Additional Information
LIP is updated with LiskArchive/lips#334
The text was updated successfully, but these errors were encountered: