allow assignment of additional tags

Author: rvanderp3

data/data/install.openshift.io_installconfigs.yaml

@@ -4420,6 +4420,14 @@ spec:
                               minLength: 1
                               pattern: ^/.*?/host/.*?/Resources.*
                               type: string
+                            tagIDs:
+                              description: tagIDs is an optional set of tags to add
+                                to an instance. Specified tagIDs must use URN-notation
+                                instead of display names. A maximum of 10 tag IDs
+                                may be specified.
+                              items:
+                                type: string
+                              type: array
                             template:
                               description: template is the inventory path of the virtual
                                 machine or template that will be used for cloning.

pkg/asset/machines/vsphere/machines.go

@@ -247,6 +247,7 @@ func provider(clusterID string, vcenter *vsphere.VCenter, failureDomain vsphere.
 			Folder:       folder,
 			ResourcePool: resourcePool,
 		},
+		TagIDs:            failureDomain.Topology.TagIDs,
 		NumCPUs:           mpool.NumCPUs,
 		NumCoresPerSocket: mpool.NumCoresPerSocket,
 		MemoryMiB:         mpool.MemoryMiB,

pkg/types/vsphere/platform.go

@@ -212,6 +212,11 @@ type Topology struct {
 	// +kubebuilder:validation:Pattern=`^/.*?/vm/.*?`
 	// +optional
 	Template string `json:"template,omitempty"`
+	// tagIDs is an optional set of tags to add to an instance. Specified tagIDs
+	// must use URN-notation instead of display names. A maximum of 10 tag IDs may be specified.
+	// +kubebuilder:example=urn:vmomi:InventoryServiceTag:5736bf56-49f5-4667-b38c-b97e09dc9578:GLOBAL
+	// +optional
+	TagIDs []string `json:"tagIDs,omitempty"`
 }
 
 // VCenter stores the vCenter connection fields

pkg/types/vsphere/validation/platform.go

@@ -117,6 +117,7 @@ func validateVCenters(p *vsphere.Platform, fldPath *field.Path) field.ErrorList
 
 func validateFailureDomains(p *vsphere.Platform, fldPath *field.Path, isLegacyUpi bool) field.ErrorList {
 	var fdNames []string
+	tagUrnPattern := regexp.MustCompile(`^(urn):(vmomi):(InventoryServiceTag):([0-9a-fA-F]{8}-[0-9a-fA-F]{4}-[0-9a-fA-F]{4}-[0-9a-fA-F]{4}-[0-9a-fA-F]{12}):([^:]+)$`)
 	allErrs := field.ErrorList{}
 	topologyFld := fldPath.Child("topology")
 	var associatedVCenter *vsphere.VCenter
@@ -172,6 +173,16 @@ func validateFailureDomains(p *vsphere.Platform, fldPath *field.Path, isLegacyUp
 			}
 		}
 
+		if len(failureDomain.Topology.TagIDs) > 10 {
+			allErrs = append(allErrs, field.Invalid(topologyFld.Child("tagIDs"), failureDomain.Topology.TagIDs, "a maximum of 10 tags are allowed"))
+		}
+
+		for _, tagID := range failureDomain.Topology.TagIDs {
+			if tagUrnPattern.FindStringSubmatch(tagID) == nil {
+				allErrs = append(allErrs, field.Invalid(topologyFld.Child("tagIDs"), failureDomain.Topology.TagIDs, "tag ID must be in the format of urn:vmomi:InventoryServiceTag:<UUID>:GLOBAL"))
+			}
+		}
+
 		if len(failureDomain.Topology.Networks) == 0 {
 			if isLegacyUpi {
 				logrus.Warn("network field empty is now deprecated, in later releases this field will be required.")

pkg/types/vsphere/validation/platform_test.go

@@ -194,6 +194,29 @@ func TestValidatePlatform(t *testing.T) {
 			}(),
 			expectedError: `^test-path\.diskType: Invalid value: "invalidDiskType": diskType must be one of \[eagerZeroedThick thick thin\]$`,
 		},
+		{
+			name: "Additional tag IDs provided",
+			platform: func() *vsphere.Platform {
+				p := validPlatform()
+				p.FailureDomains[0].Topology.TagIDs = []string{
+					"urn:vmomi:InventoryServiceTag:5736bf56-49f5-4667-b38c-b97e09dc9578:GLOBAL",
+					"urn:vmomi:InventoryServiceTag:5736bf56-49f5-4667-b38c-b97e09dc9579:GLOBAL",
+				}
+				return p
+			}(),
+		},
+		{
+			name: "Additional invalid tag IDs provided",
+			platform: func() *vsphere.Platform {
+				p := validPlatform()
+				p.FailureDomains[0].Topology.TagIDs = []string{
+					"urn:bad:InventoryServiceTag:5736bf56-49f5-4667-b38c-b97e09dc9578:GLOBAL",
+					"urn:bad:InventoryServiceTag:5736bf56-49f5-4667-b38c-b97e09dc9579:GLOBAL",
+				}
+				return p
+			}(),
+			expectedError: `^test-path\.failureDomains\.topology\.tagIDs\: Invalid value\:.*?: tag ID must be in the format of urn\:vmomi\:InventoryServiceTag\:<UUID>\:GLOBAL$`,
+		},
 
 		{
 			name:     "Valid Multi-zone platform",