Gyorgy Ruck created HDDS-5010:
---------------------------------
Summary: PrometheusMetricsSink TYPE is duplicated for
multi-dimension metrics
Key: HDDS-5010
URL: https://issues.apache.org/jira/browse/HDDS-5010
Project: Apache Ozone
Issue Type: Bug
Affects Versions: 1.0.0
Reporter: Gyorgy Ruck
Currently, all the internal metrics exposed via the *PrometheusMetricsSink* has
their own *# TYPE* definition even the metrics are repeatedly appearing in the
endpoint due to different labels, e.g.
{code:java}
# TYPE hdds_dispatcher_counter counter
hdds_dispatcher_counter{type="PutBlock",hostname="gruck-ozone-1.gruck-ozone.root.hwx.site"}
0
...
# TYPE hdds_dispatcher_counter counter
hdds_dispatcher_counter{type="ReadContainer",hostname="gruck-ozone-1.gruck-ozone.root.hwx.site"}
0
{code}
Source of this:
[https://github.com/apache/ozone/blob/9fca2a788ca0ad39bbf8abb3af2a4f733cf8de16/hadoop-hdds/framework/src/main/java/org/apache/hadoop/hdds/server/http/PrometheusMetricsSink.java#L68]
The expected output should be categorized like this, # TYPE:
[https://prometheus.io/docs/guides/multi-target-exporter/#basic-querying-of-multi-target-exporters]
{code:java}
# TYPE hdds_dispatcher_counter counter
hdds_dispatcher_counter{type="PutBlock",hostname="gruck-ozone-1.gruck-ozone.root.hwx.site"}
0
hdds_dispatcher_counter{type="ReadContainer",hostname="gruck-ozone-1.gruck-ozone.root.hwx.site"}
0
...{code}
In case we are not categorizing the same metrics with different labels under
the same # TYPE definition it could make decoders fail, like:
[https://github.com/prometheus/common/blob/90d71d7138448baa26007aa224ee917eea49f897/expfmt/text_parse.go#L511]
{code:java}
second TYPE line for metric name "hdds_dispatcher_counter", or TYPE reported
after samples
{code}
Additionally, it's not advisable to use type as a label but this is not an
issue from the decoder POV:
https://prometheus.io/docs/instrumenting/writing_exporters/#labels
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
