-
Notifications
You must be signed in to change notification settings - Fork 197
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
integrate new vm parameters #3913
Conversation
Codecov Report
@@ Coverage Diff @@
## development #3913 +/- ##
============================================
Coverage 74.84% 74.84%
============================================
Files 614 614
Lines 81721 81723 +2
============================================
+ Hits 61162 61164 +2
Misses 15899 15899
Partials 4660 4660
Continue to review full report at Codecov.
|
cmd/node/config/config.toml
Outdated
@@ -696,13 +696,17 @@ | |||
|
|||
[VirtualMachine] | |||
[VirtualMachine.Execution] | |||
TimeOutForSCExecutionInMilliseconds = 10000 # 10 seconds = 10000 milliseconds | |||
WasmerSIGSEGVPassthrough = false |
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.
Since this is a very sensitive config, I would simply use a const for this.
The const would have been altered when the system test prepared by a sed
command.
go.mod
Outdated
@@ -57,6 +57,6 @@ replace github.com/ElrondNetwork/arwen-wasm-vm/v1_2 v1.2.39 => github.com/Elrond | |||
|
|||
replace github.com/ElrondNetwork/arwen-wasm-vm/v1_3 v1.3.39 => github.com/ElrondNetwork/arwen-wasm-vm v1.3.39 | |||
|
|||
replace github.com/ElrondNetwork/arwen-wasm-vm/v1_4 v1.4.44 => github.com/ElrondNetwork/arwen-wasm-vm v1.4.44 | |||
replace github.com/ElrondNetwork/arwen-wasm-vm/v1_4 v1.4.44 => github.com/ElrondNetwork/arwen-wasm-vm v1.4.45-0.20220317104253-9919c14d121e |
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.
proper release?
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.
integrate new vm parameters