-
Notifications
You must be signed in to change notification settings - Fork 38.8k
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
Add resource printer and describer for PodDisruptionBudget #35287
Conversation
Jenkins GKE smoke e2e failed for commit c283531. Full PR test history. The magic incantation to run this job again is |
clientset.Interface | ||
} | ||
|
||
func (p *PodDisruptionBudgetDescriber) Describe(namespace, name string, describerSettings DescriberSettings) (string, error) { |
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 guess we should print some status information as well. However I have a pr in flight that changes the status struct a bit #34546. The PR should be merged on Monday.
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 am assigned as the reviewer for that PR and didn't know it was ready to review. Do yuo want me to review it? (How was it going to be merged on Monday without a review?)
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'll wait for the other PR to land and will update this accordingly, in that case.
Jenkins verification failed for commit a1b7ccb. Full PR test history. The magic incantation to run this job again is |
Jenkins GCI GKE smoke e2e failed for commit a1b7ccb. Full PR test history. The magic incantation to run this job again is |
We need this PR back in shape ASAP. I completely forgot about it. If you don't have time to work on this would it be ok for me to cherry-pick your commit and add some fixes on top of it? |
Superseded by #36495. |
@davidopp || @mml ptal
This change is![Reviewable](https://camo.githubusercontent.com/2d899f4291d07d3cd2fa4aaae1e3b243f164c23fce87d30a589ace0d496a444c/68747470733a2f2f72657669657761626c652e6b756265726e657465732e696f2f7265766965775f627574746f6e2e737667)