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

UPSTREAM: 35082: Wait for all pods to be running before checking PDB status #11714

Merged
merged 1 commit into from
Nov 2, 2016

Conversation

soltysh
Copy link
Contributor

@soltysh soltysh commented Nov 1, 2016

Fixes #11661.

I've modified the upstream PR so that if fits our current implementation, but the main idea is the same.

@smarterclayton ptal, that should limit the no of flakes in our queue

@smarterclayton
Copy link
Contributor

LGTM

@smarterclayton
Copy link
Contributor

[merge]

@openshift-bot
Copy link
Contributor

openshift-bot commented Nov 1, 2016

continuous-integration/openshift-jenkins/merge SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pr_origin/10972/) (Image: devenv-rhel7_5300)

@openshift-bot
Copy link
Contributor

Evaluated for origin merge up to 0d057f1

@openshift-bot
Copy link
Contributor

[Test]ing while waiting on the merge queue

@smarterclayton
Copy link
Contributor

[test] conformance hung forever and check flaked on docker hub

@openshift-bot
Copy link
Contributor

Evaluated for origin test up to 0d057f1

@openshift-bot
Copy link
Contributor

continuous-integration/openshift-jenkins/test SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pr_origin/10972/) (Base Commit: 90d8c62)

@pweil- pweil- added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Nov 2, 2016
@openshift-bot openshift-bot merged commit 6932bcc into openshift:master Nov 2, 2016
@soltysh soltysh deleted the issue11661 branch November 2, 2016 14:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants