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

[Bug]: Better handling for validation of tenant/db errors #1480

Open
HammadB opened this issue Dec 8, 2023 · 0 comments
Open

[Bug]: Better handling for validation of tenant/db errors #1480

HammadB opened this issue Dec 8, 2023 · 0 comments
Assignees
Labels
bug Something isn't working p0 taz-sprint-4

Comments

@HammadB
Copy link
Collaborator

HammadB commented Dec 8, 2023

What happened?

Generic errors in the validation step aren't clearly reported

Versions

We should clearly report them

Relevant log output

No response

@HammadB HammadB added bug Something isn't working taz-sprint-2 labels Dec 8, 2023
@HammadB HammadB added the p0 label Dec 8, 2023
tazarov added a commit to amikos-tech/chroma-core that referenced this issue Dec 12, 2023
tazarov added a commit to amikos-tech/chroma-core that referenced this issue Dec 15, 2023
- Fixed an issue with generic JSON errors - they need to match Chroma format with Error type and error message
- Cleaned up collection add unnecessary try/except
- Fixed unit mock tests to pass

Refs: chroma-core#1480
tazarov added a commit to amikos-tech/chroma-core that referenced this issue Dec 15, 2023
tazarov added a commit to amikos-tech/chroma-core that referenced this issue Dec 18, 2023
TODO - regen of the API is needed for error message.

Refs: chroma-core#1480
tazarov added a commit to amikos-tech/chroma-core that referenced this issue Dec 18, 2023
tazarov added a commit to amikos-tech/chroma-core that referenced this issue Dec 18, 2023
tazarov added a commit to amikos-tech/chroma-core that referenced this issue Dec 18, 2023
tazarov added a commit to amikos-tech/chroma-core that referenced this issue Dec 19, 2023
- Exported Settings in __all__

Refs: chroma-core#1480
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working p0 taz-sprint-4
Projects
None yet
Development

No branches or pull requests

2 participants