Skip to content

Enabled GasLimitStorageGrowthRatio#997

Merged
evgeny-s merged 19 commits intomasterfrom
MD-922-evgeny-storage-growth-ratio
May 12, 2025
Merged

Enabled GasLimitStorageGrowthRatio#997
evgeny-s merged 19 commits intomasterfrom
MD-922-evgeny-storage-growth-ratio

Conversation

@evgeny-s
Copy link
Contributor

@evgeny-s evgeny-s commented Apr 24, 2025

Description:
Apply GasLimitStorageGrowthRatio parameter.

Initially introduced here.

Observations:
After the storage ratio parameter is enabled, the gas limit for some tests should be increased manually.
Theoretically, we want the eth_estimateGas should be able to estimate it automatically.

Before merge:

  • Make the PR for moonkit is merged.
  • Make sure Cargo.toml updated to the branch: tanssi-polkadot-stable2412

@evgeny-s evgeny-s added A8-mergeoncegreen Pull request is reviewed well. B0-silent Changes should not be mentioned in any release notes not-breaking Does not need to be mentioned in breaking changes D3-trivial PR contains trivial changes in a runtime directory that do not require an audit labels Apr 24, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Apr 24, 2025

WASM runtime size check:

Compared to target branch

dancebox runtime: 1528 KB (no changes) ✅

flashbox runtime: 888 KB (no changes) ✅

dancelight runtime: 2332 KB (no changes) ✅

container chain template simple runtime: 1168 KB (no changes) ✅

container chain template frontier runtime: 1456 KB (no changes) ✅

@github-actions
Copy link
Contributor

github-actions bot commented Apr 24, 2025

Coverage Report

(master)

@@                          Coverage Diff                           @@
##           master   MD-922-evgeny-storage-growth-ratio      +/-   ##
======================================================================
- Coverage   64.76%                               64.75%   -0.01%     
  Files         482                                  482              
+ Lines       78171                                78229      +58     
======================================================================
+ Hits        50621                                50657      +36     
+ Misses      27550                                27572      +22     
Files Changed Coverage
/chains/container-chains/runtime-templates/frontier/src/lib.rs 58.34% (+0.21%)

Coverage generated Mon May 12 09:31:01 UTC 2025

@evgeny-s evgeny-s enabled auto-merge (squash) May 12, 2025 09:09
@evgeny-s evgeny-s merged commit 363a979 into master May 12, 2025
54 checks passed
@evgeny-s evgeny-s deleted the MD-922-evgeny-storage-growth-ratio branch May 12, 2025 09:31
nanocryk pushed a commit that referenced this pull request May 19, 2025
Co-authored-by: tmpolaczyk <44604217+tmpolaczyk@users.noreply.github.com>
@evgeny-s evgeny-s added breaking Needs to be mentioned in breaking changes and removed not-breaking Does not need to be mentioned in breaking changes labels Jul 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A8-mergeoncegreen Pull request is reviewed well. B0-silent Changes should not be mentioned in any release notes breaking Needs to be mentioned in breaking changes D3-trivial PR contains trivial changes in a runtime directory that do not require an audit

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants