origin
origin copied to clipboard
openshift
Remove test 'clone repository using git:// protocol should clone using git:// if no proxy is configured'
This test was previously marked as skipped because github disabled the git:// protocol. I assume the intention was to find another way to test the use of git:// (maybe with some other platform?) but the bug is closed and doesn't appear there's any intention to fix the test.
Despite the skip, this test still shows up in TRT's top failers list because the BeforeEach/AfterEach stuff is still executed as well as obtaining an oc client and this sometimes fail when other catastrophic things happened in the cluster[1].
This test is just creating noise, so I believe it should just be removed.
[1] https://sippy.dptools.openshift.org/sippy-ng/tests/4.12?filters=%257B%2522items%2522%253A%255B%257B%2522columnField%2522%253A%2522current_runs%2522%252C%2522operatorValue%2522%253A%2522%253E%253D%2522%252C%2522value%2522%253A%25227%2522%257D%252C%257B%2522columnField%2522%253A%2522variants%2522%252C%2522not%2522%253Atrue%252C%2522operatorValue%2522%253A%2522contains%2522%252C%2522value%2522%253A%2522never-stable%2522%257D%252C%257B%2522id%2522%253A99%252C%2522columnField%2522%253A%2522name%2522%252C%2522operatorValue%2522%253A%2522contains%2522%252C%2522value%2522%253A%2522git%253A%252F%252F%2522%257D%255D%252C%2522linkOperator%2522%253A%2522and%2522%257D&sort=asc&sortField=current_working_percentage
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: coreydaley, stbenjam
The full list of commands accepted by this bot can be found here.
The pull request process is described here
- ~~test/extended/builds/OWNERS~~ [coreydaley,stbenjam]
- ~~test/extended/util/annotate/generated/OWNERS~~ [coreydaley,stbenjam]
Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment
![openshift-ci[bot] avatar](https://avatars.githubusercontent.com/in/91280?v=4 "Published by a pub.dev verified publisher"){kind=small}
/retest-required
Remaining retests: 0 against base HEAD 1948c27d654abb0e8a4b29b72113d1bea8ad3643 and 2 for PR HEAD 757c24f381090a20a0bd11dbc3f36921fa287edb in total
/retest-required
Remaining retests: 0 against base HEAD c9444d43cdd58ffadc3aec0e0b5ce4556806190a and 1 for PR HEAD 757c24f381090a20a0bd11dbc3f36921fa287edb in total
/retest-required
Remaining retests: 0 against base HEAD 54548ade0f7bd86c00d9aa9e91f51ed51f32bea1 and 0 for PR HEAD 757c24f381090a20a0bd11dbc3f36921fa287edb in total
/hold
Revision 757c24f381090a20a0bd11dbc3f36921fa287edb was retested 3 times: holding
@stbenjam: 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-aws-ovn-single-node | 757c24f381090a20a0bd11dbc3f36921fa287edb | link | false | /test e2e-aws-ovn-single-node |
| ci/prow/e2e-aws-ovn-single-node-upgrade | 757c24f381090a20a0bd11dbc3f36921fa287edb | link | false | /test e2e-aws-ovn-single-node-upgrade |
Full PR test history. Your PR dashboard.
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.
/retest-required
Remaining retests: 0 against base HEAD 53173121bd86b940c4a70e2105ea4ebcbddec2bd and 2 for PR HEAD 757c24f381090a20a0bd11dbc3f36921fa287edb in total
/override ci/prow/e2e-aws-ovn-serial /override ci/prow/e2e-gcp-builds /skip
@stbenjam: Overrode contexts on behalf of stbenjam: ci/prow/e2e-aws-ovn-serial, ci/prow/e2e-gcp-builds
In response to this:
/override ci/prow/e2e-aws-ovn-serial /override ci/prow/e2e-gcp-builds /skip
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.
/retest-required
Remaining retests: 0 against base HEAD fde2d78faf225da2ae20d01ccfb67ef87ec36243 and 1 for PR HEAD 757c24f381090a20a0bd11dbc3f36921fa287edb in total
/override ci/prow/e2e-aws-ovn-serial /override ci/prow/e2e-gcp-ovn-upgrade
@stbenjam: Overrode contexts on behalf of stbenjam: ci/prow/e2e-aws-ovn-serial, ci/prow/e2e-gcp-ovn-upgrade
In response to this:
/override ci/prow/e2e-aws-ovn-serial /override ci/prow/e2e-gcp-ovn-upgrade
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.
@stbenjam: Overrode contexts on behalf of stbenjam: ci/prow/e2e-gcp-ovn-upgrade
In response to this:
/override ci/prow/e2e-gcp-ovn-upgrade
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.
@stbenjam: Overrode contexts on behalf of stbenjam: ci/prow/e2e-aws-ovn-serial
In response to this:
/override ci/prow/e2e-aws-ovn-serial
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.
Oh come on /override ci/prow/e2e-gcp-ovn /override ci/prow/e2e-gcp-builds /override ci/prow/e2e-aws-ovn-fips
@stbenjam: /override requires failed status contexts, check run or a prowjob name to operate on. The following unknown contexts/checkruns were given:
- ``
Only the following failed contexts/checkruns were expected:
ci/prow/e2e-agnostic-ovn-cmdci/prow/e2e-aws-csici/prow/e2e-aws-ovn-cgroupsv2ci/prow/e2e-aws-ovn-fipsci/prow/e2e-aws-ovn-serialci/prow/e2e-aws-ovn-single-nodeci/prow/e2e-aws-ovn-single-node-serialci/prow/e2e-aws-ovn-single-node-upgradeci/prow/e2e-gcp-buildsci/prow/e2e-gcp-csici/prow/e2e-gcp-ovnci/prow/e2e-gcp-ovn-rt-upgradeci/prow/e2e-gcp-ovn-upgradeci/prow/e2e-metal-ipi-ovn-ipv6ci/prow/imagesci/prow/lintci/prow/verifyci/prow/verify-depspull-ci-openshift-origin-master-e2e-agnostic-ovn-cmdpull-ci-openshift-origin-master-e2e-aws-csipull-ci-openshift-origin-master-e2e-aws-ovn-cgroupsv2pull-ci-openshift-origin-master-e2e-aws-ovn-fipspull-ci-openshift-origin-master-e2e-aws-ovn-serialpull-ci-openshift-origin-master-e2e-aws-ovn-single-nodepull-ci-openshift-origin-master-e2e-aws-ovn-single-node-serialpull-ci-openshift-origin-master-e2e-aws-ovn-single-node-upgradepull-ci-openshift-origin-master-e2e-gcp-buildspull-ci-openshift-origin-master-e2e-gcp-csipull-ci-openshift-origin-master-e2e-gcp-ovnpull-ci-openshift-origin-master-e2e-gcp-ovn-rt-upgradepull-ci-openshift-origin-master-e2e-gcp-ovn-upgradepull-ci-openshift-origin-master-e2e-metal-ipi-ovn-ipv6pull-ci-openshift-origin-master-imagespull-ci-openshift-origin-master-lintpull-ci-openshift-origin-master-verifypull-ci-openshift-origin-master-verify-depstide
If you are trying to override a checkrun that has a space in it, you must put a double quote on the context.
In response to this:
Oh come on /override ci/prow/e2e-gcp-ovn /override ci/prow/e2e-gcp-builds /override ci/prow/e2e-aws-ovn-fips
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.
/override ci/prow/e2e-gcp-ovn /override ci/prow/e2e-gcp-builds /override ci/prow/e2e-aws-ovn-fips
@stbenjam: Overrode contexts on behalf of stbenjam: ci/prow/e2e-aws-ovn-fips, ci/prow/e2e-gcp-builds, ci/prow/e2e-gcp-ovn
In response to this:
/override ci/prow/e2e-gcp-ovn /override ci/prow/e2e-gcp-builds /override ci/prow/e2e-aws-ovn-fips
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 has been churning through CI for more than a week, it's just deleting a test.
/skip /override ci/prow/e2e-agnostic-ovn-cmd /override ci/prow/e2e-aws-csi /override ci/prow/e2e-aws-ovn-cgroupsv2 /override ci/prow/e2e-aws-ovn-fips /override ci/prow/e2e-aws-ovn-serial /override ci/prow/e2e-aws-ovn-single-node /override ci/prow/e2e-aws-ovn-single-node-serial /override ci/prow/e2e-aws-ovn-single-node-upgrade /override ci/prow/e2e-gcp-builds /override ci/prow/e2e-gcp-csi /override ci/prow/e2e-gcp-ovn /override ci/prow/e2e-gcp-ovn-rt-upgrade /override ci/prow/e2e-gcp-ovn-upgrade /override ci/prow/e2e-metal-ipi-ovn-ipv6
@stbenjam: Overrode contexts on behalf of stbenjam: ci/prow/e2e-agnostic-ovn-cmd, ci/prow/e2e-aws-csi, ci/prow/e2e-aws-ovn-cgroupsv2, ci/prow/e2e-aws-ovn-fips, ci/prow/e2e-aws-ovn-serial, ci/prow/e2e-aws-ovn-single-node, ci/prow/e2e-aws-ovn-single-node-serial, ci/prow/e2e-aws-ovn-single-node-upgrade, ci/prow/e2e-gcp-builds, ci/prow/e2e-gcp-csi, ci/prow/e2e-gcp-ovn, ci/prow/e2e-gcp-ovn-rt-upgrade, ci/prow/e2e-gcp-ovn-upgrade, ci/prow/e2e-metal-ipi-ovn-ipv6
In response to this:
This has been churning through CI for more than a week, it's just deleting a test.
/skip /override ci/prow/e2e-agnostic-ovn-cmd /override ci/prow/e2e-aws-csi /override ci/prow/e2e-aws-ovn-cgroupsv2 /override ci/prow/e2e-aws-ovn-fips /override ci/prow/e2e-aws-ovn-serial /override ci/prow/e2e-aws-ovn-single-node /override ci/prow/e2e-aws-ovn-single-node-serial /override ci/prow/e2e-aws-ovn-single-node-upgrade /override ci/prow/e2e-gcp-builds /override ci/prow/e2e-gcp-csi /override ci/prow/e2e-gcp-ovn /override ci/prow/e2e-gcp-ovn-rt-upgrade /override ci/prow/e2e-gcp-ovn-upgrade /override ci/prow/e2e-metal-ipi-ovn-ipv6
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.