Merge ~blake-rouse/maas:lower-max-conns-per-worker into maas:master

[Blake Rouse]

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).

Without this change just running 2 region controllers would result in too many connections to PostgreSQL.

[Newell Jensen (newell-jensen)]

LGTM

review: Approve

[MAAS Lander (maas-lander)]

LANDING
-b lower-max-conns-per-worker lp:~blake-rouse/maas into -b master lp:~maas-committers/maas

STATUS: FAILED BUILD
LOG: http://ci-maas-jenkins.internal:8080/job/maas/job/branch-tester/73/console

diff --git a/src/maasserver/utils/threads.py b/src/maasserver/utils/threads.py
index 0528d12..38a847b 100644
--- a/src/maasserver/utils/threads.py
+++ b/src/maasserver/utils/threads.py
@@ -38,7 +38,14 @@ from twisted.internet.defer import DeferredSemaphore
 
 
 max_threads_for_default_pool = 50
-max_threads_for_database_pool = 20
+
+# Maximum number of database connections this worker can use. 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).
+max_threads_for_database_pool = 9
 
 
 def make_default_pool(maxthreads=max_threads_for_default_pool):