-
Notifications
You must be signed in to change notification settings - Fork 889
feat(cli/exp): add app testing to scaletest workspace-traffic #11633
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
Conversation
96f25c3
to
748843c
Compare
748843c
to
ef5cb9a
Compare
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.
I understand the idea of app scaletests using an echo WebSocket server, but I can see it only in tests. Do you plan to deploy a similar solution in prod?
The PR message contains an example for deploying it in prod, this is already added to a template in |
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.
👍
This implementation is quite specific to our usage and only supports app communication over WebSocket. An example command/workspace template could look like this:
coder exp scaletest workspace-traffic --template scaletest-ws --app wsec --bytes-per-tick $((1024*100)) --tick-interval 1ms --job-timeout 10s
Closes #11175 by relaxing the read/write assertion in tests, the exact byte count is unimportant since our goal is to produce load.