Skip to content

Commit

Permalink
Merge pull request #3097 from chrischdi/test-wording-vsphere-yaml
Browse files Browse the repository at this point in the history
🌱 test: fix wording in vsphere.yaml comment, always use bootstrapClusterProxy to get vCenterSimulator
  • Loading branch information
k8s-ci-robot committed Jul 5, 2024
2 parents 075f9d0 + 830d008 commit f3cb37b
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion test/e2e/config/vsphere.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -263,7 +263,7 @@ variables:
EXP_NODE_ANTI_AFFINITY: "true"
CAPI_DIAGNOSTICS_ADDRESS: ":8080"
CAPI_INSECURE_DIAGNOSTICS: "true"
# Required to be set to install capv-supervisor <= v1.10, but getting created.
# Required to be set to install capv-supervisor <= v1.10.
SERVICE_ACCOUNTS_CM_NAMESPACE: "capv-system"
SERVICE_ACCOUNTS_CM_NAME: "service-accounts-cm"

Expand Down
2 changes: 1 addition & 1 deletion test/e2e/e2e_setup_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -111,7 +111,7 @@ func Setup(specName string, f func(testSpecificSettings func() testSettings), op
testSpecificIPAddressClaims, testSpecificVariables = vcsimAddressManager.ClaimIPs(ctx, vsphereip.WithIP(options.additionalIPVariableNames...))

// variables derived from the vCenterSimulator
vCenterSimulator, err := vspherevcsim.Get(ctx, c)
vCenterSimulator, err := vspherevcsim.Get(ctx, bootstrapClusterProxy.GetClient())
Expect(err).ToNot(HaveOccurred(), "Failed to get VCenterSimulator")

Byf("Creating EnvVar %s", klog.KRef(metav1.NamespaceDefault, specName))
Expand Down

0 comments on commit f3cb37b

Please sign in to comment.