-
-
Notifications
You must be signed in to change notification settings - Fork 2k
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
[Bug]: Error spam due to prometheus metrics trying to be updated for non-premium user #7817
Comments
@mikstew curious why prometheus is being added as a callback. What does your config look like? |
Yeah, I can't figure out why it's doing that. Here's our config:
|
We'll revert to an older version until this can get fixed. |
Hey @mikstew do you still see this on latest? i don't see this anymore |
Ahh, I had been watching this issue thread to see if a fix was merged. Let me update and check. |
We updated to helm chart version
|
Hi @krrishdholakia. Were you able to still repro after your change? We're still seeing the error. |
not seeing it - can i see your config? maybe there's a specific flow i need to run |
Here's our config:
|
What happened?
We are using a non-premium installation of LiteLLM proxy, which means the prometheus metrics are not created based on this code.
It looks like the issue is that this function call is trying to update metrics that don't exist, resulting in a bunch of error spam.
Relevant log output
Are you a ML Ops Team?
No
What LiteLLM version are you on ?
v1.58.0
Twitter / LinkedIn details
No response
The text was updated successfully, but these errors were encountered: