Closed tomasaschan closed 6 months ago
[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: tomasaschan Once this PR has been reviewed and has the lgtm label, please assign justinsb for approval. For more information see the Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found here.
Hi @tomasaschan. Thanks for your PR.
I'm waiting for a kubernetes-sigs member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test
on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test
label.
I understand the commands that are listed here.
/ok-to-test
@justinsb This is currently blocking us from bumping k8s.io/kubectl
to 0.29.1 and get this fix. Any chance you can review this soon-ish? :)
/assign @justinsb Could you review this?
@justinsb Gentle ping, cloud you review this?
PR needs rebase.
@tomasaschan @justinsb Can we close this PR w/o merging, because #380 is merged? That PR had the update bumping kubectl to v0.29.1.
Main motivation is to include https://github.com/kubernetes/kubernetes/pull/122346
I don't know if there's a way to do this that touches fewer dependencies; I tried a bunch of ways with
go get [-u] k8s.io/kubectl[@0.29.1]
but all of them resulted in this full set of changes.