4 Commits (50c22ed7f953fadb32755e5881ba95a92da852b2)

Author	SHA1	Message	Date
Dominic Evans	7c74f1dd02	fix: improve logging & safety of statefulSetReady ... Confirm that the current and updated revision numbers also match as part of the readiness check. Add coverage for readiness scenarios where StatefulSet status does not reflect the most recent generation of the StatefulSet yet. Also add additional logging around the sts transitions from non-ready to ready. Fixes: #10163 Signed-off-by: Dominic Evans <dominic.evans@uk.ibm.com>	2 years ago
Bhavin Gandhi	c3310bb724	fix(pkg/kube): statefulSetReady: handle partition cases correctly ... The partition value can be greater than number of replicas, in that case no pods are rolled out. The expectedReplicas becomes a negative number. https://kubernetes.io/docs/concepts/workloads/controllers/statefulset/#partitions In the cases where the update does not change anything in the pod template, the updatedReplicas value from StatefulSet status remains unchanged. Such updates can still set some partition value, and UpdatedReplicas is always greater than expectedReplicas. Basically, the StatefulSet is ready / rolled-out. In both the above scenarios, providing `--wait` flag causes it to timeout waiting indefinitely. Because updatedReplicas can never be negative, or be equal to the expectedReplicas for the second case. This commit handles both the scenarios by checking if UpdatedReplicas is smaller than expectedReplicas. If it is, then the StatefulSet is not ready yet. Based on the code from kubectl rollout: a450ebd59c/pkg/polymorphichelpers/rollout_status.go (L138-L141) Closes #8674 Signed-off-by: Bhavin Gandhi <bhavin7392@gmail.com>	3 years ago
lsowen	249d1b5fb9	Fix SIGSEGV when job.Spec.Completions is nil ... Signed-off-by: lsowen <logan@s1network.com>	3 years ago
Joe Lanford	98d98369ed	Add ReadyChecker to decouple ready check logic from --wait ... Signed-off-by: Joe Lanford <joe.lanford@gmail.com>	4 years ago