This repository has been archived by the owner on Nov 6, 2020. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
Authority round consensus engine #3426
Merged
Merged
Changes from 92 commits
Commits
Show all changes
96 commits
Select commit
Hold shift + click to select a range
c31eab5
add non renewing timer
3a68fab
new simple authority engine
747898d
add new engine to spec
fc3d01e
add tests, fixes, simplifications
965dde8
enable TestNet with custom spec
4e75686
separate block preparation methods
cadca64
Split internal sealing from work sealing, add cli option
c0201bd
replace cli with engine method, simplify
d6e5637
Merge branch 'master' into split-internal-seal
d98f69c
Merge remote-tracking branch 'parity/split-internal-seal' into auth-r…
4bfdeea
More docs about sealing types. Bypass work in external txs.
1e20716
Merge remote-tracking branch 'parity/split-internal-seal' into auth-r…
e41b6c4
split requires_reseal, add test and new test miner
8b9e7c0
Merge remote-tracking branch 'parity/split-internal-seal' into auth-r…
6760132
make test helpers not test, add some docs
6a33f8b
state constructor used only in tests
eee6be1
implement new predicate trait
da2f117
new test_net holding Client
0994681
Merge remote-tracking branch 'parity/master' into auth-round
bdba423
Merge remote-tracking branch 'parity/master'
fef9420
enable internal sealing based on author
2bd8226
add tests, keep track of engine sealing status
0880d4a
method to check if default address is_sealer
7f05021
Merge remote-tracking branch 'parity/should-seal' into auth-round
ddb6fec
add new test specs
bedbe6e
update test spec loading
28a088e
add client own_tx import through client
7eac946
remove unused original TestNet method
faa9c7f
add new TestNet transaction issuing
b2c0a9d
introduce ethkey dependency to generate txs
68fd862
simplify constructors
c6c951d
Merge remote-tracking branch 'parity/should-seal' into auth-round
7c82a10
fix typo
a44e2b0
Merge remote-tracking branch 'parity/should-seal' into auth-round
c482b8f
enable sealing only is authority
bb59c22
docs on authority spec
d3ec858
new constructor that takes AccountProvider
44fe864
new highway to miner, set_author
3419549
revamp dummy_client to take accounts
b144bd2
add tracing
b31e732
temporary is_sealer check disable
c6c45db
add devtools dependency
6af888f
disable TemporaryPath panic
fdcda41
add tracing
49b8e14
initial mocknet with chain notify
ce3e875
additional consensus test
551b5d7
Merge remote-tracking branch 'parity/master'
49fba0b
Merge branch 'master' into auth-round
7f177f2
revert dummy_client changes
d84f949
add proposer step to seal
44c4845
fix up step switching test
9d23915
more simulation methods
c57e3ce
sync check method
ec058cd
reseal on timeout
cf88641
more mocknet tests
5fac941
Merge remote-tracking branch 'parity/master'
0c69d4a
Merge remote-tracking branch 'parity/master' into auth-round
28cf91c
remove mocknet stuff, clean up debug
1c801c9
revert random path drop panic
7e1d714
block time in decimal
44f42ac
Merge remote-tracking branch 'parity/master' into auth-round-no-mocknet
303f922
steps based on unix epoch
aa05823
Merge remote-tracking branch 'parity/master' into auth-round-no-mocknet
4ee669b
Merge remote-tracking branch 'parity/master'
3ff26dd
Merge branch 'master' into auth-round-no-mocknet
63cc5c0
Merge remote-tracking branch 'parity/master'
7976f46
Merge branch 'master' into auth-round-no-mocknet
37ca1fa
Merge remote-tracking branch 'parity/master'
fc4664a
Merge branch 'master' into auth-round-no-mocknet
a33e2f2
fix deadlock after merge
3a6b56e
check double signing, tracing
da32531
ignore block timeout when sealing internally
26155f5
proper calculation of remaining step time
03c9760
Merge remote-tracking branch 'parity/master' into auth-round-no-mocknet
b2a3851
Merge remote-tracking branch 'auth/auth-round-no-mocknet' into auth-r…
00a3619
header methods
045a65c
Merge remote-tracking branch 'auth/auth-round-no-mocknet' into auth-r…
12125dc
Merge branch 'master' into auth-round
255ec0c
Merge branch 'master' into auth-round
9de910c
Merge branch 'master' into auth-round
0e5060d
update schedule
a8b688b
allow permanent unlock for internal sealing
49f2a72
Merge branch 'master' into auth-round
501f6d3
Merge branch 'master' into auth-round
b5c8918
fix merge
165e2be
modify proposer test
8a86c11
enable force sealing
8be8b3e
ensure no double proposals
5de7d50
add test for multi proposals
1e3ae9f
typos, header functions
06ebd5f
remove most panics
ae5ae76
Merge branch 'master' into auth-round
9d46401
revert to try!
071e66c
fix flaky test
fde6ff2
nicer extra info
8efaf08
typos and warning
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,42 @@ | ||
{ | ||
"name": "TestAuthorityRound", | ||
"engine": { | ||
"AuthorityRound": { | ||
"params": { | ||
"gasLimitBoundDivisor": "0x0400", | ||
"stepDuration": "1", | ||
"authorities" : [ | ||
"0x7d577a597b2742b498cb5cf0c26cdcd726d39e6e", | ||
"0x82a978b3f5962a5b0957d9ee9eef472ee55b42f1" | ||
] | ||
} | ||
} | ||
}, | ||
"params": { | ||
"accountStartNonce": "0x0", | ||
"maximumExtraDataSize": "0x20", | ||
"minGasLimit": "0x1388", | ||
"networkID" : "0x69" | ||
}, | ||
"genesis": { | ||
"seal": { | ||
"generic": { | ||
"fields": 1, | ||
"rlp": "0x11bbe8db4e347b4e8c937c1c8370e4b5ed33adb3db69cbdb7a38e1e50b1b82fa" | ||
} | ||
}, | ||
"difficulty": "0x20000", | ||
"author": "0x0000000000000000000000000000000000000000", | ||
"timestamp": "0x00", | ||
"parentHash": "0x0000000000000000000000000000000000000000000000000000000000000000", | ||
"extraData": "0x", | ||
"gasLimit": "0x2fefd8" | ||
}, | ||
"accounts": { | ||
"0000000000000000000000000000000000000001": { "balance": "1", "nonce": "1048576", "builtin": { "name": "ecrecover", "pricing": { "linear": { "base": 3000, "word": 0 } } } }, | ||
"0000000000000000000000000000000000000002": { "balance": "1", "nonce": "1048576", "builtin": { "name": "sha256", "pricing": { "linear": { "base": 60, "word": 12 } } } }, | ||
"0000000000000000000000000000000000000003": { "balance": "1", "nonce": "1048576", "builtin": { "name": "ripemd160", "pricing": { "linear": { "base": 600, "word": 120 } } } }, | ||
"0000000000000000000000000000000000000004": { "balance": "1", "nonce": "1048576", "builtin": { "name": "identity", "pricing": { "linear": { "base": 15, "word": 3 } } } }, | ||
"9cce34f7ab185c7aba1b7c8140d620b4bda941d6": { "balance": "1606938044258990275541962092341162602522202993782792835301376", "nonce": "1048576" } | ||
} | ||
} |
2 changes: 1 addition & 1 deletion
2
ethcore/res/test_authority.json → ethcore/res/basic_authority.json
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,5 +1,5 @@ | ||
{ | ||
"name": "TestAuthority", | ||
"name": "TestBasicAuthority", | ||
"engine": { | ||
"BasicAuthority": { | ||
"params": { | ||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
convention/unrelated:
with_X
as opposed tonew_with_X
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have seen it done above in existing methods and would not be nice to have it differently for a few, maybe a separate PR renaming all of them?