Test OwnerReferences and Finalizers for Supervisor

Makefile

@@ -398,6 +398,7 @@ generate-e2e-templates-main: $(KUSTOMIZE) ## Generate test templates for the mai
 	"$(KUSTOMIZE)" --load-restrictor LoadRestrictionsNone build "$(E2E_SUPERVISOR_TEMPLATE_DIR)/main/topology" > "$(E2E_SUPERVISOR_TEMPLATE_DIR)/main/cluster-template-topology-supervisor.yaml"
 	"$(KUSTOMIZE)" --load-restrictor LoadRestrictionsNone build "$(E2E_SUPERVISOR_TEMPLATE_DIR)/main/conformance" > "$(E2E_SUPERVISOR_TEMPLATE_DIR)/main/cluster-template-conformance-supervisor.yaml"
 	"$(KUSTOMIZE)" --load-restrictor LoadRestrictionsNone build "$(E2E_SUPERVISOR_TEMPLATE_DIR)/main/install-on-bootstrap" > "$(E2E_SUPERVISOR_TEMPLATE_DIR)/main/cluster-template-install-on-bootstrap-supervisor.yaml"
+	"$(KUSTOMIZE)" --load-restrictor LoadRestrictionsNone build "$(E2E_SUPERVISOR_TEMPLATE_DIR)/main/ownerrefs-finalizers" > "$(E2E_SUPERVISOR_TEMPLATE_DIR)/main/cluster-template-ownerrefs-finalizers-supervisor.yaml"
 
 .PHONY: generate-e2e-templates-v1.10
 generate-e2e-templates-v1.10: $(KUSTOMIZE)

packaging/flavorgen/cloudprovider/csi.go

@@ -43,7 +43,7 @@ func CSICloudConfigSecret(data string) *corev1.Secret {
 			APIVersion: corev1.SchemeGroupVersion.String(),
 		},
 		ObjectMeta: metav1.ObjectMeta{
-			Name:      "vsphere-config-secret",
+			Name:      "vsphere-config-secret", // NOTE: this name is used in E2E tests.
 			Namespace: CSINamespace,
 		},
 		Type: corev1.SecretTypeOpaque,

packaging/flavorgen/flavors/crs/cpi.go

@@ -99,7 +99,7 @@ func cpiCredentials(credentials map[string]string) *corev1.Secret {
 		},
 		ObjectMeta: metav1.ObjectMeta{
 			Namespace: metav1.NamespaceSystem,
-			Name:      "cloud-provider-vsphere-credentials",
+			Name:      "cloud-provider-vsphere-credentials", // NOTE: this name is used in E2E tests.
 		},
 		Type:       corev1.SecretTypeOpaque,
 		StringData: credentials,

packaging/flavorgen/flavors/crs/util.go

@@ -77,7 +77,7 @@ func appendConfigMapToCrsResource(crs *addonsv1.ClusterResourceSet, generatedCon
 func newCPIConfig() ([]byte, error) {
 	config := map[string]interface{}{
 		"global": map[string]interface{}{
-			"secretName":      "cloud-provider-vsphere-credentials",
+			"secretName":      "cloud-provider-vsphere-credentials", // NOTE: this name is used in E2E tests.
 			"secretNamespace": metav1.NamespaceSystem,
 			"thumbprint":      env.VSphereThumbprint,
 			"port":            443,

packaging/flavorgen/flavors/flavors.go

@@ -108,7 +108,6 @@ func ClusterTopologyTemplateSupervisor() ([]runtime.Object, error) {
 	if err != nil {
 		return nil, err
 	}
-	identitySecret := newIdentitySecret()
 	clusterResourceSet := newClusterResourceSet(cluster)
 	crsResourcesCSI, err := crs.CreateCrsResourceObjectsCSI(&clusterResourceSet)
 	if err != nil {
@@ -117,7 +116,6 @@ func ClusterTopologyTemplateSupervisor() ([]runtime.Object, error) {
 	crsResourcesCPI := crs.CreateCrsResourceObjectsCPI(&clusterResourceSet)
 	MultiNodeTemplate := []runtime.Object{
 		&cluster,
-		&identitySecret,
 		&clusterResourceSet,
 	}
 	MultiNodeTemplate = append(MultiNodeTemplate, crsResourcesCSI...)
@@ -179,7 +177,6 @@ func MultiNodeTemplateSupervisor() ([]runtime.Object, error) {
 		return nil, err
 	}
 	crsResourcesCPI := crs.CreateCrsResourceObjectsCPI(&clusterResourceSet)
-	identitySecret := newIdentitySecret()
 
 	MultiNodeTemplate := []runtime.Object{
 		&cluster,
@@ -190,7 +187,6 @@ func MultiNodeTemplateSupervisor() ([]runtime.Object, error) {
 		&kubeadmJoinTemplate,
 		&machineDeployment,
 		&clusterResourceSet,
-		&identitySecret,
 	}
 
 	MultiNodeTemplate = append(MultiNodeTemplate, crsResourcesCSI...)

templates/cluster-template-supervisor.yaml

@@ -311,15 +311,6 @@ spec:
 ---
 apiVersion: v1
 kind: Secret
-metadata:
-  name: '${CLUSTER_NAME}'
-  namespace: '${NAMESPACE}'
-stringData:
-  password: "${VSPHERE_PASSWORD}"
-  username: "${VSPHERE_USERNAME}"
----
-apiVersion: v1
-kind: Secret
 metadata:
   name: vsphere-config-secret
   namespace: '${NAMESPACE}'

templates/cluster-template-topology-supervisor.yaml

@@ -96,15 +96,6 @@ spec:
         name: md-0
         replicas: ${WORKER_MACHINE_COUNT}
 ---
-apiVersion: v1
-kind: Secret
-metadata:
-  name: '${CLUSTER_NAME}'
-  namespace: '${NAMESPACE}'
-stringData:
-  password: "${VSPHERE_PASSWORD}"
-  username: "${VSPHERE_USERNAME}"
----
 apiVersion: addons.cluster.x-k8s.io/v1beta1
 kind: ClusterResourceSet
 metadata:

test/e2e/config/vsphere.yaml

@@ -178,6 +178,7 @@ providers:
           - sourcePath: "../../../test/e2e/data/infrastructure-vsphere-supervisor/main/clusterclass-quick-start-supervisor.yaml"
           - sourcePath: "../../../test/e2e/data/infrastructure-vsphere-supervisor/main/cluster-template-install-on-bootstrap-supervisor.yaml"
           - sourcePath: "../../../test/e2e/data/infrastructure-vsphere-supervisor/main/cluster-template-conformance-supervisor.yaml"
+          - sourcePath: "../../../test/e2e/data/infrastructure-vsphere-supervisor/main/cluster-template-ownerrefs-finalizers-supervisor.yaml"
           - sourcePath: "../data/shared/capv/main/metadata.yaml"
       - name: "{go://sigs.k8s.io/[email protected]}" # supported release in the v1beta1 series
         # Use manifest from source files

test/e2e/data/infrastructure-vsphere-supervisor/main/ownerrefs-finalizers/kustomization.yaml

@@ -0,0 +1,8 @@
+apiVersion: kustomize.config.k8s.io/v1beta1
+kind: Kustomization
+resources:
+  - ../base
+patchesStrategicMerge:
+  - ../commons/cluster-resource-set-label.yaml
+  - ../commons/cluster-network-CIDR.yaml
+  - ../commons/cluster-resource-set-csi-insecure.yaml

test/e2e/e2e_suite_test.go

@@ -24,6 +24,7 @@ import (
 	"path/filepath"
 	"strings"
 	"testing"
+	"time"
 
 	. "github.com/onsi/ginkgo/v2"
 	. "github.com/onsi/gomega"
@@ -210,8 +211,9 @@ var _ = SynchronizedBeforeSuite(func() []byte {
 
 	if testTarget == VCSimTestTarget {
 		Byf("Creating a vcsim server")
-		err := vspherevcsim.Create(ctx, bootstrapClusterProxy.GetClient())
-		Expect(err).ToNot(HaveOccurred(), "Failed to create VCenterSimulator")
+		Eventually(func() error {
+			return vspherevcsim.Create(ctx, bootstrapClusterProxy.GetClient())
+		}, 30*time.Second, 3*time.Second).ShouldNot(HaveOccurred(), "Failed to create VCenterSimulator")
 	}
 
 	By("Getting AddressClaim labels")