-
Notifications
You must be signed in to change notification settings - Fork 104
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
KUDO init --wait checks statefulset instead of pod #1637
Conversation
Signed-off-by: Alena Varkockova <varkockova.a@gmail.com>
I'll investigate the failures tomorrow |
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.
Checking the pod readiness is certainly an improvement. But I don't quite understand why we don't check KUDO manager StatefulSet status (and check individual Pod
s)
pkg/kudoctl/kudoinit/setup/wait.go
Outdated
|
||
s, err := managerContainerStatus(pod) | ||
if err != nil { | ||
pod, err := getFirstReadyPod(client, namespace, selector) |
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.
Now that I'm looking at it again: why do we check the individual manager pods? It is a StatefulSet
so why not simply check that with the healthutil?
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.
I thought statefulset does not have enough information about readiness but it does so ... thank you :) you're right
Signed-off-by: Alena Varkockova <varkockova.a@gmail.com>
Signed-off-by: Alena Varkockova <varkockova.a@gmail.com>
Signed-off-by: Alena Varkockova <varkockova.a@gmail.com>
19644e6
to
fdbb20a
Compare
Signed-off-by: Alena Varkockova <varkockova.a@gmail.com>
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, this looks much easier
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.
BOOM! 🚢
Signed-off-by: Alena Varkockova varkockova.a@gmail.com
What this PR does / why we need it:
Not only
--wait
, also the health module should read theready
property, notphase == running
.See https://kubernetes.io/docs/concepts/workloads/pods/pod-lifecycle/ for more context.