Use new naming scheme for SKUs and versions

[mboersma]

What type of PR is this?:

/kind feature

What this PR does / why we need it:

This catches up with the SKU and version naming changes in kubernetes-sigs/image-builder#878

Which issue(s) this PR fixes:

N/A

Special notes for your reviewer:

TODOs:

• squashed commits
• includes documentation
• adds unit tests

Release note:

Use new naming scheme for SKUs and versions

[netlify]

👷 Deploy request for kubernetes-sigs-cluster-api-provider-azure pending review.

Visit the deploys page to approve it

Name	Link
🔨 Latest commit	50463a5

[mboersma]

/retest

[mboersma]

/retest-required

[mboersma]

@jsturtevant or @Jont828, could you take a look at this when time permits?

[Jont828]

Yeah sure thing, happy to take a look.

[mboersma]

Thanks @Jont828, I made all your suggested changes and they definitely improved the unit tests. 👍🏻

[k8s-ci-robot]

@mboersma: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name	Commit	Details	Required	Rerun command
pull-cluster-api-provider-azure-apidiff	0481cff	link	false	/test pull-cluster-api-provider-azure-apidiff

Full PR test history. Your PR dashboard. Please help us cut down on flakes by linking to an open issue when you hit one in your PR.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

[CecileRobertMichon]

@mboersma I just realized the SKU name change is also going to break our e2e logic that selects the latest SKU available for a given k8s major.minor https://github.com/kubernetes-sigs/cluster-api-provider-azure/blob/main/test/e2e/helpers.go#L777

We can merge this one now to unblock users and address e2e separately, the tests will just be stuck at the latest version that was published with the old naming until we fix them.

/lgtm
/approve

[k8s-ci-robot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: CecileRobertMichon

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:

• OWNERS [CecileRobertMichon]

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

[mboersma]

the tests will just be stuck at the latest version that was published with the old naming until we fix them.

D'oh, you're right. I can take on that change to e2e logic.

[mboersma]

/cherry-pick release-1.3

[k8s-infra-cherrypick-robot]

@mboersma: new pull request created: #2357

In response to this:

/cherry-pick release-1.3

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.