-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Prometheus metrics does not work as expected. #2184
Comments
I faced the same issue using
|
Same, no metrics for me starting from 2.1.0. Everything was fine in 2.0.4 |
find it is caused by https://github.com/huggingface/text-generation-inference/pull/2129/files metric could be shown up after downgrading metrics-exporter-prometheus to v0.12.1, metric 0.23.0 could with metrics-exporter-prometheus 0.15.1 |
Missed EDIT: was included to |
System Info
Runtime environment:
Target: x86_64-unknown-linux-gnu
Cargo version: 1.79.0
Commit sha: N/A
Docker label: N/A
nvidia-smi:
N/A
xpu-smi:
N/A
Information
Tasks
Reproduction
I want to use https://github.com/huggingface/text-generation-inference/blob/main/docs/source/basic_tutorials/monitoring.md to monitor TGI case. however. I got blank output.
I use curl http://127.0.0.1:80/metrics -i to get the metrics output, find the content-length is also 0.
see
HTTP/1.1 200 OK
content-type: text/plain; charset=utf-8
vary: origin, access-control-request-method, access-control-request-headers
access-control-allow-origin: *
content-length: 0
date: Thu, 04 Jul 2024 07:46:06 GMT
which could explain why I got blank output in https://github.com/huggingface/text-generation-inference/blob/main/docs/source/basic_tutorials/monitoring.md.
this is my prometheus console. not tgi metric output either.
Expected behavior
could reproduce what's described in https://github.com/huggingface/text-generation-inference/blob/main/docs/source/basic_tutorials/monitoring.md
The text was updated successfully, but these errors were encountered: