kinto 20.6.1__tar.gz → 21.1.0__tar.gz

{kinto-20.6.1 → kinto-21.1.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: kinto
-Version: 20.6.1
+Version: 21.1.0
 Summary: Kinto Web Service - Store, Sync, Share, and Self-Host.
 Author-email: Mozilla Services <developers@kinto-storage.org>
 License: Copyright 2012 - Mozilla Foundation

{kinto-20.6.1 → kinto-21.1.0}/app.wsgi

@@ -8,6 +8,8 @@ here = os.path.dirname(__file__)
 
 ini_path = os.environ.get('KINTO_INI')
 if ini_path is None:
+    # WARNING: if you modify this default value, you should
+    # also change the default in `kinto.config.config_attributes()`
     ini_path = os.path.join(here, 'config', 'kinto.ini')
 
 # Set up logging

{kinto-20.6.1 → kinto-21.1.0}/constraints.txt

@@ -149,7 +149,7 @@ rpds-py==0.17.1
     # via
     #   jsonschema
     #   referencing
-ruff==0.11.5
+ruff==0.11.6
     # via -r constraints.in
 sentry-sdk==2.8.0
     # via -r constraints.in

{kinto-20.6.1 → kinto-21.1.0}/docs/api/1.x/utilities.rst

@@ -25,6 +25,11 @@ The returned value is a JSON mapping containing:
   - ``readonly``: Only requests with read operations are allowed.
 
 - ``capabilities``: a mapping used by clients to detect optional features of the API.
+- ``config``: attributes of the configuration file used by the server.
+
+  - ``path``: path on the server
+  - ``hash``: SHA256 hash of the file
+  - ``modified``: datetime of the file
 
   - Example:
 

{kinto-20.6.1 → kinto-21.1.0}/docs/api/index.rst

@@ -11,6 +11,12 @@ API
 Changelog
 ---------
 
+1.23 (2025-04-16)
+'''''''''''''''''
+
+- Added ``config`` field to the root URL endpoint
+
+
 1.22 (2019-03-20)
 '''''''''''''''''
 

{kinto-20.6.1 → kinto-21.1.0}/docs/configuration/settings.rst

@@ -513,6 +513,12 @@ Prometheus metrics can be enabled with (disabled by default):
 
     # kinto.prometheus_prefix = kinto-prod
 
+    # Expose metrics created time (default: true)
+    # kinto.prometheus_created_metrics_enabled = false
+
+    # Exclude certain labels to reduce cardinality (default: none)
+    # kinto.prometheus_exclude_labels = record_id group_id
+
 Metrics can then be crawled from the ``/__metrics__`` endpoint.
 
 

{kinto-20.6.1 → kinto-21.1.0}/kinto/__init__.py

@@ -14,7 +14,7 @@ from kinto.core import utils
 __version__ = pkg_resources.get_distribution(__package__).version
 
 # Implemented HTTP API Version
-HTTP_API_VERSION = "1.22"
+HTTP_API_VERSION = "1.23"
 
 # Main kinto logger
 logger = logging.getLogger(__name__)

{kinto-20.6.1 → kinto-21.1.0}/kinto/config/__init__.py

@@ -1,6 +1,9 @@
 import codecs
 import logging
 import os
+from datetime import datetime
+from functools import lru_cache
+from hashlib import sha256
 from time import strftime
 
 from kinto import __version__
@@ -69,3 +72,23 @@ def init(config_file, backend, cache_backend, host="127.0.0.1"):
     values.update(cache_backend_to_values[cache_backend])
 
     render_template("kinto.tpl", config_file, **values)
+
+
+@lru_cache(maxsize=1)
+def config_attributes():
+    """
+    Returns a hash of the config `.ini` file content.
+    The path is only known from `app.wsgi`, so we have to read
+    the environment variable again. Since tests are not run through
+    WSGI, then the variable is not set.
+    """
+    # WARNING: this default value should be the same as `app.wsgi`
+    ini_path = os.environ.get("KINTO_INI", os.path.join(".", "config", "kinto.ini"))
+    if not os.path.exists(ini_path):
+        logger.error(f"Could not find config file at {ini_path}")
+        return None
+    return {
+        "path": ini_path,
+        "hash": sha256(open(ini_path, "rb").read()).hexdigest(),
+        "modified": datetime.fromtimestamp(os.path.getmtime(ini_path)).isoformat(),
+    }

{kinto-20.6.1 → kinto-21.1.0}/kinto/core/initialization.py

@@ -1,7 +1,6 @@
 import logging
 import random
 import re
-import urllib.parse
 import warnings
 from datetime import datetime
 
@@ -466,14 +465,6 @@ def setup_metrics(config):
         request = event.request
         metrics_service = config.registry.metrics
 
-        try:
-            endpoint = utils.strip_uri_prefix(request.path)
-            endpoint = urllib.parse.quote_plus(endpoint, safe="/?&=-_")
-        except UnicodeDecodeError as e:
-            # This `on_new_response` callback is also called when a HTTP 400
-            # is returned because of an invalid UTF-8 path. We still want metrics.
-            endpoint = str(e)
-
         # Count unique users.
         user_id = request.prefixed_userid
         if user_id:
@@ -496,13 +487,27 @@ def setup_metrics(config):
             (field, enhanced_matchdict.get(field, "")) for field in metrics_matchdict_fields
         ]
 
+        status = event.response.status_code
+
+        service = request.current_service
+        if service:
+            # Use the service name as endpoint if available.
+            endpoint = service.name
+        elif route := request.matched_route:
+            # Use the route name as endpoint if we're not on a Cornice service.
+            endpoint = route.name
+        else:
+            endpoint = (
+                "unnamed" if status != 404 else "unknown"
+            )  # Do not multiply cardinality for unknown endpoints.
+
         # Count served requests.
         metrics_service.count(
             "request_summary",
             unique=[
                 ("method", request.method.lower()),
                 ("endpoint", endpoint),
-                ("status", str(event.response.status_code)),
+                ("status", str(status)),
             ]
             + metrics_matchdict_labels,
         )
@@ -510,10 +515,11 @@ def setup_metrics(config):
         try:
             current = utils.msec_time()
             duration = current - request._received_at
-            metrics_service.observe(
+            metrics_service.timer(
                 "request_duration",
-                duration,
-                labels=[("endpoint", endpoint)] + metrics_matchdict_labels,
+                value=duration,
+                labels=[("endpoint", endpoint), ("method", request.method.lower())]
+                + metrics_matchdict_labels,
             )
         except AttributeError:  # pragma: no cover
             # Logging was not setup in this Kinto app (unlikely but possible)
@@ -527,13 +533,11 @@ def setup_metrics(config):
         )
 
         # Count authentication verifications.
-        if hasattr(request, "authn_type"):
-            metrics_service.count(f"authn_type.{request.authn_type}")
-
-        # Count view calls.
-        service = request.current_service
-        if service:
-            metrics_service.count(f"view.{service.name}.{request.method}")
+        try:
+            metrics_service.count("authentication", unique=[("type", request.authn_type)])
+        except AttributeError:
+            # Not authenticated
+            pass
 
     config.add_subscriber(on_new_response, NewResponse)
 

{kinto-20.6.1 → kinto-21.1.0}/kinto/core/metrics.py

@@ -47,7 +47,7 @@ class NoOpTimer:
 
 @implementer(IMetricsService)
 class NoOpMetricsService:
-    def timer(self, key):
+    def timer(self, key, value=None, labels=[]):
         return NoOpTimer()
 
     def observe(self, key, value, labels=[]):
@@ -65,11 +65,12 @@ def watch_execution_time(metrics_service, obj, prefix="", classname=None):
     classname = classname or utils.classname(obj)
     members = dir(obj)
     for name in members:
-        value = getattr(obj, name)
-        is_method = isinstance(value, types.MethodType)
+        method = getattr(obj, name)
+        is_method = isinstance(method, types.MethodType)
         if not name.startswith("_") and is_method:
-            statsd_key = f"{prefix}.{classname}.{name}"
-            decorated_method = metrics_service.timer(statsd_key)(value)
+            statsd_key = f"{prefix}.{classname}"
+            labels = [("method", name)]
+            decorated_method = metrics_service.timer(statsd_key, labels=labels)(method)
             setattr(obj, name, decorated_method)
 
 

{kinto-20.6.1 → kinto-21.1.0}/kinto/core/views/hello.py

@@ -2,6 +2,7 @@ import colander
 from pyramid.authorization import Authenticated
 from pyramid.security import NO_PERMISSION_REQUIRED
 
+from kinto.config import config_attributes
 from kinto.core import Service
 
 
@@ -26,14 +27,17 @@ hello_response_schemas = {
 def get_hello(request):
     """Return information regarding the current instance."""
     settings = request.registry.settings
+
     project_name = settings["project_name"]
     project_version = settings["project_version"]
+
     data = dict(
         project_name=project_name,
         project_version=project_version,
         http_api_version=settings["http_api_version"],
         project_docs=settings["project_docs"],
         url=request.route_url(hello.name),
+        config=config_attributes(),
     )
 
     eos = get_eos(request)

{kinto-20.6.1 → kinto-21.1.0}/kinto/plugins/prometheus.py

@@ -6,6 +6,7 @@ from time import perf_counter as time_now
 
 from pyramid.exceptions import ConfigurationError
 from pyramid.response import Response
+from pyramid.settings import asbool, aslist
 from zope.interface import implementer
 
 from kinto.core import metrics
@@ -49,10 +50,27 @@ def _fix_metric_name(s):
 
 
 class Timer:
-    def __init__(self, summary):
-        self.summary = summary
+    """
+    A decorator to time the execution of a function. It will use the
+    `prometheus_client.Histogram` to record the time taken by the function
+    in milliseconds. The histogram is passed as an argument to the
+    constructor.
+
+    Main limitation: it does not support `labels` on the decorator.
+    """
+
+    def __init__(self, histogram):
+        self.histogram = histogram
         self._start_time = None
 
+    def set_labels(self, labels):
+        if not labels:
+            return
+        self.histogram = self.histogram.labels(*(label_value for _, label_value in labels))
+
+    def observe(self, value):
+        return self.histogram.observe(value)
+
     def __call__(self, f):
         @safe_wraps(f)
         def _wrapped(*args, **kwargs):
@@ -61,7 +79,7 @@ class Timer:
                 return f(*args, **kwargs)
             finally:
                 dt_ms = 1000.0 * (time_now() - start_time)
-                self.summary.observe(dt_ms)
+                self.histogram.observe(dt_ms)
 
         return _wrapped
 
@@ -79,13 +97,13 @@ class Timer:
         if self._start_time is None:  # pragma: nocover
             raise RuntimeError("Timer has not started.")
         dt_ms = 1000.0 * (time_now() - self._start_time)
-        self.summary.observe(dt_ms)
+        self.histogram.observe(dt_ms)
         return self
 
 
 @implementer(metrics.IMetricsService)
 class PrometheusService:
-    def __init__(self, prefix=""):
+    def __init__(self, prefix="", exclude_labels=None):
         prefix_clean = ""
         if prefix:
             # In GCP Console, the metrics are grouped by the first
@@ -94,26 +112,49 @@ class PrometheusService:
             # (eg. `remote-settings` -> `remotesettings_`, `kinto_` -> `kinto_`)
             prefix_clean = _fix_metric_name(prefix).replace("_", "") + "_"
         self.prefix = prefix_clean.lower()
+        self.exclude_labels = exclude_labels or []
+
+    def _exclude_labels(self, labels):
+        return [
+            (label_name, label_value)
+            for label_name, label_value in labels
+            if label_name not in self.exclude_labels
+        ]
 
-    def timer(self, key):
+    def timer(self, key, value=None, labels=[]):
         global _METRICS
         key = self.prefix + key
+        labels = self._exclude_labels(labels)
 
         if key not in _METRICS:
-            _METRICS[key] = prometheus_module.Summary(
-                _fix_metric_name(key), f"Summary of {key}", registry=get_registry()
+            _METRICS[key] = prometheus_module.Histogram(
+                _fix_metric_name(key),
+                f"Histogram of {key}",
+                registry=get_registry(),
+                labelnames=[label_name for label_name, _ in labels],
             )
 
-        if not isinstance(_METRICS[key], prometheus_module.Summary):
+        if not isinstance(_METRICS[key], prometheus_module.Histogram):
             raise RuntimeError(
                 f"Metric {key} already exists with different type ({_METRICS[key]})"
             )
 
-        return Timer(_METRICS[key])
+        timer = Timer(_METRICS[key])
+        timer.set_labels(labels)
+
+        if value is not None:
+            # We are timing something.
+            return timer.observe(value)
+
+        # We are not timing anything, just returning the timer object
+        # (eg. to be used as decorator or context manager).
+        # Note that in this case, the labels values will be the same for all calls.
+        return timer
 
     def observe(self, key, value, labels=[]):
         global _METRICS
         key = self.prefix + key
+        labels = self._exclude_labels(labels)
 
         if key not in _METRICS:
             _METRICS[key] = prometheus_module.Summary(
@@ -154,6 +195,7 @@ class PrometheusService:
                 label_name, label_value = unique.rsplit(".", 1)
                 unique = [(label_name, label_value)]
 
+            unique = self._exclude_labels(unique)
             labels = [
                 (_fix_metric_name(label_name), label_value) for label_name, label_value in unique
             ]
@@ -199,6 +241,11 @@ def includeme(config):
         )
         raise ConfigurationError(error_msg)
 
+    settings = config.get_settings()
+
+    if not asbool(settings.get("prometheus_created_metrics_enabled", True)):
+        prometheus_module.disable_created_metrics()
+
     config.add_api_capability(
         "prometheus",
         description="Prometheus metrics.",
@@ -220,7 +267,13 @@ def includeme(config):
             pass
     _METRICS.clear()
 
-    settings = config.get_settings()
     prefix = settings.get("prometheus_prefix", settings["project_name"])
 
-    config.registry.registerUtility(PrometheusService(prefix=prefix), metrics.IMetricsService)
+    # If we want to reduce the metrics cardinality, we can exclude certain
+    # labels (eg. records_id). This way all metrics will be grouped by the
+    # remaining labels.
+    exclude_labels = aslist(settings.get("prometheus_exclude_labels", ""))
+
+    config.registry.registerUtility(
+        PrometheusService(prefix=prefix, exclude_labels=exclude_labels), metrics.IMetricsService
+    )

{kinto-20.6.1 → kinto-21.1.0}/kinto/plugins/statsd.py

@@ -1,4 +1,5 @@
 import warnings
+from datetime import timedelta
 from urllib.parse import urlparse
 
 from pyramid.exceptions import ConfigurationError
@@ -26,7 +27,13 @@ class StatsDService:
     def __init__(self, host, port, prefix):
         self._client = statsd_module.StatsClient(host, port, prefix=prefix)
 
-    def timer(self, key):
+    def timer(self, key, value=None, labels=[]):
+        if labels:
+            # [("method", "get")] -> "method.get"
+            key = f"{key}." + ".".join(f"{label[0]}.{sanitize(label[1])}" for label in labels)
+        if value:
+            value = timedelta(seconds=value)
+            return self._client.timing(key, value)
         return self._client.timer(key)
 
     def observe(self, key, value, labels=[]):

{kinto-20.6.1 → kinto-21.1.0}/kinto.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: kinto
-Version: 20.6.1
+Version: 21.1.0
 Summary: Kinto Web Service - Store, Sync, Share, and Self-Host.
 Author-email: Mozilla Services <developers@kinto-storage.org>
 License: Copyright 2012 - Mozilla Foundation

{kinto-20.6.1 → kinto-21.1.0}/tests/core/test_initialization.py

@@ -386,7 +386,7 @@ class MetricsConfigurationTest(unittest.TestCase):
         kinto.core.initialize(self.config, "0.0.1", "settings_prefix")
         app = webtest.TestApp(self.config.make_wsgi_app())
         app.get("/v0/", headers=get_user_headers("bob"))
-        self.mocked().count.assert_any_call("authn_type.basicauth")
+        self.mocked().count.assert_any_call("authentication", unique=[("type", "basicauth")])
 
     #
     # Endpoints.
@@ -404,7 +404,7 @@ class MetricsConfigurationTest(unittest.TestCase):
         app.get("/v0/__heartbeat__")
         self.mocked().count.assert_any_call(
             "request_summary",
-            unique=[("method", "get"), ("endpoint", "/__heartbeat__"), ("status", "200")],
+            unique=[("method", "get"), ("endpoint", "heartbeat"), ("status", "200")],
         )
 
     def test_statsd_sanitizes_url_in_metrics(self):
@@ -420,7 +420,7 @@ class MetricsConfigurationTest(unittest.TestCase):
                 ("method", "get"),
                 (
                     "endpoint",
-                    "/changeset%27%257C%2522%253F%253E%253C%21DOCTYPE%2522http%3A//xh3E%27%29%2C%27/l%27%29%2520from%2520dual%29%257C%27",
+                    "unknown",
                 ),
                 ("status", "404"),
             ],
@@ -433,25 +433,19 @@ class MetricsConfigurationTest(unittest.TestCase):
         self.mocked().observe.assert_any_call(
             "request_size",
             len("{}"),
-            labels=[("endpoint", "/__heartbeat__")],
+            labels=[("endpoint", "heartbeat")],
         )
 
     def test_statsd_observe_request_duration(self):
         kinto.core.initialize(self.config, "0.0.1", "settings_prefix")
         app = webtest.TestApp(self.config.make_wsgi_app())
         app.get("/v0/__heartbeat__")
-        self.mocked().observe.assert_any_call(
+        self.mocked().timer.assert_any_call(
             "request_duration",
-            mock.ANY,
-            labels=[("endpoint", "/__heartbeat__")],
+            value=mock.ANY,
+            labels=[("endpoint", "heartbeat"), ("method", "get")],
         )
 
-    def test_statsd_counts_views_and_methods(self):
-        kinto.core.initialize(self.config, "0.0.1", "settings_prefix")
-        app = webtest.TestApp(self.config.make_wsgi_app())
-        app.get("/v0/__heartbeat__")
-        self.mocked().count.assert_any_call("view.heartbeat.GET")
-
     def test_statsd_counts_unknown_urls(self):
         kinto.core.initialize(self.config, "0.0.1", "settings_prefix")
         app = webtest.TestApp(self.config.make_wsgi_app())

{kinto-20.6.1 → kinto-21.1.0}/tests/core/test_metrics.py

@@ -24,7 +24,9 @@ class WatchExecutionTimeTest(unittest.TestCase):
 
     def test_public_methods_generates_statsd_calls(self):
         self.test_object.test_method()
-        self.mocked.timer.assert_called_with("test.testedclass.test_method")
+        self.mocked.timer.assert_called_with(
+            "test.testedclass", labels=[("method", "test_method")]
+        )
 
     def test_private_methods_does_not_generates_statsd_calls(self):
         self.test_object._private_method()

{kinto-20.6.1 → kinto-21.1.0}/tests/core/test_views_hello.py

@@ -1,7 +1,9 @@
+import os
 from unittest import mock
 
 from pyramid import testing
 
+from kinto.config import config_attributes
 from kinto.core.testing import unittest
 
 from .support import BaseWebTest
@@ -75,6 +77,22 @@ class HelloViewTest(BaseWebTest, unittest.TestCase):
 
         self.assertTrue(response.json["http_api_version"])
 
+    def test_return_config_file_info(self):
+        config_attributes.cache_clear()
+        before = os.getenv("KINTO_INI", None)
+        os.environ["KINTO_INI"] = "tests/test_configuration/test.ini"
+
+        response = self.app.get("/")
+
+        self.assertEqual(response.json["config"]["hash"], mock.ANY)
+        self.assertEqual(response.json["config"]["modified"], mock.ANY)
+        self.assertEqual(response.json["config"]["path"], mock.ANY)
+
+        if before is None:
+            del os.environ["KINTO_INI"]
+        else:
+            os.environ["KINTO_INI"] = before
+
 
 class APICapabilitiesTest(BaseWebTest, unittest.TestCase):
     def test_list_of_capabilities_contains_basicauth_by_default(self):

{kinto-20.6.1 → kinto-21.1.0}/tests/plugins/test_prometheus.py

@@ -60,7 +60,7 @@ class ServiceTest(PrometheusWebTest):
             self.assertEqual(my_func(1, 1), 2)
 
         resp = self.app.get("/__metrics__")
-        self.assertIn("TYPE kintoprod_func_latency_context summary", resp.text)
+        self.assertIn("TYPE kintoprod_func_latency_context histogram", resp.text)
 
     def test_timer_can_be_used_as_decorator(self):
         decorated = self.app.app.registry.metrics.timer("func.latency.decorator")(my_func)
@@ -68,7 +68,7 @@ class ServiceTest(PrometheusWebTest):
         self.assertEqual(decorated(1, 1), 2)
 
         resp = self.app.get("/__metrics__")
-        self.assertIn("TYPE kintoprod_func_latency_decorator summary", resp.text)
+        self.assertIn("TYPE kintoprod_func_latency_decorator histogram", resp.text)
 
     def test_timer_can_be_used_as_decorator_on_partial_function(self):
         partial = functools.partial(my_func, 3)
@@ -77,7 +77,7 @@ class ServiceTest(PrometheusWebTest):
         self.assertEqual(decorated(3), 6)
 
         resp = self.app.get("/__metrics__")
-        self.assertIn("TYPE kintoprod_func_latency_partial summary", resp.text)
+        self.assertIn("TYPE kintoprod_func_latency_partial histogram", resp.text)
 
     def test_observe_a_single_value(self):
         self.app.app.registry.metrics.observe("price", 111)
@@ -157,3 +157,49 @@ class PrometheusNoPrefixTest(PrometheusWebTest):
 
         resp = self.app.get("/__metrics__")
         self.assertIn("TYPE price summary", resp.text)
+
+
+@skip_if_no_prometheus
+class PrometheusNoCreatedTest(PrometheusWebTest):
+    @classmethod
+    def get_app_settings(cls, extras=None):
+        settings = super().get_app_settings(extras)
+        settings["prometheus_created_metrics_enabled"] = "false"
+        return settings
+
+    def test_metrics_created_not_in_response(self):
+        self.app.app.registry.metrics.observe("price", 111)
+
+        resp = self.app.get("/__metrics__")
+
+        self.assertIn("TYPE kintoprod_price summary", resp.text)
+        self.assertNotIn("TYPE kintoprod_price_created summary", resp.text)
+
+
+@skip_if_no_prometheus
+class PrometheusExcludedLabelsTest(PrometheusWebTest):
+    @classmethod
+    def get_app_settings(cls, extras=None):
+        settings = super().get_app_settings(extras)
+        settings["prometheus_exclude_labels"] = "record_id group_id"
+        return settings
+
+    def test_metrics_excluded_labels(self):
+        headers = get_user_headers("aaa")
+        self.app.put("/buckets/bid", headers=headers)
+        self.app.put("/buckets/bid/collections/cid", headers=headers)
+        self.app.put("/buckets/bid/groups/gid", headers=headers)
+        self.app.put("/buckets/bid/collections/cid/records/rid", headers=headers)
+
+        resp = self.app.get("/__metrics__")
+
+        self.assertNotIn("group_id=", resp.text)
+        self.assertNotIn("record_id=", resp.text)
+        self.assertIn(
+            'kintoprod_request_size_count{bucket_id="bid",collection_id="",endpoint="group-object"}',
+            resp.text,
+        )
+        self.assertIn(
+            'kintoprod_request_size_count{bucket_id="bid",collection_id="cid",endpoint="record-object"}',
+            resp.text,
+        )

{kinto-20.6.1 → kinto-21.1.0}/tests/test_views_metrics.py

@@ -26,20 +26,38 @@ class ViewsMetricsTest(BaseWebTest, unittest.TestCase):
         self.app.put("/buckets/beers/collections/barley", headers=self.headers)
         self.app.put("/buckets/beers/collections/barley/records/abc", headers=self.headers)
 
+        self.app.get("/buckets", headers=self.headers)
+        self.app.get("/buckets/beers/collections", headers=self.headers)
+        self.app.get("/buckets/beers/collections/barley/records", headers=self.headers)
+
         resp = self.app.get("/__metrics__")
+        print(resp.text)
+        self.assertIn(
+            'request_size_sum{bucket_id="beers",collection_id="",endpoint="bucket-object",group_id="",record_id=""}',
+            resp.text,
+        )
         self.assertIn(
-            'request_size_sum{bucket_id="beers",collection_id="",endpoint="/buckets/beers",group_id="",record_id=""}',
+            'request_size_sum{bucket_id="beers",collection_id="",endpoint="group-object",group_id="amateurs",record_id=""}',
             resp.text,
         )
         self.assertIn(
-            'request_size_sum{bucket_id="beers",collection_id="",endpoint="/buckets/beers/groups/amateurs",group_id="amateurs",record_id=""}',
+            'request_summary_total{bucket_id="beers",collection_id="barley",endpoint="collection-object",group_id="",method="put",record_id="",status="201"}',
+            resp.text,
+        )
+        self.assertIn(
+            'request_duration_sum{bucket_id="beers",collection_id="barley",endpoint="record-object",group_id="",method="put",record_id="abc"}',
+            resp.text,
+        )
+
+        self.assertIn(
+            'request_summary_total{bucket_id="",collection_id="",endpoint="bucket-plural",group_id="",method="get",record_id="",status="200"}',
             resp.text,
         )
         self.assertIn(
-            'request_summary_total{bucket_id="beers",collection_id="barley",endpoint="/buckets/beers/collections/barley",group_id="",method="put",record_id="",status="201"}',
+            'request_summary_total{bucket_id="beers",collection_id="",endpoint="collection-plural",group_id="",method="get",record_id="",status="200"}',
             resp.text,
         )
         self.assertIn(
-            'request_duration_sum{bucket_id="beers",collection_id="barley",endpoint="/buckets/beers/collections/barley/records/abc",group_id="",record_id="abc"}',
+            'request_summary_total{bucket_id="beers",collection_id="barley",endpoint="record-plural",group_id="",method="get",record_id="",status="200"}',
             resp.text,
         )