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
kubectl: enhancements for deployment progress deadline #36171
kubectl: enhancements for deployment progress deadline #36171
Conversation
@@ -2161,6 +2161,13 @@ func (dd *DeploymentDescriber) Describe(namespace, name string, describerSetting | |||
ru := d.Spec.Strategy.RollingUpdate | |||
fmt.Fprintf(out, "RollingUpdateStrategy:\t%s max unavailable, %s max surge\n", ru.MaxUnavailable.String(), ru.MaxSurge.String()) | |||
} | |||
if len(d.Status.Conditions) > 0 { | |||
fmt.Fprint(out, "Conditions:\n Type\tStatus\tReason\n") | |||
fmt.Fprint(out, " ----\t------\t------\n") |
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 think we can use tabwriter here.
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.
All the rest of the describers are not using it atm and I don't want to introduce a bigger change in this PR just before the code freeze.
Changes: * add deployment conditions in the describer * abort 'rollout status' for deployments that have exceeded their progress deadline
LGTM |
Jenkins unit/integration failed for commit f97666e. Full PR test history. The magic incantation to run this job again is |
@k8s-bot test this [submit-queue is verifying that this PR is safe to merge] |
Automatic merge from submit-queue |
Changes:
progress deadline
Depends on #35691.
@kubernetes/kubectl @kubernetes/deployment
Fixes #31319
This change is