-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
Create Sidekiq load tester #2582
Comments
Clearly visible leak with Celluloid 0.17.1.2:
|
Lookin' mighty fine, Celluloid 0.17.2:
|
Is it worth converting this into some kind of automated test as part of the suite? |
I welcome any integration-type tests if someone has an idea and wants to tackle that but it's pretty non-deterministic and it shouldn't run on Travis. I think this is a good manual test to be run occasionally; we don't upgrade Celluloid versions very often. |
The recent Celluloid 0.17 leak fiasco has shown that my rudimentary smoke testing is not sufficient. Create a script which can start up Sidekiq and then push N good/bad jobs to it while also monitoring memory and CPU usage.
Script
The script should:
Example:
The text was updated successfully, but these errors were encountered: