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

stress.py with network proxy #3050

Closed
SkidanovAlex opened this issue Jul 28, 2020 · 3 comments
Closed

stress.py with network proxy #3050

SkidanovAlex opened this issue Jul 28, 2020 · 3 comments
Assignees
Labels
A-chain Area: Chain, client & related

Comments

@SkidanovAlex
Copy link
Collaborator

We want to add various workers into stress.py that interfere with the network.

There's already an attempt to build such a worker (see local_network in the test), but that requires a certain machine setup, which can be avoided with the new proxying infrastructure.

What we want:

  1. Remove the existing network infra from the stress.py;
  2. Change local_network mode to use proxies (proxify each node, and make local_network block all messages to and from a particular node for some brief amount of time);
  3. Add a new worker drop_messages that instead of completely isolating a node just drops a certain percentage of messages between each pair of nodes;

We want eventually the following tests to pass:

pytest --timeout=2000 stress/stress.py 3 3 3 0 staking transactions local_network
pytest --timeout=2000 stress/stress.py 3 3 3 0 staking transactions drop_messages

(the arguments to stress.py are number of shrads, number of nodes initially, number of nodes max, number of observers, workers).

@SkidanovAlex SkidanovAlex added the A-chain Area: Chain, client & related label Jul 28, 2020
@SkidanovAlex SkidanovAlex added the C-epic Category: an epic label Aug 12, 2020
@SkidanovAlex
Copy link
Collaborator Author

SkidanovAlex commented Aug 12, 2020

This now only tracks local_network.

#3105 and #3106 track other network-related modes.

@SkidanovAlex
Copy link
Collaborator Author

Michael already started it here: #3076

Locally it currently fails hitting RPC timeouts.

Reassigning to Marcelo so that Michael can concentrate on mocknet tests.

@SkidanovAlex SkidanovAlex assigned SkidanovAlex and birchmd and unassigned birchmd and mfornet Aug 16, 2020
@SkidanovAlex
Copy link
Collaborator Author

The local_network mode is moderately stable in master now, closing this issue

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-chain Area: Chain, client & related
Projects
None yet
Development

No branches or pull requests

3 participants