Skip to content

check the olm --version#24009

Merged
openshift-merge-robot merged 2 commits intoopenshift:masterfrom
bparees:olmtest
Jan 14, 2020
Merged

check the olm --version#24009
openshift-merge-robot merged 2 commits intoopenshift:masterfrom
bparees:olmtest

Conversation

@bparees
Copy link
Copy Markdown
Contributor

@bparees bparees commented Oct 23, 2019

No description provided.

@openshift-ci-robot openshift-ci-robot added size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. approved Indicates a PR has been approved by an approver from all required OWNERS files. labels Oct 23, 2019
@bparees bparees mentioned this pull request Oct 23, 2019
Closed
@bparees
Copy link
Copy Markdown
Contributor Author

bparees commented Oct 23, 2019

/retest

1 similar comment
@bparees
Copy link
Copy Markdown
Contributor Author

bparees commented Oct 24, 2019

/retest

@jianzhangbjz
Copy link
Copy Markdown
Contributor

jianzhangbjz commented Oct 29, 2019

@njhale @ecordell @tkashem Could you help take a look at this PR? Thanks!

@bparees bparees force-pushed the olmtest branch 3 times, most recently from 3bfc4ea to b0b1bc1 Compare October 29, 2019 17:41
@bparees
Copy link
Copy Markdown
Contributor Author

bparees commented Oct 29, 2019

/retest

@bparees
Copy link
Copy Markdown
Contributor Author

bparees commented Oct 30, 2019

/retest

4 similar comments
@bparees
Copy link
Copy Markdown
Contributor Author

bparees commented Oct 30, 2019

/retest

@bparees
Copy link
Copy Markdown
Contributor Author

bparees commented Oct 31, 2019

/retest

@bparees
Copy link
Copy Markdown
Contributor Author

bparees commented Oct 31, 2019

/retest

@bparees
Copy link
Copy Markdown
Contributor Author

bparees commented Nov 4, 2019

/retest

@bparees
Copy link
Copy Markdown
Contributor Author

bparees commented Nov 5, 2019

@jianzhangbjz your new test appears to be failing: https://prow.svc.ci.openshift.org/view/gcs/origin-ci-test/pr-logs/pull/24009/pull-ci-openshift-origin-master-e2e-gcp/3556

@jianzhangbjz
Copy link
Copy Markdown
Contributor

jianzhangbjz commented Nov 6, 2019

@bparees Thanks! I will debug it.

@jianzhangbjz
Copy link
Copy Markdown
Contributor

jianzhangbjz commented Nov 26, 2019

/retest

@jianzhangbjz
Copy link
Copy Markdown
Contributor

jianzhangbjz commented Dec 2, 2019

@bparees Could you help cherry-pick my latest commit: 88fd1e1 for this PR? Thanks very much!

@bparees
Copy link
Copy Markdown
Contributor Author

bparees commented Dec 2, 2019

@jianzhangbjz done, please check it

@jianzhangbjz
Copy link
Copy Markdown
Contributor

jianzhangbjz commented Dec 3, 2019

/retest

@jianzhangbjz
Copy link
Copy Markdown
Contributor

jianzhangbjz commented Dec 5, 2019

/test e2e-gcp
/test e2e-aws-fips

@jianzhangbjz
Copy link
Copy Markdown
Contributor

jianzhangbjz commented Dec 5, 2019

/retest

2 similar comments
@jianzhangbjz
Copy link
Copy Markdown
Contributor

jianzhangbjz commented Dec 6, 2019

/retest

@jianzhangbjz
Copy link
Copy Markdown
Contributor

jianzhangbjz commented Dec 12, 2019

/retest

@jianzhangbjz
Copy link
Copy Markdown
Contributor

jianzhangbjz commented Dec 12, 2019

/lgtm

@jianzhangbjz
Copy link
Copy Markdown
Contributor

jianzhangbjz commented Jan 10, 2020

@bparees Thanks! I have rebased it and change it to serial run. Could you help to cherry-pick it? Thanks! #23714

@bparees
Copy link
Copy Markdown
Contributor Author

bparees commented Jan 10, 2020

@bparees Thanks! I have rebased it and change it to serial run. Could you help to cherry-pick it? Thanks! #23714

done, but really at this point you should just go add a vendor bump to your PR and get it merged. It's worth knowing how to do it (and it's as simple as running hack/update-deps.sh in origin).

@bparees
Copy link
Copy Markdown
Contributor Author

bparees commented Jan 10, 2020

/retest

@bparees
Copy link
Copy Markdown
Contributor Author

bparees commented Jan 13, 2020

/test e2e-aws-fips

@bparees
Copy link
Copy Markdown
Contributor Author

bparees commented Jan 13, 2020

/test e2e-gcp

@bparees
Copy link
Copy Markdown
Contributor Author

bparees commented Jan 13, 2020

/test all

1 similar comment
@bparees
Copy link
Copy Markdown
Contributor Author

bparees commented Jan 13, 2020

/test all

@bparees
Copy link
Copy Markdown
Contributor Author

bparees commented Jan 13, 2020

/retest

@bparees
Copy link
Copy Markdown
Contributor Author

bparees commented Jan 13, 2020

/hold cancel

@openshift-ci-robot openshift-ci-robot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Jan 13, 2020
@bparees
Copy link
Copy Markdown
Contributor Author

bparees commented Jan 13, 2020

/retest

@bparees bparees added lgtm Indicates that a PR is ready to be merged. and removed lgtm Indicates that a PR is ready to be merged. labels Jan 14, 2020
@openshift-bot
Copy link
Copy Markdown
Contributor

openshift-bot commented Jan 14, 2020

/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
Contributor

openshift-bot commented Jan 14, 2020

/retest

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

@openshift-bot
Copy link
Copy Markdown
Contributor

openshift-bot commented Jan 14, 2020

/retest

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

@openshift-bot
Copy link
Copy Markdown
Contributor

openshift-bot commented Jan 14, 2020

/retest

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

@openshift-bot
Copy link
Copy Markdown
Contributor

openshift-bot commented Jan 14, 2020

/retest

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

@openshift-ci-robot
Copy link
Copy Markdown

openshift-ci-robot commented Jan 14, 2020 •
edited
Loading

@bparees: The following test failed, say /retest to rerun all failed tests:

Test name Commit Details Rerun command
ci/prow/e2e-aws f764bae link /test e2e-aws

Full PR test history. Your PR dashboard. Please help us cut down on flakes by linking to an open issue when you hit one in your PR.

Details

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. I understand the commands that are listed here.

@openshift-bot
Copy link
Copy Markdown
Contributor

openshift-bot commented Jan 14, 2020

/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
Contributor

openshift-bot commented Jan 14, 2020

/retest

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

@openshift-merge-robot openshift-merge-robot merged commit f6452df into openshift:master Jan 14, 2020
@jianzhangbjz
Copy link
Copy Markdown
Contributor

jianzhangbjz commented Jan 15, 2020

@bparees Thanks as always for your help! I will do that, thanks for your suggestions.

@bparees bparees deleted the olmtest branch March 13, 2020 16:07
@deads2k deads2k mentioned this pull request Aug 27, 2020
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@knobunc knobunc Awaiting requested review from knobunc

@sjenning sjenning Awaiting requested review from sjenning

Assignees

@jianzhangbjz jianzhangbjz

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. vendor-update Touching vendor dir or related files

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@bparees @jianzhangbjz @openshift-bot @openshift-ci-robot @openshift-merge-robot