Merge pull request #99 from zalando/fix-metrics2

Use valid lizzy tag value
zalando-stups · Mar 2, 2017 · a4aff0f · a4aff0f
2 parents 754eab7 + 9f4f593
commit a4aff0f
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 2 deletions.
diff --git a/lizzy_client/metrics.py b/lizzy_client/metrics.py
@@ -3,6 +3,8 @@
 This module only works if metricz is installed
 """
 
+from urllib.parse import urlparse
+
 try:
     import metricz
 except ImportError:
@@ -23,9 +25,15 @@ def report_metric(metric_name: str, value: int, fail_silently: bool=True):
 
     configuration = Configuration()
 
+    try:
+        lizzy_domain = urlparse(configuration.lizzy_url).netloc
+        lizzy_name, _ = lizzy_domain.split('.', 1)
+    except:
+        lizzy_name = 'UNKNOWN'
+
     tags = {
         'version': VERSION,
-        'lizzy': configuration.lizzy_url
+        'lizzy': configuration.lizzy_name
     }
 
     # noinspection PyBroadException

diff --git a/lizzy_client/version.py b/lizzy_client/version.py
@@ -1,4 +1,4 @@
 MAJOR_VERSION = 2
 MINOR_VERSION = 7
-REVISION = 1
+REVISION = 2
 VERSION = "{MAJOR_VERSION}.{MINOR_VERSION}.{REVISION}".format_map(locals())