-
Notifications
You must be signed in to change notification settings - Fork 87
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
Change the default livenessProbe endpoint to /health
#83
Comments
Are you going to parse the response and decide if it is healthy?
Apart from |
I think that we don't need to parse it, HTTP 200 from Currently I think it's a little wrong, because we use |
any guidance on how to validate these two things? rolling upgrades and scaling down? |
well just came across these: but yea - any other advice would be welcome. thanks |
Hi @adnxn , glad to see you here :) I'm assigning you the issue and I'll make sure to get you some guidance from our experts. Looking forward to your contribution! |
@adnxn, thanks for taking this issue! So, I recommend doing the following:
Also, I'd not change |
And about the technical details how to scale up/down and how to perform the rolling updates, the blog posts you mentioned are good guidelines. I recommend using Helm Chart. Then for the scaling, all you need to do is to execute:
And for the rolling update:
Write here if you encounter any issues. I'll try to help. |
@leszko: thanks for the info
what would be the best way to do this? also - seems like changing the endpoint breaks the management console for v3.12.* hrm |
|
Hi @adnxn , |
@adnxn are you still working on this issue? |
hey i havent had time to follow up on this. if someone else wants to take it over, feel free. |
any news on this ? |
Currently the default
livenessProbe
endpoint is/health/node-state
. It should actually be/health
.The change is trivial, but before applying it we need to double check that it does not break rolling upgrade and scaling down.
The text was updated successfully, but these errors were encountered: