-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
rule: Add liveness and readiness probe #1538
Conversation
Signed-off-by: Kemal Akkoyun <kakkoyun@gmail.com>
Signed-off-by: Kemal Akkoyun <kakkoyun@gmail.com>
Signed-off-by: Kemal Akkoyun <kakkoyun@gmail.com>
cc @FUSAKLA |
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, minor nit only.
Co-Authored-By: Bartlomiej Plotka <bwplotka@gmail.com> Signed-off-by: Kemal Akkoyun <kakkoyun@gmail.com>
89b3692
to
a60659a
Compare
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
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 thanks! 👍
One tiny thing, please add those also to the https://github.com/thanos-io/thanos/blob/master/tutorials/kubernetes-demo/manifests/thanos-ruler.yaml example so it stays up to date |
Signed-off-by: Kemal Akkoyun <kakkoyun@gmail.com>
* Add prober to rule Signed-off-by: Kemal Akkoyun <kakkoyun@gmail.com> * Add changelog entry Signed-off-by: Kemal Akkoyun <kakkoyun@gmail.com> * Update README Signed-off-by: Kemal Akkoyun <kakkoyun@gmail.com> * Apply suggestions from code review Co-Authored-By: Bartlomiej Plotka <bwplotka@gmail.com> Signed-off-by: Kemal Akkoyun <kakkoyun@gmail.com> * Update rule tutorial Signed-off-by: Kemal Akkoyun <kakkoyun@gmail.com> Signed-off-by: Giedrius Statkevičius <giedriuswork@gmail.com>
Signed-off-by: Kemal Akkoyun kakkoyun@gmail.com
This PR,
/-/healthy
endpoint for liveness checks./-/ready
endpoint for readiness checks.Changes
/-/healthy
endpoint for liveness checks./-/ready
endpoint for readiness checks.prober.Prober
for readiness and liveness endpoints.Verification
make test
Started
thanos rule
and made a request to related endpoints.thanos rule
and visitedhttp://localhost:10902/rules
to check UI.