1340123 - Stop using --max-requests with gunicorn now datasource isn't leaking

Status: RESOLVED FIXED

(Tree Management :: Treeherder, defect, P2)

Description

[Ed Morley [:emorley]]

Bug 1312007 found datasource was leaking, which is why we had had to use the `--max-requests 150` option for gunicorn in bug 1307785.

Now that we're no longer using datasource, we should try removing `--max-requests` entirely, which should increase the performance of our web dynos, since they are not constantly restarting gunicorn worker threads.

If that shows us to still have small leaks we could always still use it but at a much higher number of requests.

Comment 1

[GitHub Autolander Bot]

Attachment: [treeherder] mozilla:gunicorn-no-max-requests > mozilla:master

Comment 2

[Ed Morley [:emorley]]

Comment on attachment 8838015 [details] [review]
[treeherder] mozilla:gunicorn-no-max-requests > mozilla:master

This is being tested on prod now (load too low on stage to be useful), and looks fine so far:
https://dashboard.heroku.com/apps/treeherder-prod/metrics/web?starting=2-hours-ago

Comment 3

[Treeherder GitHub Bugbot]

Commit pushed to master at https://github.com/mozilla/treeherder

https://github.com/mozilla/treeherder/commit/6502ec24b1b31a2cd42a6fea5aeb27243367ff12
Bug 1340123 - Stop using --max-requests with gunicorn

Since hopefully now we're no longer using datasource, the leaks should
have gone. The gunicorn processes will now only be restarted at the
daily Heroku dyno restart, rather than multiple times per minute,
improving performance.