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

e2e: disable running cli commands in container test because of flakes #15363

Merged
merged 1 commit into from
Jul 20, 2017

Conversation

mfojtik
Copy link
Contributor

@mfojtik mfojtik commented Jul 20, 2017

Temporarely disable this to unblock merge/test queue. This is likely journald rate limiting hitting us again...

@liggitt @smarterclayton @soltysh we need to investigate this as P0 for 3.7 :-(

Fixes #14897

@mfojtik
Copy link
Contributor Author

mfojtik commented Jul 20, 2017

[test]

@0xmichalis
Copy link
Contributor

0xmichalis commented Jul 20, 2017

Isn't this the second time we disable this test? :)

@mfojtik
Copy link
Contributor Author

mfojtik commented Jul 20, 2017

@Kargakis it might be... i'm still not sure what caused this to flake again, best guess will be something in the recent kube rebase around --attach behavior that cripples the container logs.

@0xmichalis
Copy link
Contributor

/joke

@openshift-ci-robot
Copy link

@Kargakis: A cannibal is someone who is fed up with people..

In response to this:

/joke

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@mfojtik
Copy link
Contributor Author

mfojtik commented Jul 20, 2017

/joke about this PR

@0xmichalis
Copy link
Contributor

/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Jul 20, 2017
@0xmichalis
Copy link
Contributor

/lgtm cancel
/unassign

@openshift-ci-robot openshift-ci-robot removed the lgtm Indicates that a PR is ready to be merged. label Jul 20, 2017
@mfojtik
Copy link
Contributor Author

mfojtik commented Jul 20, 2017

/assign @Kargakis

@mfojtik
Copy link
Contributor Author

mfojtik commented Jul 20, 2017

/approve

@0xmichalis
Copy link
Contributor

No approvals for Origin... At least for now

@mfojtik
Copy link
Contributor Author

mfojtik commented Jul 20, 2017

:(

@0xmichalis
Copy link
Contributor

/unassign

@mfojtik
Copy link
Contributor Author

mfojtik commented Jul 20, 2017

flake: #13977

[merge] (based on @bparees approval in the issue ;-)

@openshift-bot
Copy link
Contributor

openshift-bot commented Jul 20, 2017

continuous-integration/openshift-jenkins/merge Waiting: You are in the build queue at position: 13

@openshift-bot
Copy link
Contributor

Evaluated for origin merge up to fc434b6

@mfojtik
Copy link
Contributor Author

mfojtik commented Jul 20, 2017

test flake: #13977

@0xmichalis
Copy link
Contributor

[test]

@openshift-bot
Copy link
Contributor

Evaluated for origin test up to fc434b6

@openshift openshift deleted a comment from 0xmichalis Jul 20, 2017
@mfojtik mfojtik merged commit ded9a03 into openshift:master Jul 20, 2017
@mfojtik
Copy link
Contributor Author

mfojtik commented Jul 20, 2017

force merging this as every second PR failing to merge/test without this.

@liggitt
Copy link
Contributor

liggitt commented Jul 20, 2017

make sure there's an open blocking issue to determine the root cause

@mfojtik
Copy link
Contributor Author

mfojtik commented Jul 20, 2017

@liggitt will keep #14897 open

@openshift-bot
Copy link
Contributor

continuous-integration/openshift-jenkins/test SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pull_request_origin/3336/) (Base Commit: bfb45e9) (PR Branch Commit: fc434b6)

@mfojtik mfojtik deleted the disable-e2e-test-for-flake branch September 5, 2018 21:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants