-
Notifications
You must be signed in to change notification settings - Fork 40k
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
Automated cherry pick of #28604 #29062 #29063
Automated cherry pick of #28604 #29062 #29063
Conversation
57d2f21
to
4dd5345
Compare
GCE e2e build/test passed for commit 4dd5345. |
any reason not to LGTM this now? |
@bgrant0607 are you going to do the cherrypick? I labeled the two PRs as cherrypick-candidate and 1.3 milestone. |
@davidopp I was looking over the changes in this PR first. If we LGTM it, I believe it should be merged automatically. |
@bprashanth Were any conflicts resolved manually, or was this a straightforward patch? |
This is a bigger change than we'd like, but I think we should go ahead with cherrypicking #28604, because I believe there will additional bugs we want to fix. LGTM |
@k8s-bot test this [submit-queue is verifying that this PR is safe to merge] |
There were some conflicts that I resolved manually (function args changed from 1.3 to HEAD, I kept the 1.3 version). I only cherrypicked the one pr I needed from this stack (#28832). I suppose things should fail if i picked wrong. |
GCE e2e build/test passed for commit 4dd5345. |
Automatic merge from submit-queue |
…pick-of-#28604-kubernetes#29062-upstream-release-1.3 Automatic merge from submit-queue Automated cherry pick of kubernetes#28604 kubernetes#29062 Cherry pick of kubernetes#28604 kubernetes#29062 on release-1.3.
Cherry pick of #28604 #29062 on release-1.3.