-
Notifications
You must be signed in to change notification settings - Fork 636
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
fix(proxy_restart): Consistent way to restart nodes with proxy #3272
Conversation
@@ -326,6 +327,10 @@ def proxify_node(node, ps, handler, global_stopped, error, proxy): | |||
outer_port = inner_port + 100 | |||
|
|||
def start_proxy(): | |||
# local_stopped denotes the current of state of the proxy: | |||
# 0: The proxy is running | |||
# 1: The proxy is running but should be closed soon |
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.
where is 1 used
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.
To kill a node, this value is set to 1
so the underlying proxy finishes
https://github.com/nearprotocol/nearcore/blob/master/pytest/lib/cluster.py#L343
Codecov Report
@@ Coverage Diff @@
## master #3272 +/- ##
=======================================
Coverage 87.22% 87.22%
=======================================
Files 217 217
Lines 42397 42397
=======================================
Hits 36981 36981
Misses 5416 5416 Continue to review full report at Codecov.
|
On proxy restart wait for the previous instance to get closed before starting a new one fixes #3134 Test plan ======= sanity/proxy_restart.py pass consistently http://nayduck.eastus.cloudapp.azure.com:3000/#/run/285
On proxy restart wait for the previous instance to get closed before starting a new one fixes #3134 Test plan ======= sanity/proxy_restart.py pass consistently http://nayduck.eastus.cloudapp.azure.com:3000/#/run/285
On proxy restart wait for the previous instance to get closed before starting a new one
fixes #3134
Test plan
sanity/proxy_restart.py pass consistently
http://nayduck.eastus.cloudapp.azure.com:3000/#/run/285