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.

Sign up for GitHub

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

Jump to bottom

Use /readyz and /livez endpoints #71

Closed
timvisee opened this issue Sep 4, 2023 · 3 comments · Fixed by #73
Closed

Use /readyz and /livez endpoints #71

timvisee opened this issue Sep 4, 2023 · 3 comments · Fixed by #73
Labels
enhancement New feature or request

Comments

@timvisee
Copy link
Member

timvisee commented Sep 4, 2023 •
edited
Loading

qdrant/qdrant#2409 has implemented /livez and /readyz endpoints for health checking. This will be part of the next Qdrant release (1.5.0).

I think we should incorporate those in our Helm Chart rather than relying /. /livez and /readyz do also properly work if the user has an API key configured.
@timvisee timvisee added the enhancement New feature or request label Sep 4, 2023
@timvisee timvisee mentioned this issue Sep 4, 2023
Merged
14 tasks
@timvisee
Copy link
Member Author

timvisee commented Sep 7, 2023

These endpoints are now released as part of v1.5.0.

The documentation PR on it can be found here: qdrant/landing_page#280

@bashofmann
Copy link
Collaborator

Thanks, I'll work on this this afternoon.

bashofmann added a commit that referenced this issue Sep 7, 2023
@bashofmann
Update to Qdrant 1.5.0
815224d 
This also changes the probes to use the new healthz readyz and livez endpoints

Resolves #72
Resolves #71
@bashofmann bashofmann mentioned this issue Sep 7, 2023
Merged
@timvisee
Copy link
Member Author

timvisee commented Sep 7, 2023

Thank you for picking this up so quickly!

bashofmann added a commit that referenced this issue Sep 7, 2023
@bashofmann @timvisee
Update to Qdrant 1.5.0 (#73)
ae5fb05 
* Update to Qdrant 1.5.0

This also changes the probes to use the new healthz readyz and livez endpoints

Resolves #72
Resolves #71

* Update charts/qdrant/templates/statefulset.yaml

Co-authored-by: Tim Visée <tim+github@visee.me>

* Update qdrant_probes_test.go

---------

Co-authored-by: Tim Visée <tim+github@visee.me>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Update to Qdrant 1.5.0 qdrant/qdrant-helm
2 participants
@bashofmann @timvisee