-
Notifications
You must be signed in to change notification settings - Fork 435
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
Support the Arbitrum BoLD Challenge Protocol in Nitro #2362
Open
amsanghi
wants to merge
604
commits into
master
Choose a base branch
from
bold-review
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
+2,542
−352
Open
Changes from all commits
Commits
Show all changes
604 commits
Select commit
Hold shift + click to select a range
47c3d2f
fix lock
amsanghi 9d2b418
reenable
rauljordan 17ed36e
bold
rauljordan a4d590f
storage
rauljordan 0fbc8de
min
rauljordan 30d3716
edit
rauljordan 8f53605
slice storage
rauljordan dd184c4
edits
rauljordan a9e1940
bold commit
rauljordan abe6a64
edit bold
rauljordan 17fe12c
update bold
rauljordan 992a30a
Update BOLD state provider for overflow assertions
PlasmaPower 0fe01f3
test fix
rauljordan a0f607c
Merge branch 'sepolia-tooling-merge' into overflow-assertions-state-p…
PlasmaPower a3acc9a
update timer cache
rauljordan fbc6088
revive e2e test
rauljordan 2d12159
add in
rauljordan 9dfdc14
wait for safe
rauljordan b140600
Merge branch 'sepolia-tooling-merge' of github.com:OffchainLabs/nitro…
rauljordan d45f125
edit bold
rauljordan 147a7ff
add execution node offline metrics
amsanghi 34eac84
bold
rauljordan d315508
Merge branch 'sepolia-tooling-merge' of github.com:OffchainLabs/nitro…
rauljordan 1114526
Merge branch 'sepolia-tooling-merge' into overflow-assertions-state-p…
PlasmaPower 8cc83e6
update bold
rauljordan f6aa184
many chal levels
rauljordan 4f0578c
sync bold
rauljordan 175bf8f
test to check for challenge win
rauljordan d4b39c9
add in test fixes and lint
rauljordan 7612671
update bold commit
rauljordan d84fed7
tag
rauljordan de8f91c
edited
rauljordan f766cab
bold ref
rauljordan 0aebf14
trying to fix
rauljordan 22ec174
test edit
rauljordan ab39b5a
config test
rauljordan 6b7e642
test fix
rauljordan 11117b4
lint
rauljordan 8593f5f
lint
rauljordan aa0af74
db path
rauljordan a7ca863
lint
rauljordan 315132f
revamp
rauljordan c8f2a04
support history commit in assertions
rauljordan abbc2f8
test
rauljordan a46caf7
add hist commit
rauljordan bdcd5d9
challenge test run
rauljordan 9e3be32
para
rauljordan 1d5c3c7
add log
rauljordan 16927dc
more checks
rauljordan 74a93c4
rev
rauljordan 3d69fc4
flakey with bold
rauljordan 614c719
add bold
rauljordan f815e04
hist commit
rauljordan ee5d1e4
rem
rauljordan 8a25e9e
commit edit
rauljordan 0ac8cb0
fix up
rauljordan 568285d
Increase challenge tests timeout to 30m
PlasmaPower 2542157
Fix TestChallengeProtocolBOLD shutdown
PlasmaPower f5deb86
test check
rauljordan 061bb89
build tag
rauljordan 94bf8d2
Merge branch 'master' into sepolia-tooling-merge
amsanghi 9e4f6e1
fix
amsanghi ed69007
unskip test with fix
amsanghi 86fddb2
Merge branch 'sepolia-tooling-merge' into overflow-assertions-state-p…
amsanghi 17a9d02
Add back FindBatchContainingMessageIndex, was removed in the nitro la…
amsanghi e037379
unskip more test
amsanghi a459cb8
unskip more test
amsanghi 6ee4ec0
Merge pull request #2201 from OffchainLabs/overflow-assertions-state-…
rauljordan bc030c0
rem debug bin
rauljordan 5c6c886
bold commit
rauljordan 0d2c8c6
finish once osp
rauljordan cdcd4c7
confirm at osp
rauljordan 877f618
revert build tag
rauljordan 820a1bc
parallel
rauljordan 77b90c8
Merge branch 'master' into sepolia-tooling-merge
rauljordan ea84f61
update bold commit
rauljordan d1cb639
add in bold commit
rauljordan ed5edb8
update bold commit
rauljordan 4c63dfb
log line fixes
rauljordan 3302de6
nogo
rauljordan dc88572
edit bold pin
rauljordan 67c5ab4
update bold commit
rauljordan 43b0529
bold commit
rauljordan 37e2f1b
bold commit
rauljordan ed41af8
latest bold code
rauljordan 09ec53c
go errors
rauljordan 46aaca3
update bold
rauljordan 1369f29
sync edits from master
rauljordan 1cb57e3
update geth
rauljordan 450ccd8
sync more edits
rauljordan cce4b22
update submods
rauljordan 95df92b
update bold commit
rauljordan 9cc5cdb
edits to build
rauljordan 573110c
build system test
rauljordan 7ba9dc2
update bold ref
rauljordan 6158550
Merge branch 'master' into sepolia-tooling-merge
rauljordan 3f2b6d7
bold commit
rauljordan 296c0fc
fix build
rauljordan 4746270
pass state provider tests again
rauljordan b9e2552
edit back tags
rauljordan 574d7bd
edits testing perf
rauljordan 0cc28bb
bold commit
rauljordan 60fa5a0
Merge branch 'master' into sepolia-tooling-merge
amsanghi 703630a
bold commit
rauljordan 0d8cf2e
Merge branch 'sepolia-tooling-merge' of github.com:OffchainLabs/nitro…
rauljordan df66e5b
pass test
rauljordan 90e5cc8
Cleanup data_poster.go
amsanghi cf718e5
clean up block_processor.go
amsanghi a6ca05b
clean up backlog_test.go
amsanghi 7785926
clean up block_recorder.go
amsanghi 162edd9
clean up executionengine.go
amsanghi d078915
clean up node.go
amsanghi 3eb42c7
clean up nitro.log
amsanghi 8d96ae9
clean up block_validator.go
amsanghi 920ac59
clean up state_provider.go
amsanghi 941d2a2
clean up stateless_block_validator.go
amsanghi 5bba5e0
clean up assertion_on_large_number_of_block_test.go
amsanghi 9d720a8
clean up bold_challenge_protocol_test.go
amsanghi 2467ac0
clean up CreateBoldExecutionRun
amsanghi 22ccdb7
Clean up manager call
amsanghi 2a44c92
Merge branch 'master' into bold-review
amsanghi b8c6502
support bold main
rauljordan 100ad6e
support config
rauljordan abbcd20
edits
rauljordan 5935794
bold commit
rauljordan 2cb59b9
get staked info
rauljordan a7f2fbe
support bold initialization in staker
rauljordan 053cb36
support macos in download machine
rauljordan 6c4c401
Merge branch 'master' into bold-review
rauljordan 7b77f7f
address some review comments
rauljordan 4e7b4a5
Merge branch 'bold-review' of github.com:OffchainLabs/nitro into bold…
rauljordan cbc3f7e
revert arbitrator edits
rauljordan 983a1e8
revert some old stuff
rauljordan 2a313c1
arbitrator
rauljordan 6835a7b
cache comments
rauljordan 099d03c
sync
rauljordan bfba244
commentary and cleanup
rauljordan 5ccbbb4
edit names
rauljordan af8cc34
staker switch elsewhere
rauljordan ba2e3b1
bridgegen
rauljordan a171c95
more porting
rauljordan 0375950
multiprotocol switcher
rauljordan b9c6c10
define the multiprotocol staker
rauljordan c3d8ec8
builds
rauljordan 4e599d4
Merge branch 'master' into bold-review
rauljordan 751f1d9
bold commit
rauljordan 9a9012d
Merge branch 'bold-review' of github.com:OffchainLabs/nitro into bold…
rauljordan 192e047
bold item
rauljordan 6a33e3e
provide block validator
rauljordan 33e2ca8
rev
rauljordan 6470551
use stateless
rauljordan e57b0b0
add challenge tags
rauljordan 737b99e
Merge branch 'master' into bold-review
rauljordan 9ae02fc
Merge branch 'master' into bold-review
rauljordan 28ba3e6
Merge branch 'bold-review' of github.com:OffchainLabs/nitro into bold…
rauljordan 0b217ac
edit
rauljordan 573880a
Merge branch 'master' into bold-review
rauljordan 9363a29
build
rauljordan e721188
Merge branch 'bold-review' of github.com:OffchainLabs/nitro into bold…
rauljordan 0b59ee3
master
rauljordan 494d2a6
geth
rauljordan 43d3da8
lint
rauljordan 9aa0a82
Merge branch 'master' into bold-review
rauljordan 99a6252
chal cache
rauljordan a2d9e45
check positive condition
rauljordan b2f8e53
config items
rauljordan 06f49df
dedup
rauljordan 6c2781e
logs on switch
rauljordan 6704f92
ensure single instance init and use block latest validated state
rauljordan 2c58a9e
ci for state provider pass
rauljordan c2b0f82
ci
rauljordan ad8b825
ci runs but need diff validators
rauljordan 4426b46
ci challenge
rauljordan e5e4ea1
gotags
rauljordan 6abb7cb
Merge branch 'master' into bold-review
rauljordan 928a8f8
unit tests
rauljordan d66cc0e
Merge branch 'bold-review' of github.com:OffchainLabs/nitro into bold…
rauljordan d7efbab
chal timeout
rauljordan 57a0d95
Merge branch 'master' into bold-review
rauljordan b3f29c0
test flakey when parallel
rauljordan 9565823
rem
rauljordan 528f868
Merge branch 'bold-review' of github.com:OffchainLabs/nitro into bold…
rauljordan 69bc7bd
tags
rauljordan 886a21d
much simpler execution run
rauljordan e9cc824
include server arb changes
rauljordan 58e093f
exec run pass
rauljordan f89cbd8
bold commit
rauljordan 6a8b741
Merge branch 'get-machine-hashes-with-step' into bold-review
rauljordan 06663c2
include latest
rauljordan c3f737e
Merge branch 'bold-challenge-cache' into bold-review
rauljordan 7ed6155
builds
rauljordan d770502
Merge branch 'get-machine-hashes-with-step' into bold-review
rauljordan 8f8459d
state provider changes
rauljordan 513bee2
Merge branch 'bold-challenge-cache' into bold-review
rauljordan 30f125b
Merge branch 'master' into bold-review
rauljordan 25d0437
Merge branch 'master' into bold-review
rauljordan 8fb0788
bold item
rauljordan 9ab81eb
bold commit
rauljordan 59fc8a5
bold commit
rauljordan 5774b3d
update bold branch
rauljordan 8e3e19a
update bold
rauljordan 0437e7b
add osp logging
rauljordan 4bfe480
Merge branch 'master' into bold-review
rauljordan 9973230
edits
rauljordan 746978c
add in validator
rauljordan a6a2312
edits
rauljordan a6a29e1
edit bold commit
rauljordan a63950c
norace
rauljordan 20a9857
edit bold main
rauljordan e0154ec
Merge branch 'master' into bold-review
amsanghi 65819f0
Merge branch 'get-machine-hashes-with-step' into bold-review
amsanghi 5589f2b
minor fix
amsanghi 6656d0e
Merge branch 'bold-review' of github.com:OffchainLabs/nitro into bold…
rauljordan 3b9ad4d
edits to state provider
rauljordan 2506150
sync master
rauljordan 7012c91
Merge branch 'master' into bold-review
rauljordan 5f2e60d
edits
rauljordan 658a5ce
commits
rauljordan 7ce2089
Merge branch 'master' into bold-review
rauljordan 8265691
Merge branch 'master' into bold-review
rauljordan a525bb1
Merge branch 'master' into bold-review
rauljordan 0a078a7
Merge branch 'master' into bold-review
rauljordan 051095f
Merge branch 'master' into bold-review
amsanghi c56a509
Changes based on PR comments
amsanghi d3e62a4
fix build and lint
amsanghi 6011b74
Changes based on PR comments and fix build
amsanghi 633f175
Merge branch 'master' into bold-review
amsanghi 57af6a4
reword ExecutionStateAfterPreviousState comment
amsanghi 533fceb
add dangerous option where validator is nil and node just assumes all…
amsanghi e3b8d70
checks for the wasmModuleRoot in the rollup and updates validator reg…
amsanghi c6faf78
Changes based on PR comments
amsanghi 2d479d1
Merge branch 'master' into bold-review
amsanghi e1d40d3
fix test
amsanghi 4914c34
fix test
amsanghi abdc7a1
rename function
amsanghi efdeb8d
Merge branch 'master' into bold-review
amsanghi 8446078
Changes based on PR comments
amsanghi 3b2cfdb
Send UpdateLatestStaked and UpdateLatestConfirmed notifications
amsanghi b28b78c
Merge branch 'master' into bold-review
amsanghi 220d133
Merge branch 'master' into bold-review
amsanghi 3c86220
clean up
amsanghi 2205adb
Merge branch 'master' into bold-review
amsanghi d97ae8e
fix build
amsanghi 748a9f6
fix build and panic
amsanghi f3bf2ac
fix test
amsanghi e734a4d
Add build tags
amsanghi 88d143b
update bold main branch
rauljordan 8e12821
update main
rauljordan c102262
Merge branch 'master' into bold-review
eljobe c80e048
Fix rawdb -> ethdb change
eljobe 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
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
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
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 |
---|---|---|
|
@@ -120,5 +120,6 @@ type RollupAddresses struct { | |
UpgradeExecutor common.Address `json:"upgrade-executor"` | ||
ValidatorUtils common.Address `json:"validator-utils"` | ||
ValidatorWalletCreator common.Address `json:"validator-wallet-creator"` | ||
StakeToken common.Address `json:"stake-token"` | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. BOLD requires this new field as part of the rollup addresses, which is the address of the token used for challenge bonds |
||
DeployedAt uint64 `json:"deployed-at"` | ||
} |
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.
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.
This function was added to aid in the high-concurrency of BOLD moves and transactions