Skip to content

Exclude more tests from disconnected environment#26987

Closed
elfosardo wants to merge 1 commit intoopenshift:masterfrom
elfosardo:moar-disconnected
Closed

Exclude more tests from disconnected environment#26987
elfosardo wants to merge 1 commit intoopenshift:masterfrom
elfosardo:moar-disconnected

Conversation

@elfosardo
Copy link
Contributor

The Feature:Router tests should really all not run in disconnected
environment.

@openshift-ci openshift-ci bot requested review from jaypoulz and sinnykumari April 7, 2022 10:02
@elfosardo
Copy link
Contributor Author

/test e2e-metal-ipi-ovn-ipv6
/test e2e-metal-ipi-ovn-dualstack

The Feature:Router tests should really all not run in disconnected
environment.
@elfosardo elfosardo force-pushed the moar-disconnected branch from e46bfdc to 611c935 Compare April 7, 2022 12:19
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Apr 7, 2022

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: elfosardo
To complete the pull request process, please assign derekwaynecarr after the PR has been reviewed.
You can assign the PR to them by writing /assign @derekwaynecarr 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

@elfosardo
Copy link
Contributor Author

/hold
this may work even better #26979

@openshift-ci openshift-ci bot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Apr 7, 2022
@elfosardo
Copy link
Contributor Author

/test e2e-metal-ipi-ovn-ipv6
/test e2e-metal-ipi-ovn-dualstack

@openshift-ci openshift-ci bot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Apr 7, 2022
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Apr 7, 2022

@elfosardo: 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.

@elfosardo elfosardo closed this Apr 7, 2022
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Apr 7, 2022

@elfosardo: 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/prow/e2e-gcp-csi 611c935 link false /test e2e-gcp-csi
ci/prow/e2e-metal-ipi-ovn-dualstack 611c935 link false /test e2e-metal-ipi-ovn-dualstack
ci/prow/e2e-agnostic-cmd 611c935 link false /test e2e-agnostic-cmd
ci/prow/e2e-gcp 611c935 link true /test e2e-gcp
ci/prow/e2e-metal-ipi-ovn-ipv6 611c935 link false /test e2e-metal-ipi-ovn-ipv6

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

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.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

Comments