Closed clobrano closed 6 months ago
Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all
/test 4.15-openshift-e2e
/test 4.15-openshift-e2e
/test 4.15-openshift-e2e
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: clobrano, slintes
The full list of commands accepted by this bot can be found here.
The pull request process is described here
/retest
@clobrano: 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/4.13-openshift-e2e | a3be2568d1dd632877d506d083cd289e72fe56ad | link | unknown | /test 4.13-openshift-e2e |
ci/prow/4.12-openshift-e2e | a3be2568d1dd632877d506d083cd289e72fe56ad | link | unknown | /test 4.12-openshift-e2e |
Full PR test history. Your PR dashboard.
I've been allucinated here. I merged the branch without noticing the failed tests :facepalm:
I've been allucinated here. I merged the branch without noticing the failed tests 🤦
Since 4.14 + passed, you are most likely in the clear. To make sure let's keep an eye on unrelated #303 and see that all the tests are clear.
Why we need this PR
Currently, NHC only checks if it runs on OCP before trying to access MachineAPI resources, however not all OCP clusters have MachineAPI enabled and the controller fails to start in such conditions.
Changes made
Verify if MachineAPI is in the list of the Cluster's EnabledCapabilities
Which issue(s) this PR fixes
https://issues.redhat.com/browse/ECOPROJECT-1902
Todo