chore: bump cluster install versions

[Jakob Gray (JakobGray)]

What

Update cluster install versions

Why

Use latest available z version

Testing

Special notes for your reviewer

PR Checklist

• PR is scoped to a single task (no mixed concerns)
• Title follows Conventional Commits format
• Summary explains the "Why" behind the change
• Linked to relevant ticket/issue
• Screenshots included (if graph/UI/metrics changes)
• Self-reviewed the diff
• CI/CD checks are passing (ignore Tide)
• Draft PR used for WIP (if applicable)
• Commit history is clean (rebased/squashed)
• Tricky code blocks are commented
• Specific reviewers tagged
• All comment threads resolved before merge

[Copilot]

Pull request overview

Updates the default OpenShift patch versions used when converting RP-provided X.Y versions into Cluster Service “openshift-vX.Y.Z” identifiers, aligning unit test expectations and default builders with the new patch levels.

Changes:

• Bump default stable patch versions for 4.20 and 4.21 (and associated expected values in tests).
• Update NewOpenShiftVersionXYZ’s hardcoded X.Y→patch mapping to reflect the new defaults.
• Refresh default cluster/nodepool version expectations in internal/ocm tests.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File	Description
internal/ocm/convert_test.go	Updates expected CS version IDs and defaults to match the bumped patch versions.
internal/ocm/client.go	Updates the X.Y→patch mapping used to synthesize CS-readable version IDs.

[Manyanda Chitimbo (machi1990)]

/lgtm

[Manyanda Chitimbo (machi1990)]

/lgtm cancel

[Jakob Gray (JakobGray)]

/test e2e-parallel

[Jakob Gray (JakobGray)]

/test e2e-parallel

[Manyanda Chitimbo (machi1990)]

/lgtm

[Copilot]

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

Comments suppressed due to low confidence (1)

internal/ocm/client.go:813

• When v includes a prerelease suffix but is still in X.Y format (e.g. 4.22-0.nightly-...), the patch lookup uses the original v (including the suffix) so it won’t match the map and will incorrectly fall back to patch 0. Since versionPart is already split out, the lookup key should be based on the parsed X.Y (and the comment referencing PR #4477 looks stale now that we still maintain an explicit mapping).

		if len(parts) == 2 {
			// Patch version is managed by Red Hat. This will be computed automatically to the latest
			// as part of https://github.com/Azure/ARO-HCP/pull/4477
			if patch, ok := map[string]string{"4.19": "34", "4.20": "25", "4.21": "20", "4.22": "1"}[v]; ok {
				parts = append(parts, patch)
			} else {
				parts = append(parts, "0")
			}

[Manyanda Chitimbo (machi1990)]

/lgtm

[Gerd Oberlechner (geoberle)]

/approve

[openshift-ci]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: geoberle, JakobGray, machi1990

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:

• internal/OWNERS [geoberle]

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

[Jakob Gray (JakobGray)]

/test e2e-parallel

[Jakob Gray (JakobGray)]

/test e2e-parallel

[Jakob Gray (JakobGray)]

/test e2e-parallel

[openshift-merge-bot]

/retest-required

Remaining retests: 0 against base HEAD 30228fa and 2 for PR HEAD 1111c46 in total