Fix metrics, output in ms

dominickp · Feb 14, 2024 · e7126e2 · e7126e2
1 parent ac7ed90
commit e7126e2
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/python/src/chan_client.py b/python/src/chan_client.py
@@ -24,8 +24,8 @@ def handle_request(self, method, url, headers={}):
         """
         start = time()
         chan_r = requests.request(method, url, timeout=REQUEST_TIMEOUT, headers=headers)
-        total_time = time() - start
-        METRIC_FANOUT_REQUEST_TIME.labels(method, url, chan_r.status_code).observe(total_time)
+        total_time_ms = (time() - start) * 1000
+        METRIC_FANOUT_REQUEST_TIME.labels(method, url, chan_r.status_code).observe(total_time_ms)
         return chan_r
 
     def get_catalog(self, board, headers={}):

diff --git a/python/src/main.py b/python/src/main.py
@@ -41,11 +41,11 @@ def after_request(response):
     Capture the time spent on the request and record it as a metric.
     """
     # Calculate the total time spent on the request
-    total_time = time() - g.start
+    total_time_ms = (time() - g.start) * 1000
     # Get the normalized route like "/<string:name>/<int:page>" if url_rule is present
     normalized_path = request.url_rule.rule if request.url_rule else "unmatched-route"
     # Ensure unmatched routes are recorded as "not_found"
-    METRIC_TOTAL_REQUEST_TIME.labels(request.method, normalized_path, response.status_code).observe(total_time)
+    METRIC_TOTAL_REQUEST_TIME.labels(request.method, normalized_path, response.status_code).observe(total_time_ms)
     return response
 
 @app.route("/")