-
Notifications
You must be signed in to change notification settings - Fork 64
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
Configurable network limits #82
Comments
Sharing some experiences here, perhaps that's useful. I'm using Rally to run some scale tests. Rally uses multiple Thespian actors with With this amount of sockets used by After removal of those logs the
After adjusting the log level below Expected state (wake-up every second + additional
Unhealthy state (at one point periodic wake-ups just stop):
At the time the first missing wake-up should have been received
Edit: TX-only mode is unavoidable in current Rally implementation, as there are 2 messages sent from It would be useful to have some warning or error in that case. Rally schedules next wake-up as part of processing of the previous wake-up. That's fragile - even a single wake-up drop breaks this process. Is there a better way? Anyways, after lifting the limits, TX-only mode is avoided so wake-ups work correctly. |
Configurability has been added and merged and will be released soon. I'd like to leave this open to think more about the issues raised by @gbanasiak above. |
Async transport has the following hard-coded limits:
Thespian/thespian/system/transport/asyncTransportBase.py
Lines 37 to 40 in acf5cb2
In setups with high number of actors these arbitrary limits might be too small for the actor system to operate correctly. I'd like to have an option to modify these values through environment variables.
There's an existing PR for this: #75
The text was updated successfully, but these errors were encountered: