-
Notifications
You must be signed in to change notification settings - Fork 714
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
[Net][GUI][RPC] Add enable/disable network activity feature #2606
Conversation
1b477fe
to
769ec19
Compare
rebased on master, conflicts solved. |
backports btc@7c9a98aac843c9efabd8653caebc35e968b2f335
769ec19
to
1f21536
Compare
rebased, conflicts solved. |
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.
ACK 1f21536
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.
Looking good. One minor thing: the checked state of the buttons pushNetEnable
/pushNetDisable
should change if the activity has been changed via RPC.
1f21536
to
102dcaf
Compare
done, updated per feedback. |
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.
ACK 102dcaf
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.
ACK 102dcaf
Allowing the user/test to stop and restart the network activity.
Pretty useful for some manual isolation test cases.
When the network activity is disabled the client will close all connections, stop accepting inbound connections, and stop opening new outbound connections, until the network activity is reenabled.
Quick demo video:
network_activity.mp4
TODO:
Add release-notes for the new RPC command and the GUI control.