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 #37293 #37379 upstream release 1.4 #37938

Conversation

wojtek-t
Copy link
Member

@wojtek-t wojtek-t commented Dec 2, 2016

Fix bug in scheduler that could cause that scheduler could incorrectly assign pods to nodes where they do not fit ignoring nodes when they fit.

Automated cherry pick of #37293 #37379 upstream release 1.4

@wojtek-t wojtek-t added cherrypick-candidate release-note-none Denotes a PR that doesn't merit a release note. labels Dec 2, 2016
@wojtek-t wojtek-t added this to the v1.4 milestone Dec 2, 2016
@k8s-ci-robot k8s-ci-robot added cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. labels Dec 2, 2016
@k8s-ci-robot
Copy link
Contributor

Jenkins GCE Node e2e failed for commit 21ccfb0033c1aa2d17c90c33a9d96169a3f96346. Full PR test history.

The magic incantation to run this job again is @k8s-bot node e2e test this. Please help us cut down flakes by linking to an open flake issue when you hit one in your PR.

@davidopp davidopp added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Dec 2, 2016
@davidopp
Copy link
Member

davidopp commented Dec 2, 2016

LGTM

@k8s-ci-robot
Copy link
Contributor

Jenkins GCE etcd3 e2e failed for commit 21ccfb0033c1aa2d17c90c33a9d96169a3f96346. Full PR test history.

The magic incantation to run this job again is @k8s-bot gce etcd3 e2e test this. Please help us cut down flakes by linking to an open flake issue when you hit one in your PR.

@k8s-ci-robot
Copy link
Contributor

Jenkins unit/integration failed for commit 21ccfb0033c1aa2d17c90c33a9d96169a3f96346. Full PR test history.

The magic incantation to run this job again is @k8s-bot unit test this. Please help us cut down flakes by linking to an open flake issue when you hit one in your PR.

@k8s-ci-robot
Copy link
Contributor

Jenkins GKE smoke e2e failed for commit 21ccfb0033c1aa2d17c90c33a9d96169a3f96346. Full PR test history.

The magic incantation to run this job again is @k8s-bot cvm gke e2e test this. Please help us cut down flakes by linking to an open flake issue when you hit one in your PR.

@k8s-ci-robot
Copy link
Contributor

Jenkins GCE e2e failed for commit 21ccfb0033c1aa2d17c90c33a9d96169a3f96346. Full PR test history.

The magic incantation to run this job again is @k8s-bot cvm gce e2e test this. Please help us cut down flakes by linking to an open flake issue when you hit one in your PR.

@k8s-ci-robot
Copy link
Contributor

Jenkins GCI GCE e2e failed for commit 21ccfb0033c1aa2d17c90c33a9d96169a3f96346. Full PR test history.

The magic incantation to run this job again is @k8s-bot gci gce e2e test this. Please help us cut down flakes by linking to an open flake issue when you hit one in your PR.

@k8s-ci-robot
Copy link
Contributor

Jenkins GCI GKE smoke e2e failed for commit 21ccfb0033c1aa2d17c90c33a9d96169a3f96346. Full PR test history.

The magic incantation to run this job again is @k8s-bot gci gke e2e test this. Please help us cut down flakes by linking to an open flake issue when you hit one in your PR.

@k8s-ci-robot
Copy link
Contributor

Jenkins Kubemark GCE e2e failed for commit 21ccfb0033c1aa2d17c90c33a9d96169a3f96346. Full PR test history.

The magic incantation to run this job again is @k8s-bot kubemark e2e test this. Please help us cut down flakes by linking to an open flake issue when you hit one in your PR.

@wojtek-t wojtek-t force-pushed the automated-cherry-pick-of-#37293-#37379-upstream-release-1.4 branch from 21ccfb0 to 9937f9a Compare December 2, 2016 09:35
@k8s-github-robot
Copy link

/lgtm cancel //PR changed after LGTM, removing LGTM. @davidopp @jessfraz @wojtek-t

@k8s-github-robot
Copy link

This PR is not for the master branch but does not have the cherrypick-approved label. Adding the do-not-merge label.

@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/L Denotes a PR that changes 100-499 lines, ignoring generated files. and removed lgtm "Looks good to me", indicates that a PR is ready to be merged. labels Dec 2, 2016
@wojtek-t
Copy link
Member Author

wojtek-t commented Dec 2, 2016

I just fixed compile errors from manuall cherrypick.

@k8s-ci-robot
Copy link
Contributor

Jenkins verification failed for commit 21ccfb0033c1aa2d17c90c33a9d96169a3f96346. Full PR test history.

The magic incantation to run this job again is @k8s-bot verify test this. Please help us cut down flakes by linking to an open flake issue when you hit one in your PR.

@jessfraz jessfraz added 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. and removed do-not-merge DEPRECATED. Indicates that a PR should not merge. Label can only be manually applied/removed. labels Dec 2, 2016
@jessfraz
Copy link
Contributor

jessfraz commented Dec 2, 2016

all tests are green so merging manually

@jessfraz jessfraz merged commit 770bc36 into kubernetes:release-1.4 Dec 2, 2016
@k8s-cherrypick-bot
Copy link

Commit found in the "release-1.4" branch appears to be this PR. Removing the "cherrypick-candidate" label. If this is an error find help to get your PR picked.

@wojtek-t wojtek-t deleted the automated-cherry-pick-of-#37293-#37379-upstream-release-1.4 branch January 2, 2017 12:25
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. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. lgtm "Looks good to me", indicates that a PR is ready to be merged. release-note-none Denotes a PR that doesn't merit a release note. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants