Merge ~blake-rouse/maas:lower-max-conns-per-worker into maas:master
Proposed by
Blake Rouse
| Status: | Merged |
|---|---|
| Approved by: | Blake Rouse |
| Approved revision: | 62dec0c9d462133639f12821f5e36d4a117370bd |
| Merge reported by: | MAAS Lander |
| Merged at revision: | not available |
| Proposed branch: | ~blake-rouse/maas:lower-max-conns-per-worker |
| Merge into: | maas:master |
| Diff against target: |
20 lines (+8/-1) 1 file modified
src/maasserver/utils/threads.py (+8/-1) |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Newell Jensen (community) | Approve | ||
|
Review via email:
|
|||
Commit message
Lower the maximum number of connections per worker to 9 connections is allowed as 1 is reserved for the `PostgresListener`.
Allowing each worker to have a maximum of 10 connections open at a time. A region controller with 4 workers will use a maximum of 40 connections. This allows 2 region controllers to run without needing to increase the maximum number of PostgreSQL connection (default is 100 connections).
Description of the change
Without this change just running 2 region controllers would result in too many connections to PostgreSQL.
To post a comment you must log in.
Revision history for this message
| MAAS Lander (maas-lander) wrote | # |
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
LGTM