Merge ~pwlars/testflinger/+git/testflinger-charm-reactive-layer:better-scaling into ~canonical-hw-cert/testflinger/+git/testflinger-charm-reactive-layer:master

Proposed by Paul Larson
Status: Merged
Approved by: Paul Larson
Approved revision: 3cb1b35766831f2a5ddc67d3b75b27ef5d931ad0
Merged at revision: 1b6d1dd3720196a74681f82f59d6d7a2aea859f0
Proposed branch: ~pwlars/testflinger/+git/testflinger-charm-reactive-layer:better-scaling
Merge into: ~canonical-hw-cert/testflinger/+git/testflinger-charm-reactive-layer:master
Diff against target: 22 lines (+2/-1)
2 files modified
reactive/testflinger.py (+1/-0)
templates/testflinger.service (+1/-1)
Reviewer Review Type Date Requested Status
Sheila Miguez (community) Approve
Review via email: mp+337729@code.staging.launchpad.net

This proposal supersedes a proposal from 2018-02-14.

Description of the change

Bump the number of workers a bit (4 is still quite reasonable) and use gevent. This should help us avoid timeouts due to the large increase in agents and users we've had, combined with small delays in processing requests when they require writing/reading to redis or files on the server.

These changes are running in production right, and seem to be going well so far. I'd like to codify them here though in case we ever need to redeploy.

To post a comment you must log in.
Revision history for this message
Sheila Miguez (codersquid) wrote :

+1

review: Approve

There was an error fetching revisions from git servers. Please try again in a few minutes. If the problem persists, contact Launchpad support.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches