op-batcher: Support generic DA posting #10489
Conversation
…respect op-plasma spec
… to specify da svc
…nt propagation from test da server
…op-plasma/generic-da-commitment
epociask
changed the title
…ssertion when reading in op-node
…c DA client <-> interactions
…c DA client <-> interactions
WalkthroughWalkthroughThe changes involve significant updates to the handling of Data Availability (DA) services across multiple components. Key updates include the addition of a new flag for DA service type in the CLI, modifications to the Changes
Assessment against linked issues
Recent Review DetailsConfiguration used: .coderabbit.yml Files selected for processing (2)
Files skipped from review as they are similar to previous changes (1)
Additional Context UsedRuff (6)
Additional comments not posted (2)
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (invoked as PR comments)
Additionally, you can add CodeRabbit Configration File (
|
…op-plasma/generic-da-commitment
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 0
Out of diff range and nitpick comments (5)
bedrock-devnet/devnet/__init__.py (5)
Line range hint
7-7: Remove unused importcalendar.-import calendar
Line range hint
12-12: Remove unused importgzip.-import gzip
Line range hint
18-18: Remove unused importdevnet.log_setup.-import devnet.log_setup
Line range hint
221-221: Avoid equality comparisons toFalse; useif not os.path.exists(l2_allocs_path):for false checks.- if os.path.exists(l2_allocs_path) == False or DEVNET_FPAC == True: + if not os.path.exists(l2_allocs_path) or DEVNET_FPAC:
Line range hint
312-312: Local variableeis assigned to but never used. Consider removing it or using it in the logging statement.- except Exception as e: + except Exception:
db82b31
Description
op-batcher
DA_SERVICEflag that specifies whether to use precomputed commitments or an alt DA service providerop-node
Keccak256typeop-plasma
daClientto support writing without a precomputed commitmentdaServerto support both interaction types based on prevalence of commitment query param in URL pathdaServerwith more verbose logging to ease debugging / devnet introspectiondevnet
docker-compose.ymlto useDA_SERVICEflag as perop-nodeandop-batcherenv vars__init__.pyto setDA_SERVICEflag in compose environmentExtend
Tests
DA_SERVICE=truein compose environment, tests were ran as per commit (6a1ac766a1ac76) and passed for the alternative DA service interaction. See test run hereda_client.gotests to assert put/get interactions using generic commitment typeAdditional context
Metadata