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
Elasticsearch: Implement CheckHealth method in the backend #81671
Conversation
Hey @mikelv92 ! Thanks for your contribution. We'll reviewing as soon as we have some bandwidth. Stay tunned! |
Thank you in advance for the review. I have already signed the CLA but I think the bot isn't updating the workflow... |
Thanks @mikelv92. Just to be sure, did you add the email used for the commit to your Github account? That's needed for the CLA to be marked as signed. |
Unfortunately, I used the wrong email in the first commit... It's fixed now :) |
2ef899e
to
00521cc
Compare
This reverts commit 676265f.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice, LGTM! Thank you for your contribution, @mikelv92!
* Elasticsearch: Implement CheckHealth method * improve logger output * remove frontend healthcheck * Revert "remove frontend healthcheck" This reverts commit 676265f. * adapt test --------- Co-authored-by: Sven Grossmann <svennergr@gmail.com>
What is this feature?
ElasticSearch does not have a CheckHealth method in the backend code implemented. This PR adds a CheckHealth so that the
/api/datasources/uid/:uid/health
endpoint will return the health information.Why do we need this feature?
Currently the /health endpoint is not working.
Who is this feature for?
Users that want to check the health of the datasource when using Elasticsearch
Which issue(s) does this PR fix?:
Fixes #63727
Special notes for your reviewer:
make devenv sources=elastic
gdev-elasticsearch
datasource, openhttp://localhost:3000/api/datasources/uid/gdev-elasticsearch/health
Please check that: