-
Notifications
You must be signed in to change notification settings - Fork 431
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
UI metrics missing #6168
Comments
Hi @kovacsur , do you have more logs in |
HI @martikan , Do you have something like that configured in your [telemetry]
metricsEnabled = true
tracingEnabled = true
[telemetry.Exporters]
[telemetry.Exporters.Jaeger]
HTTPCollectorEndpoint = "https://your-jaeger-server:12345"
collectorEndpoint = "https://your-jaeger-server:12345/api/traces"
samplingProbability = 0.25
serviceName = "Your-Team-Service-Here"
[telemetry.Exporters.Prometheus]
ReporteringPeriod = 10 Do you have some Feel free to reopen if needed. |
Hi @yesnault, thank you for your help. The problem really is connected to our conf.toml
Notice the last line setting Digging deeper I found a suspicious part in the source code:
I suppose that the available variables to set are specified here as exactly these variables are present in our config file but again |
@yesnault Could you please reopen the ticket? We don't have the required permissions to do so. Manually adding the (apparently no longer supported) metricsEnabled flag did not solve the issue, as described above. The issue still persists, ie. no metrics data is published by the UI component even though the same configuration works for other components. |
We tried to implement metrics collection on a CDS instance. Documentation is unfortunately lacking in this respect. It was our impression that metrics data should be exposed on the endpoint /mon/metrics which appears to be working on most components (API, hooks, etc.), but not on UI, which returns
{"id":38,"message":"resource not found","request_id":"9483067c-6d92-44ad-acd4-678a4b47a681"}
Should we be able to get metrics info from the UI component or is this intended?
The text was updated successfully, but these errors were encountered: