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

Comment out racey part of the multi-scheduler test. #23717

Merged
merged 1 commit into from
Apr 5, 2016

Conversation

mml
Copy link
Contributor

@mml mml commented Mar 31, 2016

Should fix #22848.

@k8s-github-robot
Copy link

Labelling this PR as size/M

@k8s-github-robot k8s-github-robot added size/M Denotes a PR that changes 30-99 lines, ignoring generated files. release-note-label-needed labels Mar 31, 2016
@k8s-bot
Copy link

k8s-bot commented Apr 1, 2016

GCE e2e build/test passed for commit 6ace15e.

@spxtr spxtr assigned davidopp and unassigned spxtr Apr 1, 2016
@davidopp
Copy link
Member

davidopp commented Apr 3, 2016

LGTM

@davidopp davidopp added 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. and removed release-note-label-needed labels Apr 3, 2016
@k8s-github-robot
Copy link

@k8s-bot test this

Tests are more than 48 hours old. Re-running tests.

@k8s-bot
Copy link

k8s-bot commented Apr 3, 2016

GCE e2e build/test passed for commit 6ace15e.

@mml
Copy link
Contributor Author

mml commented Apr 4, 2016

@k8s-oncall you might want to manually merge this to help with this flake.

@k8s-github-robot
Copy link

@k8s-bot test this

Tests are more than 48 hours old. Re-running tests.

@k8s-bot
Copy link

k8s-bot commented Apr 5, 2016

GCE e2e build/test passed for commit 6ace15e.

@mml
Copy link
Contributor Author

mml commented Apr 5, 2016

Good grief!

[node-pull-build-e2e-test] $ /bin/bash /tmp/hudson813183256621992275.sh
+ cd go/src/k8s.io/kubernetes
/tmp/hudson813183256621992275.sh: line 4: cd: go/src/k8s.io/kubernetes: No such file or directory
Build step 'Execute shell' marked build as failure

@mml mml closed this Apr 5, 2016
@mml mml reopened this Apr 5, 2016
@k8s-github-robot
Copy link

@mml
You must link to the test flake issue which caused you to request this manual re-test.
Re-test requests should be in the form of: k8s-bot test this issue: #<number>
Here is the list of open test flakes.

@k8s-github-robot k8s-github-robot removed the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Apr 5, 2016
@saad-ali
Copy link
Member

saad-ali commented Apr 5, 2016

@k8s-oncall you might want to manually merge this to help with this flake.

Will manually merge on green.

@k8s-bot
Copy link

k8s-bot commented Apr 5, 2016

GCE e2e build/test passed for commit 6ace15e.

@k8s-bot
Copy link

k8s-bot commented Apr 5, 2016

GCE e2e build/test passed for commit 6ace15e.

@saad-ali
Copy link
Member

saad-ali commented Apr 5, 2016

Manually merging. Jenkins GCE e2e, Jenkins unit/integration, and travis-ci are all green. Jenkins GCE Node e2e is red, but that is an environment issue blocking the merge queue not unique to this PR.

@saad-ali saad-ali merged commit 48fa998 into kubernetes:master Apr 5, 2016
@mml mml deleted the until branch April 5, 2016 23:15
openshift-publish-robot pushed a commit to openshift/kubernetes that referenced this pull request Sep 3, 2019
UPSTREAM: <carry>: kill kube-controller-manager on CSR file content change

Origin-commit: e795244084b7d236a8f2641c553592240da79148
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release-note-none Denotes a PR that doesn't merit a release note. size/M Denotes a PR that changes 30-99 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

integration flake: TestMultiScheduler is flaky
8 participants