-
Notifications
You must be signed in to change notification settings - Fork 0
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
[Merged by Bors] - Add proper startup, liveness and readiness probes #273
Conversation
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.
LGTM. I checked the docs and think you got it right :)
bors r+ |
# Description For #142 It would be very good if you could check the druid docs to make sure if not miss-understood the checks or if you have a better idea
Pull request successfully merged into main. Build succeeded: |
Sorry, was a bit late to this. Just to add that an extra motivation behind this, IIRC, was to avoid a lot of the error logging in the tests. Am running the tests locally now and the output is a lot cleaner. Nice job! |
Awesome, thx! Sorry for the merging so fast ^^ |
Description
For #142
It would be very good if you could check the druid docs to make sure if not miss-understood the checks or if you have a better idea
Review Checklist
Once the review is done, comment
bors r+
(orbors merge
) to merge. Further information