metal3-io / baremetal-operator

Bare metal host provisioning integration for Kubernetes
Apache License 2.0
570 stars 253 forks source link

WIP: 🌱 Use equinix runner instead of ubuntu for e2e tests #1775

Open kashifest opened 4 months ago

metal3-io-bot commented 4 months ago

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: Once this PR has been reviewed and has the lgtm label, please ask for approval from kashifest. For more information see the Kubernetes Code Review Process.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files: - **[OWNERS](https://github.com/metal3-io/baremetal-operator/blob/main/OWNERS)** Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment
tuminoid commented 3 months ago

/retest

tuminoid commented 3 months ago

Please add proper description and link to the relevant GH doc.

It seems, e2e pipeline did not get a runner, as they are just "failed" with no run logs. Do we really want to use Equinix as they seem to be off most of the time?

kashifest commented 3 months ago

Please add proper description and link to the relevant GH doc.

Its a WIP so not ready for review, I will add description once we know it works.

kashifest commented 3 months ago

It seems, e2e pipeline did not get a runner, as they are just "failed" with no run logs. Do we really want to use Equinix as they seem to be off most of the time?

We were suggested to try them in the CNCF cluster issue here https://github.com/cncf/cluster/issues/266#issuecomment-2153144241 . I have replied back what we experienced here.

Rozzii commented 2 months ago

/retest

Rozzii commented 2 months ago

/ok-to-test

kashifest commented 3 weeks ago

/retest