We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
I'm migrating some rust plugins from router v1 to router v2. For metrics, I'm using the new approach with:
let meter = meter_provider().meter("apollo/router");
I'm using counter, histograms and gauge. Counters and histograms are exported in /metrics used by prometheus, as expected. Gauges are not exported.
/metrics
I still don't understand what is wrong only with Gauges. By looking at the code, I've found it that Gauge is being commented out in
Gauge
router/apollo-router/src/plugins/telemetry/config_new/instruments.rs
Line 1281 in 4c7114d
The text was updated successfully, but these errors were encountered:
reproduce issue with Gauge
046c51d
Reproduce apollographql#6859
Here is a test probably reproducing the issue #6860
Sorry, something went wrong.
In 911780c, I've tried to add support for SyncGauge, but the test is still failing.
SyncGauge
I could finally fix by adding b76d71b
Successfully merging a pull request may close this issue.
Describe the bug
I'm migrating some rust plugins from router v1 to router v2.
For metrics, I'm using the new approach with:
I'm using counter, histograms and gauge.
Counters and histograms are exported in
/metrics
used by prometheus, as expected.Gauges are not exported.
I still don't understand what is wrong only with Gauges.
By looking at the code, I've found it that
Gauge
is being commented out inrouter/apollo-router/src/plugins/telemetry/config_new/instruments.rs
Line 1281 in 4c7114d
Desktop (please complete the following information):
The text was updated successfully, but these errors were encountered: