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

Trusty: fix breakage by #26413 and #26109 #26599

Merged
merged 1 commit into from
Jun 1, 2016
Merged

Trusty: fix breakage by #26413 and #26109 #26599

merged 1 commit into from
Jun 1, 2016

Conversation

andyzheng0831
Copy link

The code https://github.com/kubernetes/kubernetes/tree/master/cluster/gce/trusty in both master and release-1.2 branches is broken. Although we already switched to using cluster/gce/gci for 1.3 branch, we will still need to maintain the cluster/gce/trusty code as long as release-1.2 branch is not deprecated. I will make a couple of PR and cherry-picks to make cluster/gce/trusty back to normal. This PR is one of them, which will not be cherry picked to release-1.2.

@roberthbailey @dchen1107 @fabioy @zmerlynn

cc/ @kubernetes/goog-image

@andyzheng0831 andyzheng0831 added the release-note-none Denotes a PR that doesn't merit a release note. label May 31, 2016
@andyzheng0831 andyzheng0831 added this to the v1.3 milestone May 31, 2016
@k8s-github-robot k8s-github-robot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label May 31, 2016
@zmerlynn zmerlynn added lgtm "Looks good to me", indicates that a PR is ready to be merged. priority/critical-urgent Highest priority. Must be actively worked on as someone's top priority right now. labels Jun 1, 2016
@k8s-github-robot k8s-github-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Jun 1, 2016
@k8s-github-robot k8s-github-robot removed lgtm "Looks good to me", indicates that a PR is ready to be merged. needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. labels Jun 1, 2016
@zmerlynn zmerlynn added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Jun 1, 2016
@andyzheng0831
Copy link
Author

I add "e2e-not-required" as the change in this PR is not covered by default e2e tests.

@k8s-bot
Copy link

k8s-bot commented Jun 1, 2016

GCE e2e build/test failed for commit 9a413b3.

Please reference the list of currently known flakes when examining this failure. If you request a re-test, you must reference the issue describing the flake.

@andyzheng0831
Copy link
Author

@k8s-bot unit test this issue #26256

@k8s-github-robot
Copy link

Automatic merge from submit-queue

@k8s-github-robot k8s-github-robot merged commit 06f742a into kubernetes:master Jun 1, 2016
@andyzheng0831 andyzheng0831 deleted the fix-trusty branch June 1, 2016 21:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm "Looks good to me", indicates that a PR is ready to be merged. priority/critical-urgent Highest priority. Must be actively worked on as someone's top priority right now. release-note-none Denotes a PR that doesn't merit a release note. size/S Denotes a PR that changes 10-29 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants