Skip to content
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

Bump max gas wanted per tx to 60 mil (backport #7833) #7861

Merged
merged 1 commit into from
Mar 27, 2024

Conversation

mergify[bot]
Copy link
Contributor

@mergify mergify bot commented Mar 27, 2024

Closes: #XXX

What is the purpose of the change

Bumps max gas wanted per tx default to 6 million.

This would resolve all the issues projects were having when they were uploading contracts.

Testing and Verifying

This change is a trivial rework / code cleanup without any test coverage.

Documentation and Release Note

  • Does this pull request introduce a new feature or user-facing behavior changes?
  • Changelog entry added to Unreleased section of CHANGELOG.md?

Where is the change documented?

  • Specification (x/{module}/README.md)
  • Osmosis documentation site
  • Code comments?
  • N/A

Summary by CodeRabbit

  • Performance Improvements
    • Increased the maximum gas limit per transaction to enhance transaction processing capabilities.

This is an automatic backport of pull request #7833 done by [Mergify](https://mergify.com).

* Bump max gas wanted per tx

* Add changelog

* Add app default values

* drive by correct overwriteAppTomlValues logic

* remove extra line in changelog

* clean up

---------

Co-authored-by: Adam Tucker <[email protected]>
Co-authored-by: Adam Tucker <[email protected]>
(cherry picked from commit 7b0c14a)
@PaddyMc PaddyMc merged commit d97d3ac into v24.x Mar 27, 2024
1 check passed
@PaddyMc PaddyMc deleted the mergify/bp/v24.x/pr-7833 branch March 27, 2024 11:36
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.

2 participants