-
Notifications
You must be signed in to change notification settings - Fork 162
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
feat: support persist active api #78
Merged
Merged
Conversation
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
* copy over develop chainsgo * stage rollup config changes * final rollup config values
Update to upstream optimism(v1.1.0) release
… and handleReceipt access state concurrently (bnb-chain#5)
* chore: update readme, add testnet assets * doc: clarify readme
…ain#11) * ci: add the ci code used to package and release docker images (bnb-chain#7) * ci: add the ci code used to package and release docker images Co-authored-by: Welkin <[email protected]> * fix: add latest tag for docker image (bnb-chain#9) Co-authored-by: Welkin <[email protected]> * try to use cache for docker build (bnb-chain#10) Co-authored-by: Welkin <[email protected]> --------- Co-authored-by: Welkin <[email protected]>
(cherry picked from commit d252ea2)
…equencer (cherry picked from commit 449321b)
(cherry picked from commit 4352684)
Improve commenting/organisation. (cherry picked from commit a370ecf)
Log info about loaded sequencer state (cherry picked from commit 9d5c144)
(cherry picked from commit fe1fdf8)
(cherry picked from commit 18f5efd)
(cherry picked from commit 02116fa)
(cherry picked from commit 3d19b6d)
(cherry picked from commit 4163278)
Returns true if the node is actively sequencing, otherwise false. This is essentially a parallel with the `eth_mining` API from execution clients. (cherry picked from commit ef9c209)
Support persist active api
owen-reorg
requested review from
xuan-cu and
bnoieh
and removed request for
krish-nr and
redhdx
October 27, 2023 11:41
bendanzhentan
approved these changes
Oct 30, 2023
xuan-cu
approved these changes
Oct 30, 2023
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Description
Merge a previous PR #24 to develop branch.
Rationale
The previous pull request, #24, introduced the
rpc.admin-state
flag for the op-node, enabling high availability scenarios. Initially, it only existed on a separate branch. We have now merged it into the develop branch.Example
Add
--rpc.admin-state=/server/admin-state.log
parameter for op-node to enable this feature.Changes
Notable changes:
rpc.admin-state
flag for op-node.