Tags: gas-six/rollkit
Tags
da: handle tx too large (rollkit#1620) ## Overview This PR updates the block submission loop to additionally handle `tx too large` error. ## Checklist - [x] New and updated code has appropriate documentation - [x] New and updated code has new and/or updated testing - [x] Required CI checks are passing - [x] Visual proof for any user facing features like CLI or documentation updates - [x] Linked issues closed with keywords <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit - **New Features** - Introduced handling for transactions exceeding the maximum size limit. - **Bug Fixes** - Updated logic to correctly handle oversized transactions. - **Tests** - Added new test cases to verify the handling of transactions that are too large. <!-- end of auto-generated comment: release notes by coderabbit.ai -->
fix(manager): only reset custom gas price (rollkit#1595) ## Overview This PR fixes custom gas price reset after a successful submission. ## Checklist - [x] New and updated code has appropriate documentation - [x] New and updated code has new and/or updated testing - [x] Required CI checks are passing - [x] Visual proof for any user facing features like CLI or documentation updates - [x] Linked issues closed with keywords <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit - **Bug Fixes** - Modified the logic for adjusting `gasPrice` to prevent division operation when not required. - **Refactor** - Refactored test cases for submitting blocks to a mock DA to cover different scenarios. - **Validation** - Added a validation check for `DAGasMultiplier` in `nodeConfig` to meet specific criteria. <!-- end of auto-generated comment: release notes by coderabbit.ai -->
fix(da): increase backoff for mempool errors (rollkit#1535) ## Overview This PR increases the backoff duration for mempool errors, avoiding the scenario where a temporary mempool error e.g due to congestion can exhaust all retries. Fixes rollkit#1522 ## Checklist - [ ] New and updated code has appropriate documentation - [ ] New and updated code has new and/or updated testing - [ ] Required CI checks are passing - [ ] Visual proof for any user facing features like CLI or documentation updates - [ ] Linked issues closed with keywords <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit - **New Features** - Introduced a default expiration time for mempool transactions to improve transaction management. - Enhanced block submission with handling for `maxBlobSize` and `gasPrice`, offering better flexibility and efficiency. - Added a new flag in `rollkit start` command for setting DA gas price multiplier, facilitating more robust transaction retries. - **Improvements** - Extended the submission timeout period from 60 to 90 seconds, allowing for more reliable block submissions under network delays. - Enriched error messaging and introduced new status codes for block submission to improve debugging and operational transparency. - **Refactor** - Replaced `MockDA` struct with `mock.MockDA` in tests for a more streamlined testing approach. - Implemented a mock `DA` interface in `da/mock/mock.go`, enhancing testability and development efficiency. - **Tests** - Updated various test functions to accommodate new parameters and improved error handling, ensuring the reliability of changes. - **Documentation** - Documented the addition of the DA gas price multiplier flag in `rollkit start` command usage. <!-- end of auto-generated comment: release notes by coderabbit.ai -->
build(deps): bump go-da to v0.2.0 (rollkit#1448) ## Overview This PR updates go-da to v0.2.0 which accepts a context as the first argument to DA interface methods. ## Checklist - [x] New and updated code has appropriate documentation - [x] New and updated code has new and/or updated testing - [x] Required CI checks are passing - [x] Visual proof for any user facing features like CLI or documentation updates - [x] Linked issues closed with keywords <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit - **Refactor** - Enhanced the data access client to support context parameters for improved operation handling. - **Tests** - Updated tests to align with the new context parameter integration in data access methods. <!-- end of auto-generated comment: release notes by coderabbit.ai -->
PreviousNext