Skip to content

add request_name to src_graphql_field_seconds metric

Warren Gifford requested to merge sg/request-name into master

Created by: slimsag

This is needed to support #9986 I mistakenly thought this metric already included request_name because of its large cardinality, but I was wrong.

This increases the cardinality of this metric by 1, so we will now have 1802 unique time series. This is already down from 2500 unique time series thanks to my earlier change in #9895 (closed)

Additionally, to reduce risk here further since this is a last-minute change, I have reduced the cardinality of that metric even further to guarantee this change is safe.

  • Before: 1800 unique time series in src_graphql_field_seconds
  • After: 1569

This was done by excluding some GraphQL types that are generally useless (just data storage and without any computation).

Merge request reports