Skip to content
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

Conversation

bprashanth
Copy link
Contributor

Cherry pick of #28604 #29062 on release-1.3.

@k8s-github-robot k8s-github-robot added do-not-merge DEPRECATED. Indicates that a PR should not merge. Label can only be manually applied/removed. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. release-note-label-needed labels Jul 16, 2016
@bprashanth bprashanth assigned gmarek and unassigned mikedanese Jul 16, 2016
@bprashanth bprashanth force-pushed the automated-cherry-pick-of-#28604-#29062-upstream-release-1.3 branch from 57d2f21 to 4dd5345 Compare July 16, 2016 22:55
@bgrant0607
Copy link
Member

cc @gmarek @mqliang @cjcullen

@k8s-bot
Copy link

k8s-bot commented Jul 16, 2016

GCE e2e build/test passed for commit 4dd5345.

@davidopp
Copy link
Member

any reason not to LGTM this now?

@bgrant0607 bgrant0607 added release-note Denotes a PR that will be considered when it comes time to generate release notes. cherry-pick-approved Indicates a cherry-pick PR into a release branch has been approved by the release branch manager. and removed release-note-label-needed labels Jul 16, 2016
@davidopp
Copy link
Member

@bgrant0607 are you going to do the cherrypick? I labeled the two PRs as cherrypick-candidate and 1.3 milestone.

@bgrant0607
Copy link
Member

@davidopp I was looking over the changes in this PR first. If we LGTM it, I believe it should be merged automatically.

@bgrant0607
Copy link
Member

@bprashanth Were any conflicts resolved manually, or was this a straightforward patch?

@bgrant0607 bgrant0607 removed the do-not-merge DEPRECATED. Indicates that a PR should not merge. Label can only be manually applied/removed. label Jul 16, 2016
@bgrant0607
Copy link
Member

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

@bgrant0607 bgrant0607 added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Jul 16, 2016
@k8s-github-robot
Copy link

@k8s-bot test this [submit-queue is verifying that this PR is safe to merge]

@bprashanth
Copy link
Contributor Author

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.

@k8s-bot
Copy link

k8s-bot commented Jul 17, 2016

GCE e2e build/test passed for commit 4dd5345.

@k8s-github-robot
Copy link

Automatic merge from submit-queue

@k8s-github-robot k8s-github-robot merged commit 2ba1a1d into kubernetes:release-1.3 Jul 17, 2016
shyamjvs pushed a commit to shyamjvs/kubernetes that referenced this pull request Dec 1, 2016
…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.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cherry-pick-approved Indicates a cherry-pick PR into a release branch has been approved by the release branch manager. lgtm "Looks good to me", indicates that a PR is ready to be merged. release-note Denotes a PR that will be considered when it comes time to generate release notes. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants