Skip to content

Comments

Remove green tests from baremetal exclusion list#18583

Merged
openshift-merge-robot merged 1 commit intoopenshift:masterfrom
ardaguclu:enable-passing-metal-tests
May 21, 2021
Merged

Remove green tests from baremetal exclusion list#18583
openshift-merge-robot merged 1 commit intoopenshift:masterfrom
ardaguclu:enable-passing-metal-tests

Conversation

@ardaguclu
Copy link
Member

@ardaguclu ardaguclu commented May 14, 2021

This PR removes tests from exclusion list in baremetalds.

@openshift-ci openshift-ci bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label May 14, 2021
@openshift-ci openshift-ci bot requested review from dmage and vrutkovs May 14, 2021 10:40
@ardaguclu ardaguclu force-pushed the enable-passing-metal-tests branch 2 times, most recently from 5c4789e to 5e82c46 Compare May 14, 2021 13:57
@ardaguclu ardaguclu changed the title WIP: Remove green tests from baremetal exclusion list Remove green tests from baremetal exclusion list May 14, 2021
@openshift-ci openshift-ci bot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label May 14, 2021
@stbenjam
Copy link
Member

stbenjam commented May 14, 2021

/approve

@openshift-ci openshift-ci bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label May 14, 2021
@ardaguclu
Copy link
Member Author

ardaguclu commented May 14, 2021

/retest

3 similar comments
@ardaguclu
Copy link
Member Author

ardaguclu commented May 14, 2021

/retest

@ardaguclu
Copy link
Member Author

ardaguclu commented May 15, 2021

/retest

@ardaguclu
Copy link
Member Author

ardaguclu commented May 17, 2021

/retest

@ardaguclu
Copy link
Member Author

ardaguclu commented May 18, 2021

/test pj-rehearse

@ardaguclu
Copy link
Member Author

ardaguclu commented May 20, 2021

/retest

@ardaguclu ardaguclu changed the title Remove green tests from baremetal exclusion list WIP: Remove green tests from baremetal exclusion list May 21, 2021
@openshift-ci openshift-ci bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label May 21, 2021
@openshift-ci
Copy link
Contributor

openshift-ci bot commented May 21, 2021

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

Test name Commit Details Rerun command
ci/rehearse/periodic-ci-openshift-release-master-nightly-4.8-e2e-metal-ipi-compact-trigger c94fc105cc8f67516529f56c84fb0e1fb89e45e6 link /test pj-rehearse
ci/rehearse/periodic-ci-openshift-release-master-nightly-4.8-e2e-metal-ipi-ovn-dualstack-trigger c94fc105cc8f67516529f56c84fb0e1fb89e45e6 link /test pj-rehearse
ci/rehearse/periodic-ci-openshift-release-master-nightly-4.8-e2e-metal-ipi-upgrade 5e82c465fc113f7d6b2f22f96d7f3c95d7caa52f link /test pj-rehearse
ci/prow/pj-rehearse 5e82c465fc113f7d6b2f22f96d7f3c95d7caa52f link /test pj-rehearse
ci/rehearse/openshift/cluster-network-operator/release-4.9/e2e-metal-ipi-ovn-ipv6-ipsec 3603c96adcdab86147547c79a8ed12a8bd0aae98 link /test pj-rehearse
ci/rehearse/openshift/router/release-4.9/e2e-metal-ipi-ovn-ipv6 3603c96adcdab86147547c79a8ed12a8bd0aae98 link /test pj-rehearse

Full PR test history. Your PR dashboard.

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.

@ardaguclu ardaguclu force-pushed the enable-passing-metal-tests branch from 3603c96 to 4f30fd6 Compare May 21, 2021 16:13
@ardaguclu ardaguclu changed the title WIP: Remove green tests from baremetal exclusion list Remove green tests from baremetal exclusion list May 21, 2021
@openshift-ci openshift-ci bot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label May 21, 2021
@andfasano
Copy link
Contributor

andfasano commented May 21, 2021

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label May 21, 2021
@openshift-ci
Copy link
Contributor

openshift-ci bot commented May 21, 2021

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: andfasano, ardaguclu, stbenjam

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-merge-robot openshift-merge-robot merged commit 7841d42 into openshift:master May 21, 2021
@openshift-ci
Copy link
Contributor

openshift-ci bot commented May 21, 2021

@ardaguclu: Updated the step-registry configmap in namespace ci at cluster app.ci using the following files:

  • key baremetalds-e2e-test-ref.yaml using file ci-operator/step-registry/baremetalds/e2e/test/baremetalds-e2e-test-ref.yaml
Details

In response to this:

This PR removes tests from exclusion list in baremetalds.

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.

This was referenced May 22, 2021
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@dmage dmage Awaiting requested review from dmage

@vrutkovs vrutkovs Awaiting requested review from vrutkovs

Assignees

@andfasano andfasano

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.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@ardaguclu @stbenjam @andfasano @openshift-merge-robot