Skip to content

Commit

Permalink
Increase the maximum number of users, after stress testing show is it…
Browse files Browse the repository at this point in the history
… working fine with 60
  • Loading branch information
jeffkaufman committed Jan 14, 2021
1 parent 32dc287 commit 65c708b
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 4 deletions.
6 changes: 3 additions & 3 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -75,7 +75,7 @@ trade-offs. This project is almost entirely limited by CPU, for
encoding and decoding audio, which means there's no reason to get an
instance with large amounts of memory.
If you want to support up to about 40 users, any single core server
If you want to support up to about 60 users, any single core server
should be fine. The public instance is running on Amazon Lightsail,
With their smallest server (512 MB RAM, 1 vCPU, 20 GB SSD,
$3.50/month).
Expand Down Expand Up @@ -127,7 +127,7 @@ sudo python3 -mpip install -r requirements.txt
### Simple Configuration
Handles up to ~40 users.
Handles up to ~60users.
In /etc/systemd/system/ create `uwsgi-echo-01.service` as:
Expand Down Expand Up @@ -158,7 +158,7 @@ location /api {
### Sharded Configuration
Handles up to ~1000 users, at ~40/core. The instructions below assume
Handles up to ~1000 users, at ~60/core. The instructions below assume
you are using a 12 core machine: one core for nginx, one core for
bucket brigade, and ten cores for the shards.
Expand Down
5 changes: 4 additions & 1 deletion server.py
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,10 @@
# best-effort basis by the client. If many people are calibrating at
# the same time this will be exceeded, because we only check before
# calibration.
MAX_USERS = 22 # XXX needs tuning
#
# In stress testing, the server seems to do fine with 61 users, but
# the video call might change that (stress test includes no video).
MAX_USERS = 35 # XXX needs tuning

try:
# Grab these on startup, when they are very very likely to be the actual
Expand Down

0 comments on commit 65c708b

Please sign in to comment.