Skip to content

op-e2e: bump gas limit so its safe#5255

Merged
mergify[bot] merged 1 commit intodevelopfrom
fix/gaslimits
Mar 24, 2023
Merged

op-e2e: bump gas limit so its safe#5255
mergify[bot] merged 1 commit intodevelopfrom
fix/gaslimits

Conversation

@tynes
Copy link
Contributor

@tynes tynes commented Mar 24, 2023

Description

The gas limit must be set to 30 million to ensure that the default deposit gas market config is safe to use. This was missed when #5233 was merged.

The gas limit must be set to 30 million to ensure that
the default deposit gas market config is safe to use.
This was missed when
#5233 was merged.
@tynes tynes requested a review from a team as a code owner March 24, 2023 19:36
@tynes tynes requested a review from trianglesphere March 24, 2023 19:36
@changeset-bot
Copy link

changeset-bot bot commented Mar 24, 2023

⚠️ No Changeset found

Latest commit: 0aba809

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@netlify
Copy link

netlify bot commented Mar 24, 2023

Deploy Preview for opstack-docs canceled.

Name Link
🔨 Latest commit 0aba809
🔍 Latest deploy log https://app.netlify.com/sites/opstack-docs/deploys/641dfba1b9541300077b83cd

@codecov
Copy link

codecov bot commented Mar 24, 2023

Codecov Report

Merging #5255 (0aba809) into develop (d3e9d7b) will decrease coverage by 3.86%.
The diff coverage is n/a.

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff             @@
##           develop    #5255      +/-   ##
===========================================
- Coverage    39.77%   35.91%   -3.86%     
===========================================
  Files          381      226     -155     
  Lines        24187    19683    -4504     
  Branches       837        0     -837     
===========================================
- Hits          9621     7070    -2551     
+ Misses       13836    11916    -1920     
+ Partials       730      697      -33     
Flag Coverage Δ
bedrock-go-tests 35.91% <ø> (-0.06%) ⬇️
common-ts-tests ?
contracts-bedrock-tests ?
contracts-tests ?
core-utils-tests ?
dtl-tests ?
fault-detector-tests ?
sdk-tests ?

Flags with carried forward coverage won't be shown. Click here to find out more.

see 157 files with indirect coverage changes

@mergify mergify bot merged commit ef45bec into develop Mar 24, 2023
@mergify mergify bot deleted the fix/gaslimits branch March 24, 2023 19:55
@mergify
Copy link
Contributor

mergify bot commented Mar 24, 2023

This PR has been added to the merge queue, and will be merged soon.

@mergify
Copy link
Contributor

mergify bot commented Mar 24, 2023

This PR is next in line to be merged, and will be merged as soon as checks pass.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants