Don't forbid '_' in metric basenames any more, to allow things like foo_time

1 files changed, 0 insertions, 5 deletions

diff --git a/synapse/metrics/__init__.py b/synapse/metrics/__init__.py

index c161c17e9f..f85c6418e5 100644
--- a/synapse/metrics/__init__.py
+++ b/synapse/metrics/__init__.py
@@ -43,11 +43,6 @@ class Metrics(object):
         self.name_prefix = name
 
     def _register(self, metric_class, name, *args, **kwargs):
-        if "_" in name:
-            raise ValueError("Metric names %s is invalid as it cannot contain an underscore"
-                % (name)
-            )
-
         full_name = "%s_%s" % (self.name_prefix, name)
 
         metric = metric_class(full_name, *args, **kwargs)