search

metal3-io / ironic-image

Container image to run OpenStack Ironic as part of Metal³
Apache License 2.0
56 stars 117 forks source link

[DNM] test ci #475

Closed elfosardo closed 6 months ago

elfosardo commented 7 months ago

What this PR does / why we need it:

Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged): Fixes #

metal3-io-bot commented 7 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 elfosardo. 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/ironic-image/blob/release-24.0/OWNERS)** Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment
elfosardo commented 7 months ago

/test-ubuntu-integration-main

elfosardo commented 7 months ago

/markdownlint

elfosardo commented 7 months ago

/test-ubuntu-integration-main

elfosardo commented 7 months ago

/test-ubuntu-integration-main

tuminoid commented 7 months ago

/test-ubuntu-integration-main //test-centos-integration-main

tuminoid commented 7 months ago

/test-centos-integration-main

tuminoid commented 7 months ago

/test-ubuntu-integration-main /test-centos-integration-main

tuminoid commented 7 months ago

/test-ubuntu-integration-main /test-centos-integration-main

tuminoid commented 7 months ago

/test-ubuntu-integration-main /test-centos-integration-main

tuminoid commented 7 months ago

/test-ubuntu-integration-main /test-centos-integration-main

kashifest commented 7 months ago

the triggers doesnt look correct, we using main branch test trigger here, perhaps we should use the correct branch name. Let me check if everything is up to date in gerrit repo

kashifest commented 7 months ago

/test-ironic-image-build

kashifest commented 7 months ago

ok it seems like we do not have the logic to test the release branches yet , cause for that we have to decide which branch of BMO we should test for a given ironic release branch, currently we run these integration tests based on CAPM3 branches and CAPM3 and BMO branches are coupled together for the integration tests (we should change this), we can also use BMO e2e tests here but then again we need to know for which branch of BMO we should test which branch of ironic!!! the ironic-image-build pipeline is another thing that we can leverage, currently its only building ironic image and that also from rpm for some legacy reason, we can perhaps improve this and add options to test integration of different ironic image branches with BMO/CAPM3.