search

metal3-io / metal3-docs

Architecture documentation that describes the components being built under Metal³.
http://metal3.io
Apache License 2.0
263 stars 111 forks source link

Design: remove ironic-inspector in favor of ironic alone #362

Closed dtantsur closed 7 months ago

dtantsur commented 8 months ago

/cc @hardys

hardys commented 8 months ago

/cc @hardys

Thanks, overall lgtm - my main concern is that we clearly define/communicate the approach for those downstream consumers who may not be able to immediately switch to the latest Ironic release.

A BMO stable branch seems like a good solution, but it would perhaps be helpful to expand on the details around that (BMO stable release cadence, backport policy, define dependency to specific OpenStack release)

lentzi90 commented 8 months ago

I take it the LTS discussion is done? From what I understand we will keep supporting ironic-inspector in some kind of long term branch of BMO. Sounds good to me /lgtm

dtantsur commented 7 months ago

I take it the LTS discussion is done?

Yeah. The contention point was about ironic-image LTS branches. This design does not require them.

elfosardo commented 7 months ago

/approve

adilGhaffarDev commented 7 months ago

cc @kashifest @Rozzii @zaneb This one needs approve tag. Please check.

zaneb commented 7 months ago

/approve

metal3-io-bot commented 7 months ago

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: elfosardo, zaneb

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

The pull request process is described here

Needs approval from an approver in each of these files: - ~~[design/OWNERS](https://github.com/metal3-io/metal3-docs/blob/main/design/OWNERS)~~ [zaneb] Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment