Łukasz Magiera
|
d3e5092b43
|
Merge pull request #3611 from filecoin-project/fix/tscache-nil-best
chain events: if cache best() is nil, return chain head
|
2020-09-07 18:05:34 +02:00 |
|
vyzo
|
d9faff19a5
|
fix tests
|
2020-09-07 15:55:53 +03:00 |
|
vyzo
|
4f0ab4f226
|
remove notifee after subscribing
|
2020-09-07 15:55:53 +03:00 |
|
Dirk McCormick
|
76a1b3286b
|
fix: if cache best is nil, return chain head
|
2020-09-07 14:43:06 +02:00 |
|
Alan Shaw
|
9d9d2a2a98
|
fix: second argument error message
Second argument should be a pointer, not runtime.
|
2020-09-07 12:29:40 +01:00 |
|
Raúl Kripalani
|
798061506e
|
decommission chain validation.
|
2020-09-07 11:34:36 +01:00 |
|
Travis Person
|
f7b52d16f5
|
metrics: add expected height metric
|
2020-09-07 02:57:12 +00:00 |
|
vyzo
|
e24a29b146
|
fix tests that touch the mpool; mock block must have a ParentBaseFee
|
2020-09-06 11:55:43 +03:00 |
|
vyzo
|
ffb2640736
|
use faster lookup for base fee
|
2020-09-06 11:55:43 +03:00 |
|
vyzo
|
f9492691a6
|
don't check baseFee lower bound for local messages
|
2020-09-06 11:55:43 +03:00 |
|
vyzo
|
1c0e6d76f0
|
add defensive check for minimum GasFeeCap for inclusion within the next 20 blocks
|
2020-09-06 11:55:43 +03:00 |
|
Łukasz Magiera
|
6bdd433570
|
Merge pull request #3538 from filecoin-project/steb/safe-exports
Robustify state manager against holes in actor method numbers
|
2020-09-05 21:58:03 +02:00 |
|
Łukasz Magiera
|
f58cbc2594
|
Merge pull request #3557 from filecoin-project/fix/mpool-repub-negative
republish messages even if the chains have negative performance
|
2020-09-05 21:48:39 +02:00 |
|
Łukasz Magiera
|
1e190e462a
|
Merge pull request #3562 from filecoin-project/fix/mpool-log-spam
reduce mpool add failure log spam
|
2020-09-05 21:47:29 +02:00 |
|
vyzo
|
5037282b98
|
add test for priority selection with negative chains
|
2020-09-05 22:15:18 +03:00 |
|
vyzo
|
0b3685e529
|
fix selection bug; priority messages were not included if other's chains were negative
|
2020-09-05 21:30:02 +03:00 |
|
vyzo
|
275f071b12
|
reduce mpool add failure log spam
|
2020-09-05 00:28:13 +03:00 |
|
vyzo
|
ad8d550d73
|
remove unused ts argument from Trim, replace it with allowNegative specifier
|
2020-09-04 22:32:30 +03:00 |
|
Steven Allen
|
fceeaf42ca
|
Robustify state manager against holes in actor method numbers
Also, don't simply assume that the field order matches the method numbers in
`builtin.Method*` structs.
|
2020-09-04 12:26:55 -07:00 |
|
Steven Allen
|
30fdff17b5
|
Skip nil exports
The actors may skip previously used but no longer defined method numbers.
|
2020-09-04 12:26:55 -07:00 |
|
vyzo
|
6d53862142
|
republish messages even if the chains have negative performance
|
2020-09-04 22:16:10 +03:00 |
|
Łukasz Magiera
|
01449e486c
|
Merge pull request #3528 from filecoin-project/fix/sele-gas-reward
Fix calculation of GasReward in messagepool
|
2020-09-03 20:41:28 +02:00 |
|
Jakub Sztandera
|
8111b227c4
|
Appease the linter
Signed-off-by: Jakub Sztandera <kubuxu@protocol.ai>
|
2020-09-03 20:19:01 +02:00 |
|
Jakub Sztandera
|
8bb471bccd
|
Add getGasPerf test
Signed-off-by: Jakub Sztandera <kubuxu@protocol.ai>
|
2020-09-03 20:19:01 +02:00 |
|
Jakub Sztandera
|
5074b526d0
|
Fix calculation of GasReward in messagepool
Signed-off-by: Jakub Sztandera <kubuxu@protocol.ai>
|
2020-09-03 19:36:03 +02:00 |
|
vyzo
|
d20b6adfd2
|
refuse to add duplicates in the mpool
|
2020-09-03 15:28:44 +03:00 |
|
vyzo
|
a4c650316a
|
fix test
|
2020-09-03 15:28:44 +03:00 |
|
vyzo
|
28f57667f0
|
cap MaxNonceGap to 4, add delay between batch messages during republish
|
2020-09-03 15:28:44 +03:00 |
|
vyzo
|
f53d2e3a46
|
cap MaxNonceGap to 16
|
2020-09-03 15:28:44 +03:00 |
|
vyzo
|
7b76aa2078
|
warn when adding a nonce-gapped message to the mpool
|
2020-09-03 15:28:44 +03:00 |
|
vyzo
|
d76a3b87c5
|
ignore messages with large nonce gaps
|
2020-09-03 15:28:44 +03:00 |
|
vyzo
|
da6d384300
|
fix tests
|
2020-09-03 15:28:44 +03:00 |
|
vyzo
|
4c489d18a2
|
track expected nonce in mpool, refuse messages with large gaps
|
2020-09-03 15:28:44 +03:00 |
|
Lucas Molas
|
ca8f5861fe
|
Update chain/blocksync/server.go
Co-authored-by: Łukasz Magiera <magik6k@users.noreply.github.com>
|
2020-09-02 17:07:39 -03:00 |
|
whyrusleeping
|
9a5542100a
|
load fewer messages from disk when serving blocksync requests
|
2020-09-02 11:43:05 -07:00 |
|
whyrusleeping
|
2b16e69e90
|
allow snapshot importing
|
2020-09-02 17:15:10 +02:00 |
|
whyrusleeping
|
09ebd1a557
|
allow exporting a number of recent chain state trees
|
2020-09-02 17:15:10 +02:00 |
|
austinabell
|
ba881eb4cf
|
Remove height from chain rand
|
2020-09-01 15:48:16 -04:00 |
|
Aayush Rajasekaran
|
93814cc85a
|
Merge pull request #3392 from filecoin-project/fix/mpool-priority-selection
Tweak priority selection to not select negative perfoming chains
|
2020-08-31 19:22:12 -04:00 |
|
刘林欣
|
0926f95979
|
remove the unnecessary err check
|
2020-08-31 15:11:45 +08:00 |
|
vyzo
|
648130eec5
|
don't include negative performing chains in priority message selection
|
2020-08-29 15:55:19 +03:00 |
|
Łukasz Magiera
|
953effcc43
|
Merge pull request #3350 from filecoin-project/fix/some-fsm-issues
Fix some fsm issues
|
2020-08-29 02:16:59 +02:00 |
|
Łukasz Magiera
|
a62e44c4e5
|
Fix lint
|
2020-08-29 02:13:38 +02:00 |
|
Łukasz Magiera
|
84a632787c
|
Merge pull request #3313 from filecoin-project/feat/mpool-balance-tracking
Message Pool Rudimentary Spam Protection Measures
|
2020-08-28 22:35:30 +02:00 |
|
vyzo
|
806f7dd15f
|
windowSize should not exceed blocksync.MaxRequestLength
|
2020-08-28 23:18:06 +03:00 |
|
vyzo
|
63a60cdd75
|
be more conservative in increasing the window size, check that we actually had more messages
|
2020-08-28 23:09:48 +03:00 |
|
vyzo
|
3935fd77ac
|
remember syncer windowSize
|
2020-08-28 22:52:40 +03:00 |
|
vyzo
|
4e7fd5b6d7
|
sync: increase window size when successful fetching messages
|
2020-08-28 22:03:06 +03:00 |
|
vyzo
|
1f660c9e04
|
sync: half window size and retry when fetching messages fails
|
2020-08-28 18:45:20 +03:00 |
|
Łukasz Magiera
|
20ffce5c0d
|
Merge remote-tracking branch 'origin/master' into fix/some-fsm-issues
|
2020-08-28 16:47:25 +02:00 |
|
lanzafame
|
64768f093d
|
rename local recordFailure func
|
2020-08-28 17:15:11 +10:00 |
|
lanzafame
|
a24db6c584
|
refactor block validate recordFailure func
|
2020-08-28 17:11:59 +10:00 |
|
lanzafame
|
050a0ec5e4
|
further reduce code duplication
|
2020-08-28 17:01:45 +10:00 |
|
lanzafame
|
0254e4b365
|
reduce code duplication
|
2020-08-28 16:53:59 +10:00 |
|
lanzafame
|
094baf8aa9
|
distinguish local message validation failures from remote
|
2020-08-28 16:11:24 +10:00 |
|
Łukasz Magiera
|
86c936759b
|
Merge pull request #3357 from filecoin-project/fix/block-validator-power
ignore, don't reject block if we can't determine power
|
2020-08-28 00:24:53 +02:00 |
|
austinabell
|
39ef3a701b
|
Remove unsigned message pointer from Runtime
|
2020-08-27 17:34:45 -04:00 |
|
vyzo
|
708a8b6543
|
flag to allow add to skip limit check
essential for correctness in the revert case
|
2020-08-28 00:04:21 +03:00 |
|
Łukasz Magiera
|
6c7874c883
|
Merge remote-tracking branch 'origin/master' into fix/some-fsm-issues
|
2020-08-27 21:37:08 +02:00 |
|
vyzo
|
6c3d8ec1de
|
ignore, don't reject block if we can't determine power
|
2020-08-27 16:53:53 +03:00 |
|
Łukasz Magiera
|
df635579c4
|
storagefsm: Handle sectors with expired deals better
|
2020-08-27 13:51:38 +02:00 |
|
whyrusleeping
|
a33784c434
|
print block 'age' when validating blocks during sync
|
2020-08-26 14:48:53 -07:00 |
|
vyzo
|
038e83b647
|
log error if we trip over too many pending messages for actor
|
2020-08-26 22:55:39 +03:00 |
|
vyzo
|
82ef0529d8
|
named error for too many pending messages
|
2020-08-26 22:48:03 +03:00 |
|
vyzo
|
6abfbbddd5
|
fix tests
|
2020-08-26 22:44:47 +03:00 |
|
vyzo
|
ad889a7240
|
rate limit messages per actor by enforcing maximum pending message limit
|
2020-08-26 22:40:10 +03:00 |
|
vyzo
|
d3e95d6b77
|
cast min base fee to uint64
|
2020-08-26 21:37:32 +03:00 |
|
vyzo
|
8db262c941
|
check gas fee cap for minimum value
|
2020-08-26 21:23:28 +03:00 |
|
vyzo
|
793eda986b
|
comment out Value inclusion for required funds tracking
we are not sure whether we want it yet.
|
2020-08-26 21:10:46 +03:00 |
|
vyzo
|
8848c549a0
|
move msg serialization outside of lock, consistently use curTs
|
2020-08-26 19:42:56 +03:00 |
|
vyzo
|
4919a005d3
|
always verify message min gas
|
2020-08-26 19:37:27 +03:00 |
|
vyzo
|
7a7066896e
|
better code for getRequiredFunds
|
2020-08-26 15:37:42 +03:00 |
|
vyzo
|
a2c0c10d2a
|
correctly account for message replacement in required funds tracking
|
2020-08-26 15:27:09 +03:00 |
|
vyzo
|
c473d3c682
|
rename ErrValidationFailure to ErrSoftValidationFailure
to make it clear to readers that this failure is soft; we might be out of sync.
|
2020-08-26 15:16:04 +03:00 |
|
vyzo
|
87e6c0973a
|
include message Value only in soft validation failures
|
2020-08-26 15:13:50 +03:00 |
|
vyzo
|
78876947a4
|
ErrBroadcastAnyway is now ErrValidationFailure
|
2020-08-26 15:03:13 +03:00 |
|
vyzo
|
b59f58435c
|
fix tests
|
2020-08-26 14:58:32 +03:00 |
|
vyzo
|
4adb83eaae
|
check aggregate pending message required funds against balance
|
2020-08-26 14:58:26 +03:00 |
|
vyzo
|
4ac9828b0c
|
track required funds in message set
|
2020-08-26 14:38:26 +03:00 |
|
vyzo
|
77f6e0d4f8
|
refactor balance check, apply it in PushWithNonce
|
2020-08-26 14:23:34 +03:00 |
|
vyzo
|
835dbfe001
|
check message before adding in PushWithNonce
|
2020-08-26 14:13:46 +03:00 |
|
vyzo
|
da9630c561
|
remove duplicate check
gas limit is checked against block gas limit in ValidForBlockInclusion
|
2020-08-26 14:02:34 +03:00 |
|
Whyrusleeping
|
14cda27e9f
|
Merge pull request #3294 from filecoin-project/feat/sync-tweaking
Add some tracing and the ability to tweak message fetch window size
|
2020-08-25 13:25:22 -07:00 |
|
whyrusleeping
|
a553db4b82
|
Add some tracing and the ability to tweak message fetch window size
|
2020-08-25 12:39:17 -07:00 |
|
vyzo
|
6b3282150b
|
appease linter
|
2020-08-25 14:03:49 +03:00 |
|
vyzo
|
58cfac9f67
|
add test for message republish
|
2020-08-25 13:57:02 +03:00 |
|
vyzo
|
62b15ecc12
|
remove bug line
|
2020-08-25 13:41:20 +03:00 |
|
vyzo
|
6b9d392d0e
|
add another priority selection test
|
2020-08-25 13:36:03 +03:00 |
|
vyzo
|
9c68123ede
|
add test for Updates, fix behaviour for closed mpool
|
2020-08-25 13:29:09 +03:00 |
|
vyzo
|
7049b6478c
|
add test for Clear
|
2020-08-25 13:24:11 +03:00 |
|
vyzo
|
0e6d34c0c9
|
add test for loadLocal
|
2020-08-25 13:16:55 +03:00 |
|
vyzo
|
a01fa11a26
|
exercise debug code, fix deadlock
|
2020-08-25 13:03:50 +03:00 |
|
vyzo
|
7e81081899
|
initialize mpool ts _before_ loading local messages
|
2020-08-25 08:49:15 +03:00 |
|
Jakub Sztandera
|
8cc8a277a1
|
Remove Println
Signed-off-by: Jakub Sztandera <kubuxu@protocol.ai>
|
2020-08-25 04:37:35 +02:00 |
|
whyrusleeping
|
a5c56dde10
|
add test to verify actor state fix works properly
|
2020-08-24 19:35:54 -07:00 |
|
Jakub Sztandera
|
7f427b8001
|
Fix harness weirdness
Signed-off-by: Jakub Sztandera <kubuxu@protocol.ai>
|
2020-08-25 04:12:33 +02:00 |
|
Jakub Sztandera
|
33b12782e4
|
Take into account messages in current tispet by executing it
Signed-off-by: Jakub Sztandera <kubuxu@protocol.ai>
|
2020-08-25 04:02:06 +02:00 |
|
Jakub Sztandera
|
b2ff231c48
|
Fix error fmt string
Signed-off-by: Jakub Sztandera <kubuxu@protocol.ai>
|
2020-08-25 03:14:52 +02:00 |
|
Jakub Sztandera
|
d921791c55
|
Add comment
Signed-off-by: Jakub Sztandera <kubuxu@protocol.ai>
|
2020-08-25 03:13:43 +02:00 |
|
whyrusleeping
|
05d65c3f88
|
slightly better than panicking
|
2020-08-24 17:40:12 -07:00 |
|