Skip to content
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

Fix pydantic warning in prometheus integration #7583

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

SmartManoj
Copy link
Contributor

@SmartManoj SmartManoj commented Jan 6, 2025

Fixes #7560

Remove the Config class from the UserAPIKeyLabelValues class in litellm/types/integrations/prometheus.py because the fields and values are the same and the alias is unnecessary.


For more details, open the Copilot Workspace session.

Fixes BerriAI#7560

Remove the `Config` class from the `UserAPIKeyLabelValues` class in `litellm/types/integrations/prometheus.py`.

* Remove the `fields` key in the `Config` class which was causing the pydantic warning.
* Ensure the `UserAPIKeyLabelValues` class no longer uses the `Config` class.
* Fix the pydantic warning related to the 'fields' config key removed in V2.

---

For more details, open the [Copilot Workspace session](https://copilot-workspace.githubnext.com/BerriAI/litellm/issues/7560?shareId=XXXX-XXXX-XXXX-XXXX).
Copy link

vercel bot commented Jan 6, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
litellm ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 6, 2025 0:47am

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug]: Valid config keys have changed in V2
1 participant