Skip to content

Commit

Permalink
Include statsd plugin by default
Browse files Browse the repository at this point in the history
  • Loading branch information
leplatrem committed Oct 7, 2024
1 parent 9971e17 commit 51b2ef0
Show file tree
Hide file tree
Showing 3 changed files with 8 additions and 1 deletion.
1 change: 0 additions & 1 deletion kinto/core/__init__.py
Original file line number Diff line number Diff line change
Expand Up @@ -66,7 +66,6 @@
"kinto.core.initialization.setup_authentication",
"kinto.core.initialization.setup_backoff",
"kinto.core.initialization.setup_sentry",
"kinto.core.initialization.setup_statsd",
"kinto.core.initialization.setup_listeners",
"kinto.core.events.setup_transaction_hook",
),
Expand Down
3 changes: 3 additions & 0 deletions kinto/core/initialization.py
Original file line number Diff line number Diff line change
Expand Up @@ -613,3 +613,6 @@ def initialize(config, version=None, settings_prefix="", default_settings=None):
# Include kinto.core views with the correct api version prefix.
config.include("kinto.core", route_prefix=api_version)
config.route_prefix = api_version

# While statsd is deprecated, we include its plugin by default.
config.include("kinto.plugins.statsd")
5 changes: 5 additions & 0 deletions tests/core/test_initialization.py
Original file line number Diff line number Diff line change
Expand Up @@ -318,6 +318,11 @@ def test_statsd_is_instanciated_from_plugin(self):
)
self.mocked.assert_called_with("host", 8080, "kinto.core")

def test_statsd_is_included_by_default(self):
kinto.core.initialize(self.config, "0.0.1", "name")

self.mocked.assert_called_with("host", 8080, "kinto.core")


class RequestsConfigurationTest(unittest.TestCase):
def _get_app(self, settings={}):
Expand Down

0 comments on commit 51b2ef0

Please sign in to comment.