-
Notifications
You must be signed in to change notification settings - Fork 467
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
Conversation
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 is very initial. Didn't go over most of it - but already some points that we can start working on.
staker/challenge-cache/cache_test.go
Outdated
@@ -0,0 +1,318 @@ | |||
// Copyright 2023, Offchain Labs, Inc. | |||
// For license information, see https://github.com/offchainlabs/bold/blob/main/LICENSE | |||
package challengecache |
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.
note to self: this package not yet reviewed
if err != nil { | ||
return nil, err | ||
} | ||
creationBlock = node.CreatedAtBlock |
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.
Sounds good, since this only blocks L3 support I don't think it needs to block merging
@rauljordan I pushed up a commit with what I was thinking of re: useBoldMachine. Let me know what you think |
Looks great thanks @PlasmaPower I was struggling a bit with changing all the different places, as there were several interfaces and uses intertwined, but I'm glad we can just use a bool in most areas |
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 haven't reviewed the tests fully, but I don't think that needs to block merging. LGTM
@rauljordan the BOLD challenge tests don't seem to be currently building https://github.com/OffchainLabs/nitro/actions/runs/12201699985/job/34040676723?pr=2362#step:29:837 I think it'd be good to fix that before merging |
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.
LGTM
Background
This PR includes the Arbitrum BoLD challenge protocol as part of a Nitro node, enabled optionally. The goal is to have Nitro
master
capable of supporting BoLD challenges on Arbitrum Sepolia once the upgrade occurs and on the existing BoLD testnet.BoLD, standing for Bounded Liquidity Delay, is a protocol for resolving disputes over execution of the Arbitrum inbox. The BoLD protocol allows for permissionless validation of Arbitrum One and Nova. It also ensures an upper-bound on withdrawal delays from L2 to L1.
To learn more about the basics of BoLD, see the gentle introduction here, or the research paper here. A presentation explaining what the BoLD software does and how it works can be found here
BoLD Software Responsibilities
BoLD is a modular component of Nitro nodes that can do the following tasks:
All of these responsibilities are performed by the BoLD challenge manager service, which is currently included as a git submodule of Nitro, with its git repository OffchainLabs/bold, with long term plans to port it fully intro Nitro.
Supporting BoLD in Nitro
In order for Nitro to support this BoLD module, Nitro has to provide a dependency to the bold challenge manager that defines the following methods:
We implement these methods in
staker/bold_state_provider.go
and also add a new RPC method to the validation node calledGetMachineHashesWithStepSize
that can execute an Arbitrator machine at a start program counter and collect machine hashes in increments ofstep_size
as required by BoLD.We also implement a new
challengecache
package which can cache hashes retrieved from the validation node in a filesystem hierarchy for fast access by the Nitro node during BoLD challenges. This cache is required for efficient computation of challenge moves, and is a new addition. It stores hashes as bytes in files nested by challenge level hierarchy.Switching from Old Protocol to BoLD at Runtime
To change from the old staker to the BoLD staker at runtime, we provide a new service called
MultiProtocolStaker
, which holds a reference to the old, L1 staker, and the new BoLD staker and can swap to BoLD once it is supported. The way it works is it checks if the rollup contract supports a certain method calledExtraChallengeBlocks
. If the method is unsupported, the Nitro validator should be switching to BoLD.Changelog Summary
Git / Deps
Arbitrator
Nitro Node
StakeToken
address field to the L2 chain info JSONMultiProtocolStaker
instead of aStaker
which is aware of BoLD and can switch protocolschallengecache
package to cache hashes obtained from validation node computation to be used when making moves in BoLDValidation Node API
GetMachineHashesWithStepSize
method that can execute Arbitrator machines at specific program counters, step through them in custom step sizes, and collect machine hashes along the wayMulti Protocol Staker Definition
Testing
TestChallengeProtocolBOLD
which runs a complete challenge, with an L1 node two different L2 nodes disagreeing about the execution of a certain batch posted to the Arbitrum inbox. It uses the BoLD dispute protocol to fully resolve the challengeTestChallengeProtocolBOLD_StateProvider
which tests the invariants of thebold_state_provider.go
implementation needed by the BoLD challenge managerMissing Features
This is now part of NIT-2793