New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fixes #256: % formatted performance data #257
Conversation
@@ -77,7 +77,19 @@ def prepare_data(item): | |||
m = re.search(r'^(.*)\.[\d]{10}$', fields['name']) | |||
if m: | |||
fields['name'] = m.group(1) | |||
fields['name'] = fields['name'].replace(" ", "_") | |||
|
|||
# Sanitize field name for Graphite: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sanitize field name for timeserie database
, in this case it's for graphite and influxdb
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
AFAIK, Influxdb do not have such constraints on field names, as such it is not necessary to change the fields names
Oups, I was meaning the contrary of what I wrote ... whereas it is for Graphite or Influx, metrics outgoing from Alignak will have the same name
'value': fields['max'], | ||
'uom': fields['uom'] | ||
} | ||
) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1 cool ;)
Manage min and max correctly Sanitize metrics names for Graphite/Grafana Set version as 0.8.5.1
Manage min and max correctly