Skip to content

add extended-platform-tests binary to e2e template, run e2e-extended-platform in kcmo #6789

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.

Sign up for GitHub

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

Jump to bottom
Closed
sallyom wants to merge 2 commits into from
Closed

add extended-platform-tests binary to e2e template, run e2e-extended-platform in kcmo #6789

sallyom wants to merge 2 commits into from

Conversation

@sallyom
Copy link
Contributor

@sallyom sallyom commented Jan 21, 2020 •
edited
Loading

This adds a test job to a single openshift repo, kcmo, to verify the new test binaries include all test suites and that suites pass.

Add e2e-new target in kube-controller-manager-operator repo. This will be a temporary target to verify new test binaries can replace old openshift-tests binary.
e2e-new runs the ci template that uses extended-platform-tests, openshfit-tests-kubernetes binaries built from openshift/openshift-tests repository.
Depends on openshift/openshift-tests#12

This will be the last PR that needs to pass b4 renaming extended-platform-tests image to release:tests and then modifying ci templates to run new binaries.

/assign @soltysh
/cc @tnozicka @deads2k

@openshift-ci-robot openshift-ci-robot added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label Jan 21, 2020
@sallyom sallyom mentioned this pull request Jan 21, 2020
Closed
@sallyom sallyom force-pushed the kcm-new-job branch 2 times, most recently from 7f1d9d1 to 894f451 Compare January 22, 2020 02:12
@sallyom sallyom changed the title test job for new qe test-suite add extended-platform-tests binary to e2e template, run e2e-extended-platform in kcmo Jan 22, 2020
@sallyom sallyom mentioned this pull request Jan 22, 2020
Merged
@deads2k
Copy link
Contributor

deads2k commented Jan 22, 2020

/retest

2 similar comments
@deads2k
Copy link
Contributor

deads2k commented Jan 22, 2020

/retest

@deads2k
Copy link
Contributor

deads2k commented Jan 22, 2020

/retest

@deads2k
Copy link
Contributor

deads2k commented Jan 22, 2020

I hate teraform

/retest

@deads2k
Copy link
Contributor

deads2k commented Jan 22, 2020

/retest

@deads2k
Copy link
Contributor

deads2k commented Jan 23, 2020

terraform is going to become a four letter work

/retest

@deads2k
Copy link
Contributor

deads2k commented Jan 23, 2020

terraform

/retest

@mfojtik
Copy link
Contributor

mfojtik commented Jan 23, 2020

/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Jan 23, 2020
@deads2k
Copy link
Contributor

deads2k commented Jan 23, 2020

w00t. it worked.

/hold

@openshift-ci-robot openshift-ci-robot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Jan 23, 2020
@openshift-ci-robot openshift-ci-robot removed the lgtm Indicates that a PR is ready to be merged. label Feb 18, 2020
@openshift-ci-robot
Copy link
Contributor

openshift-ci-robot commented Feb 18, 2020

New changes are detected. LGTM label has been removed.

@openshift-ci-robot openshift-ci-robot added size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. and removed size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Feb 18, 2020
@openshift-ci-robot
Copy link
Contributor

openshift-ci-robot commented Feb 18, 2020

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: mfojtik, sallyom
To complete the pull request process, please assign hongkailiu, wking
You can assign the PR to them by writing /assign @hongkailiu @wking in a comment when ready.

The full list of commands accepted by this bot can be found 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

@sallyom
Copy link
Contributor Author

sallyom commented Feb 18, 2020

/hold

need this first: openshift/openshift-tests#12

@sallyom sallyom mentioned this pull request Feb 18, 2020
Closed
@openshift-ci-robot openshift-ci-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Mar 9, 2020
@openshift-ci-robot
Copy link
Contributor

openshift-ci-robot commented Mar 9, 2020

@sallyom: PR needs rebase.

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.

1 similar comment
@openshift-ci-robot
Copy link
Contributor

openshift-ci-robot commented Mar 9, 2020

@sallyom: PR needs rebase.

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.

@openshift-ci-robot
Copy link
Contributor

openshift-ci-robot commented Mar 12, 2020

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

Test name Commit Details Rerun command
ci/rehearse/openshift/cluster-kube-controller-manager-operator/master/e2e-aws-qe 3d6fa96f9684d26bacc57313920a46327a56ba41 link /test pj-rehearse
ci/prow/ordered-prow-config 3739058 link /test ordered-prow-config
ci/prow/generated-config 3739058 link /test generated-config
ci/prow/prow-config 3739058 link /test prow-config
ci/rehearse/openshift/cluster-kube-controller-manager-operator/master/e2e-new 3739058 link /test pj-rehearse
ci/prow/pj-rehearse 3739058 link /test pj-rehearse
ci/prow/app-ci-config 3739058 link /test app-ci-config
ci/prow/generated-cluster 3739058 link /test generated-cluster

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.

@sallyom
Copy link
Contributor Author

sallyom commented Apr 3, 2020

openshift-tests-kubernetes is going in a different direction, closing this

@sallyom sallyom closed this Apr 3, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mfojtik mfojtik Awaiting requested review from mfojtik

@soltysh soltysh Awaiting requested review from soltysh

Assignees

@mfojtik mfojtik

Labels

do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@sallyom @deads2k @mfojtik @openshift-ci-robot