-
Notifications
You must be signed in to change notification settings - Fork 198
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
extracted max gas limit for vm query to config #4057
Conversation
require.True(t, runSCWasCalled) | ||
}) | ||
|
||
t.Run("custom gas defined, should use max uint64", func(t *testing.T) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should not use
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
changed
Codecov Report
@@ Coverage Diff @@
## master #4057 +/- ##
==========================================
- Coverage 74.79% 74.79% -0.01%
==========================================
Files 609 609
Lines 80935 80939 +4
==========================================
+ Hits 60536 60537 +1
- Misses 15764 15766 +2
- Partials 4635 4636 +1
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
System test passed .
@@ Log scanner @@
max-gas-limit-vmquery-config-value
================================================================================
- Known Warnings 10
- New Warnings 2
- Known Errors 0
- New Errors 0
- Panics 0
================================================================================
cmd/node/config/config.toml
Outdated
@@ -588,6 +588,9 @@ | |||
SameSourceRequests = 10000 | |||
# SameSourceResetIntervalInSec time frame between counter reset, in seconds | |||
SameSourceResetIntervalInSec = 1 | |||
# MaxGasPerVmQuery defines the maximum amount of gas to be allocated for VM Queries comming from API | |||
# If set to 0, then MaxUInt64 will be used | |||
MaxGasPerVmQuery = 1500000000 #1.5b |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This has nothing to do with the antiflood param. Please move it in the vm queries configs from api
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
moved
86f3197
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
System test passed.
Warnings are present before the upgrade.
@@ Log scanner @@
max-gas-limit-vmquery-config-value
================================================================================
- Known Warnings 17
- New Warnings 3
- Known Errors 0
- New Errors 0
- Panics 0
================================================================================
extracted the maximum gas limit to be used for vm queries inside config.toml