PowerVS: Fix VPC subnets

The cluster-api-provider-ibmcloud uses a fixed set of subnets per VPC.  So, until we can ask
the provider what it is, bring that data in so we can use it.

Author: hamzy

pkg/asset/manifests/cloudproviderconfig.go

@@ -257,16 +257,18 @@ func (cpc *CloudProviderConfig) Generate(dependencies asset.Parents) error {
 
 		if len(vpcSubnets) == 0 {
 			if capiutils.IsEnabled(installConfig) {
+				vpcZones, err := powervstypes.AvailableVPCZones(installConfig.Config.PowerVS.Region)
+				if err != nil {
+					return err
+				}
+
 				// The PowerVS CAPI provider generates three subnets.  One for
 				// each of the endpoint.
 				// @TODO the provider should export a function which gives us
 				// an array
-				for i := 1; i <= 3; i++ {
+				for _, zone := range vpcZones {
 					vpcSubnets = append(vpcSubnets,
-						fmt.Sprintf("%s-vpcsubnet-%s-%d",
-							clusterID.InfraID,
-							vpcRegion,
-							i))
+						fmt.Sprintf("%s-vpcsubnet-%s", clusterID.InfraID, zone))
 				}
 			} else {
 				vpcSubnets = append(vpcSubnets, fmt.Sprintf("vpc-subnet-%s", clusterID.InfraID))

pkg/types/powervs/powervs_regions.go

@@ -17,6 +17,7 @@ type Region struct {
 	COSRegion   string
 	Zones       []string
 	SysTypes    []string
+	VPCZones    []string
 }
 
 // Regions holds the regions for IBM Power VS, and descriptions used during the survey.
@@ -27,34 +28,39 @@ var Regions = map[string]Region{
 		COSRegion:   "us-south",
 		Zones:       []string{"dal10", "dal12"},
 		SysTypes:    []string{"s922", "e980"},
+		VPCZones:    []string{"us-south-1", "us-south-2", "us-south-3"},
 	},
 	"eu-de": {
 		Description: "Frankfurt, Germany",
 		VPCRegion:   "eu-de",
 		COSRegion:   "eu-de",
 		Zones:       []string{"eu-de-1", "eu-de-2"},
 		SysTypes:    []string{"s922", "e980"},
+		VPCZones:    []string{"eu-de-2", "eu-de-3"},
 	},
 	"mad": {
 		Description: "Madrid, Spain",
 		VPCRegion:   "eu-es",
 		COSRegion:   "eu-de", // @HACK - PowerVS says COS not supported in this region
 		Zones:       []string{"mad02", "mad04"},
 		SysTypes:    []string{"s1022"},
+		VPCZones:    []string{"eu-es-1", "eu-es-2"},
 	},
 	"sao": {
 		Description: "São Paulo, Brazil",
 		VPCRegion:   "br-sao",
 		COSRegion:   "br-sao",
 		Zones:       []string{"sao04"},
 		SysTypes:    []string{"s922", "e980"},
+		VPCZones:    []string{"br-sao-1", "br-sao-2"},
 	},
 	"wdc": {
 		Description: "Washington DC, USA",
 		VPCRegion:   "us-east",
 		COSRegion:   "us-east",
 		Zones:       []string{"wdc06", "wdc07"},
 		SysTypes:    []string{"s922", "e980"},
+		VPCZones:    []string{"us-east-1", "us-east-2", "us-east-3"},
 	},
 }
 
@@ -143,6 +149,15 @@ func AllKnownSysTypes() sets.Set[string] {
 	return sysTypes
 }
 
+// AvailableVPCZones returns the known VPC zones for a specified region.
+func AvailableVPCZones(region string) ([]string, error) {
+	knownRegion, ok := Regions[region]
+	if !ok {
+		return nil, fmt.Errorf("unknown region name provided")
+	}
+	return knownRegion.VPCZones, nil
+}
+
 // COSRegionForVPCRegion returns the corresponding COS region for the given VPC region.
 func COSRegionForVPCRegion(vpcRegion string) (string, error) {
 	for r := range Regions {