✨ Switch to Ironic 32.0 by default, prepare release-0.6

[elfosardo]

Signed-off-by: Riccardo Pittau elfosardo@gmail.com

[elfosardo]

/hold
wait for metal3-io/project-infra#1112 to merge

[elfosardo]

/cc @tuminoid

[tuminoid]

Some test fixes needed:

--- FAIL: TestValidateIronic (0.00s)
    --- FAIL: TestValidateIronic/with_unsupported_version (0.00s)
        validation_test.go:343: 
            	Error Trace:	/home/runner/work/ironic-standalone-operator/ironic-standalone-operator/pkg/ironic/validation_test.go:343
            	Error:      	Error "version 42.42 is not supported, supported versions are 30.0, 31.0, 32.0, latest" does not contain "version 42.42 is not supported, supported versions are 30.0, 31.0, latest"
            	Test:       	TestValidateIronic/with_unsupported_version

[dtantsur]

I think we've skipped the entire step of preparing the main branch before branching :-/ We need to update supported versions there, add missing tests, then, I guess, backport that commit.

[elfosardo]

@dtantsur I just realized that there are discrepancies in main, specifically the new version of ironic has been added, but of course not the new release
we still need to update https://github.com/metal3-io/ironic-standalone-operator/blob/main/docs/releasing.md
this is still on hold, I will update main with what needs to be updated before merging this

[elfosardo]

this is the update on main #390

[dtantsur]

specifically the new version of ironic has been added, but of course not the new release

Correct, because I needed a version check, but the release itself did not exist at that point.

[dtantsur]

/approve

[metal3-io-bot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: dtantsur

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

The pull request process is described here

Details

Needs approval from an approver in each of these files:

• OWNERS [dtantsur]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[tuminoid]

@dtantsur I just realized that there are discrepancies in main, specifically the new version of ironic has been added, but of course not the new release we still need to update https://github.com/metal3-io/ironic-standalone-operator/blob/main/docs/releasing.md this is still on hold, I will update main with what needs to be updated before merging this

I think the IRSO process has gone bit off track versus the intended release process here. The branch etc should be created by the release automation, for example 0.6.0-beta.0 (beta cuts branch, alpha doesn't). Then you implement the CI support for the branch etc. Then you merge stuff there to cut the stable. As mentioned the 32.0 support should've also been in main before that.

Do we have some shortcomings in the release docs related to the workflow, or is it not working for IRSO?

[elfosardo]

@tuminoid simply put, the release docs for IrSO is outdated
we need to add at least a mention to verify support for recent ironic-image releases, the rest kind of work, just need a refresh

[elfosardo]

this #391 is needed before release preparation

[tuminoid]

@tuminoid simply put, the release docs for IrSO is outdated we need to add at least a mention to verify support for recent ironic-image releases, the rest kind of work, just need a refresh

Maybe we can amend verify-release.sh to check for these as well.

[elfosardo]

/unhold

[tuminoid]

/lgtm