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

increase geth gaslimit (AUTO-9280) #12571

Merged
merged 4 commits into from
Mar 25, 2024
Merged

increase geth gaslimit (AUTO-9280) #12571

merged 4 commits into from
Mar 25, 2024

Conversation

anirudhwarrier
Copy link
Contributor

@anirudhwarrier anirudhwarrier commented Mar 25, 2024

@anirudhwarrier anirudhwarrier changed the title increase geth gaslimit increase geth gaslimit (AUTO-9280) Mar 25, 2024
Comment on lines +204 to +205
"startGaslimit": "20000000",
"targetGasLimit": "30000000",
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Keeping these values hardcoded for now. If need be, can be made configurable.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How will the start and target limits be used?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Here is the corresponding changes in the geth chart - https://github.com/smartcontractkit/chainlink-testing-framework/pull/875/files

@anirudhwarrier anirudhwarrier marked this pull request as ready for review March 25, 2024 17:26
@anirudhwarrier anirudhwarrier requested a review from a team as a code owner March 25, 2024 17:26
@cl-sonarqube-production
Copy link

Quality Gate passed Quality Gate passed

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarQube

@anirudhwarrier anirudhwarrier added this pull request to the merge queue Mar 25, 2024
Merged via the queue into develop with commit 20e730a Mar 25, 2024
104 checks passed
@anirudhwarrier anirudhwarrier deleted the load-test-auto branch March 25, 2024 18:51
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