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

Revert changes for high fixed gas prices #680

Closed
konradkonrad opened this issue May 27, 2021 · 2 comments
Closed

Revert changes for high fixed gas prices #680

konradkonrad opened this issue May 27, 2021 · 2 comments
Labels
bug Something isn't working

Comments

@konradkonrad
Copy link
Contributor

konradkonrad commented May 27, 2021

Description

During the Bespin release candidate testing, we switched the SP CI setups to use a fixed gas price setting of a rather high 500 gwei. (See https://github.com/raiden-network/scenario-player/pull/679/files).
This, paired with an increased node funding, was meant to reduce failure noise that originated in weird gas price fluctuations in the goerli network.

Since this is neither a realistic test setup, nor a sustainable way to operate the scenario player (we'd run out of goerliETH quite soon), we need to roll back these changes and come up with more reasonable solutions (see also raiden-network/raiden#7095).

The current change is activated through adjusting the variable

- SP_BRANCH=develop
+ SP_BRANCH=higher_funding

in /var/lib/scenario-player/env.sh on both SP CI servers.

Due to the related changes in raiden-network/raiden#7087 and raiden-network/raiden#7088 we may also have to adjust the default environments in this repository.

@konradkonrad konradkonrad added the bug Something isn't working label May 27, 2021
@konradkonrad
Copy link
Contributor Author

@ezdac please add anything I may have missed above!

@karlb
Copy link
Contributor

karlb commented May 31, 2021

Due to the related changes in raiden-network/raiden#7087 and raiden-network/raiden#7088 we may also have to adjust the default environments in this repository.

I fixed that with d5e863e, already.

@karlb karlb closed this as completed Jun 1, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants