Merge ~cgrabowski/maas:add_rpc_conn_pool_metric into maas:master

Proposed by Christian Grabowski
Status: Merged
Approved by: Christian Grabowski
Approved revision: ecff69b7c7dc52bb947ef85f40dc7e6f4c2f66d4
Merge reported by: MAAS Lander
Merged at revision: not available
Proposed branch: ~cgrabowski/maas:add_rpc_conn_pool_metric
Merge into: maas:master
Diff against target: 40 lines (+11/-0)
2 files modified
src/provisioningserver/prometheus/metrics.py (+9/-0)
src/provisioningserver/rpc/connectionpool.py (+2/-0)
Reviewer Review Type Date Requested Status
MAAS Lander Approve
Anton Troyanov Approve
Alexsander de Souza Approve
Review via email: mp+429985@code.launchpad.net

Commit message

add metric counting number of times RPC pool scales to max size

To post a comment you must log in.
Revision history for this message
MAAS Lander (maas-lander) wrote :

UNIT TESTS
-b add_rpc_conn_pool_metric lp:~cgrabowski/maas/+git/maas into -b master lp:~maas-committers/maas

STATUS: FAILED
LOG: http://maas-ci.internal:8080/job/maas-tester/634/consoleText
COMMIT: 49c5020e79abb31fe768ee724186659b7c0dcfe6

review: Needs Fixing
Revision history for this message
Alexsander de Souza (alexsander-souza) wrote :

+1

review: Approve
Revision history for this message
MAAS Lander (maas-lander) wrote :

UNIT TESTS
-b add_rpc_conn_pool_metric lp:~cgrabowski/maas/+git/maas into -b master lp:~maas-committers/maas

STATUS: FAILED
LOG: http://maas-ci.internal:8080/job/maas-tester/636/consoleText
COMMIT: 72e51acb143aefc89da985a09083eefe5c37e0fb

review: Needs Fixing
Revision history for this message
MAAS Lander (maas-lander) wrote :

UNIT TESTS
-b add_rpc_conn_pool_metric lp:~cgrabowski/maas/+git/maas into -b master lp:~maas-committers/maas

STATUS: FAILED
LOG: http://maas-ci.internal:8080/job/maas-tester/648/consoleText
COMMIT: ecff69b7c7dc52bb947ef85f40dc7e6f4c2f66d4

review: Needs Fixing
Revision history for this message
Anton Troyanov (troyanov) wrote :

+1

review: Approve
Revision history for this message
Christian Grabowski (cgrabowski) wrote :

jenkins: !test

Revision history for this message
MAAS Lander (maas-lander) wrote :

UNIT TESTS
-b add_rpc_conn_pool_metric lp:~cgrabowski/maas/+git/maas into -b master lp:~maas-committers/maas

STATUS: SUCCESS
COMMIT: ecff69b7c7dc52bb947ef85f40dc7e6f4c2f66d4

review: Approve
Revision history for this message
MAAS Lander (maas-lander) wrote :

LANDING
-b add_rpc_conn_pool_metric lp:~cgrabowski/maas/+git/maas into -b master lp:~maas-committers/maas

STATUS: FAILED BUILD
LOG: http://maas-ci.internal:8080/job/maas-tester/674/consoleText

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/src/provisioningserver/prometheus/metrics.py b/src/provisioningserver/prometheus/metrics.py
2index 8f117ff..bc2e6bb 100644
3--- a/src/provisioningserver/prometheus/metrics.py
4+++ b/src/provisioningserver/prometheus/metrics.py
5@@ -108,6 +108,15 @@ METRICS_DEFINITIONS = [
6 "maas_lxd_fetch_machine_failure",
7 "failures for fetching LXD machines",
8 ),
9+ MetricDefinition(
10+ "Counter",
11+ "maas_rpc_pool_exhaustion_count",
12+ """
13+ counts the number of occurances of the RPC
14+ connection pool allocate its maxmimum number
15+ of connections
16+ """,
17+ ),
18 # Common metrics
19 *node_metrics_definitions(),
20 ]
21diff --git a/src/provisioningserver/rpc/connectionpool.py b/src/provisioningserver/rpc/connectionpool.py
22index 3acaf08..55b00cb 100644
23--- a/src/provisioningserver/rpc/connectionpool.py
24+++ b/src/provisioningserver/rpc/connectionpool.py
25@@ -8,6 +8,7 @@ import random
26 from twisted.internet.defer import inlineCallbacks
27 from twisted.internet.endpoints import connectProtocol, TCP6ClientEndpoint
28
29+from provisioningserver.prometheus.metrics import PROMETHEUS_METRICS
30 from provisioningserver.rpc import exceptions
31
32
33@@ -80,6 +81,7 @@ class ConnectionPool:
34 def stage_connection(self, eventloop, connection):
35 self.try_connections[eventloop] = connection
36
37+ @PROMETHEUS_METRICS.failure_counter("maas_rpc_pool_exhaustion_count")
38 @inlineCallbacks
39 def scale_up_connections(self):
40 for ev, ev_conns in self.connections.items():

Subscribers

People subscribed via source and target branches