Skip to content
This repository was archived by the owner on Nov 28, 2022. It is now read-only.

Port 4093 to our 0.17#846

Merged
openshift-merge-robot merged 1 commit into
openshift:release-v0.17.2from
matzew:port_4093_to_017
Sep 28, 2020
Merged

Port 4093 to our 0.17#846
openshift-merge-robot merged 1 commit into
openshift:release-v0.17.2from
matzew:port_4093_to_017

Conversation

@matzew
Copy link
Copy Markdown
Member

@matzew matzew commented Sep 24, 2020

as per title

…e#4093)

* Use WaitForPodRunning instead of WaitForResourceReady on pods

Signed-off-by: Pierangelo Di Pilato <pierangelodipilato@gmail.com>

* Check if pod is running in isResourceReady

Signed-off-by: Pierangelo Di Pilato <pierangelodipilato@gmail.com>

* Revert "Use WaitForPodRunning instead of WaitForResourceReady on pods"

This reverts commit 6d59fd2.
@openshift-ci-robot openshift-ci-robot added the area/test-and-release Issues or PRs related to test and release label Sep 24, 2020
@openshift-ci-robot openshift-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Sep 24, 2020
@matzew
Copy link
Copy Markdown
Member Author

matzew commented Sep 24, 2020 via email

@matzew
Copy link
Copy Markdown
Member Author

matzew commented Sep 24, 2020 via email

@matzew
Copy link
Copy Markdown
Member Author

matzew commented Sep 24, 2020 via email

@matzew
Copy link
Copy Markdown
Member Author

matzew commented Sep 24, 2020

/retest

@matzew
Copy link
Copy Markdown
Member Author

matzew commented Sep 25, 2020

/refresh
/test all

@matzew matzew closed this Sep 25, 2020
@matzew matzew reopened this Sep 25, 2020
@matzew
Copy link
Copy Markdown
Member Author

matzew commented Sep 25, 2020

/retest

@lberk
Copy link
Copy Markdown
Member

lberk commented Sep 25, 2020

/test 46-e2e-aws-ocp-46

@matzew
Copy link
Copy Markdown
Member Author

matzew commented Sep 26, 2020 via email

@devguyio
Copy link
Copy Markdown

/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Sep 28, 2020
@openshift-ci-robot
Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: devguyio, matzew

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-bot
Copy link
Copy Markdown

/retest

Please review the full test history for this PR and help us cut down flakes.

1 similar comment
@openshift-bot
Copy link
Copy Markdown

/retest

Please review the full test history for this PR and help us cut down flakes.

@matzew
Copy link
Copy Markdown
Member Author

matzew commented Sep 28, 2020 via email

@openshift-bot
Copy link
Copy Markdown

/retest

Please review the full test history for this PR and help us cut down flakes.

4 similar comments
@openshift-bot
Copy link
Copy Markdown

/retest

Please review the full test history for this PR and help us cut down flakes.

@openshift-bot
Copy link
Copy Markdown

/retest

Please review the full test history for this PR and help us cut down flakes.

@openshift-bot
Copy link
Copy Markdown

/retest

Please review the full test history for this PR and help us cut down flakes.

@openshift-bot
Copy link
Copy Markdown

/retest

Please review the full test history for this PR and help us cut down flakes.

@openshift-merge-robot openshift-merge-robot merged commit 5f1756c into openshift:release-v0.17.2 Sep 28, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. area/test-and-release Issues or PRs related to test and release lgtm Indicates that a PR is ready to be merged.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants