Merge pull request #3186 from matrix-org/erikj/fix_int_values_metrics

Fix metrics that have integer value labels
This commit is contained in:
Erik Johnston 2018-05-08 09:48:50 +01:00 committed by GitHub
commit 0b7dfbb194
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23

View file

@ -71,7 +71,8 @@ class BaseMetric(object):
"""Render this metric for a single set of labels """Render this metric for a single set of labels
Args: Args:
label_values (list[str]): values for each of the labels label_values (list[object]): values for each of the labels,
(which get stringified).
value: value of the metric at with these labels value: value of the metric at with these labels
Returns: Returns:
@ -324,4 +325,4 @@ def _escape_character(m):
def _escape_label_value(value): def _escape_label_value(value):
"""Takes a label value and escapes quotes, newlines and backslashes """Takes a label value and escapes quotes, newlines and backslashes
""" """
return re.sub(r"([\n\"\\])", _escape_character, value) return re.sub(r"([\n\"\\])", _escape_character, str(value))