Skip to content

Comments

Enable skipped tests in disconnected for connected workflows#21771

Merged
openshift-merge-robot merged 1 commit intoopenshift:masterfrom
ardaguclu:enable-skipped-tests
Sep 13, 2021
Merged

Enable skipped tests in disconnected for connected workflows#21771
openshift-merge-robot merged 1 commit intoopenshift:masterfrom
ardaguclu:enable-skipped-tests

Conversation

@ardaguclu
Copy link
Member

This PR enables tests, whose are skipped in disconnected configurations,
in connected metal ipi workflows.

These tests can be enabled after openshift/origin#26051 and openshift/origin#26223 has been merged(with also their backports into 4.8)

@openshift-ci openshift-ci bot requested review from kirankt and sadasu September 12, 2021 15:11
@ardaguclu
Copy link
Member Author

/retest

@ardaguclu
Copy link
Member Author

After enabling those tests;
4.9;
https://prow.ci.openshift.org/view/gs/origin-ci-test/pr-logs/pull/openshift_release/21771/rehearse-21771-pull-ci-openshift-cluster-baremetal-operator-release-4.9-e2e-metal-ipi-virtualmedia-trigger/1437147963150307328
https://prow.ci.openshift.org/view/gs/origin-ci-test/pr-logs/pull/openshift_release/21771/rehearse-21771-pull-ci-openshift-cluster-baremetal-operator-release-4.9-e2e-metal-ipi-ovn-ipv6-trigger/1437147962860900352
https://prow.ci.openshift.org/view/gs/origin-ci-test/pr-logs/pull/openshift_release/21771/rehearse-21771-pull-ci-openshift-cluster-baremetal-operator-release-4.9-e2e-metal-ipi-trigger/1437147962999312384
https://prow.ci.openshift.org/view/gs/origin-ci-test/pr-logs/pull/openshift_release/21771/rehearse-21771-pull-ci-openshift-cluster-baremetal-operator-release-4.9-e2e-metal-ipi-ovn-dualstack-trigger/1437147962735071232

4.8;
https://prow.ci.openshift.org/view/gs/origin-ci-test/pr-logs/pull/openshift_release/21771/rehearse-21771-pull-ci-openshift-cluster-baremetal-operator-release-4.8-e2e-metal-ipi-virtualmedia-trigger/1437147962609242112
https://prow.ci.openshift.org/view/gs/origin-ci-test/pr-logs/pull/openshift_release/21771/rehearse-21771-pull-ci-openshift-cluster-baremetal-operator-release-4.8-e2e-metal-ipi-ovn-ipv6-trigger/1437147962357583872
https://prow.ci.openshift.org/view/gs/origin-ci-test/pr-logs/pull/openshift_release/21771/rehearse-21771-pull-ci-openshift-cluster-baremetal-operator-release-4.8-e2e-metal-ipi-trigger/1437147962483412992
https://prow.ci.openshift.org/view/gs/origin-ci-test/pr-logs/pull/openshift_release/21771/rehearse-21771-pull-ci-openshift-cluster-baremetal-operator-release-4.8-e2e-metal-ipi-ovn-dualstack-trigger/1437147962063982592

This PR enables tests, whose are skipped in disconnected configurations,
in connected metal ipi workflows.
@ardaguclu
Copy link
Member Author

/assign @andfasano

@ardaguclu ardaguclu changed the title WIP: Enable skipped tests in disconnected for connected workflows Enable skipped tests in disconnected for connected workflows Sep 13, 2021
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Sep 13, 2021

@ardaguclu: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/rehearse/openshift/cluster-baremetal-operator/release-4.8/e2e-metal-ipi-trigger c9e5eae912c9c92e9b58a9a48d468b68fa22c23c link true /test pj-rehearse
ci/rehearse/openshift/cluster-baremetal-operator/release-4.9/e2e-metal-ipi-virtualmedia-trigger c9e5eae912c9c92e9b58a9a48d468b68fa22c23c link true /test pj-rehearse
ci/rehearse/openshift/cluster-baremetal-operator/release-4.8/e2e-metal-ipi-ovn-dualstack-trigger c9e5eae912c9c92e9b58a9a48d468b68fa22c23c link true /test pj-rehearse
ci/rehearse/periodic-ci-openshift-release-master-nightly-4.9-e2e-metal-ipi-virtualmedia 231d166 link true /test pj-rehearse
ci/rehearse/openshift/router/release-4.9/e2e-metal-ipi-ovn-ipv6 231d166 link true /test pj-rehearse
ci/rehearse/periodic-ci-openshift-release-master-nightly-4.9-e2e-metal-ipi-upgrade 231d166 link true /test pj-rehearse
ci/prow/pj-rehearse 231d166 link false /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
Copy link
Member Author

@andfasano none of the current failing jobs fails due to enabled tests.

@andfasano
Copy link
Contributor

Thanks @ardaguclu

/lgtm

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

openshift-ci bot commented Sep 13, 2021

[APPROVALNOTIFIER] This PR is APPROVED

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

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-ci openshift-ci bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Sep 13, 2021
@openshift-bot
Copy link
Contributor

/retest-required

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

@openshift-merge-robot openshift-merge-robot merged commit bc74cd6 into openshift:master Sep 13, 2021
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Sep 13, 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 enables tests, whose are skipped in disconnected configurations,
in connected metal ipi workflows.

These tests can be enabled after openshift/origin#26051 and openshift/origin#26223 has been merged(with also their backports into 4.8)

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.

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. lgtm Indicates that a PR is ready to be merged.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants