-
Notifications
You must be signed in to change notification settings - Fork 25
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
remove high cardinality labels server_route_id and server_gateway_id #192
Conversation
Signed-off-by: Caleb Lloyd <caleb@synadia.com>
@@ -112,132 +111,6 @@ and no additional NATS metrics when there is no connectivity to the NATS system. | |||
allows users to differentiate between a problem with the exporter itself connectivity with | |||
the NATS system. | |||
|
|||
### Scrape Output |
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.
This section is wildly out of date, so just removing it
Since there can be multiple connections per route as of nats 2.10.x, removing the route ID is likely causing an issue |
Added a commit to remap Route/Gateway IDs to indexes, which will reduce cardinality coming from config reloads where routes/gateways always get new IDs |
Signed-off-by: Caleb Lloyd <caleb@synadia.com>
ba62ef0
to
c32e1d6
Compare
54264ea
to
b9672ad
Compare
Resolves #191