Skip to content

Commit

Permalink
Merge pull request #1995 from chrischdi/pr-1-5-affinity
Browse files Browse the repository at this point in the history
[release-1.5] 🌱 test/e2e: add check at anti-affinity test to ensure enough hosts exist
  • Loading branch information
k8s-ci-robot authored Jul 12, 2023
2 parents 6f4f2a2 + c8dab57 commit 26ff4be
Showing 1 changed file with 8 additions and 3 deletions.
11 changes: 8 additions & 3 deletions test/e2e/anti_affinity_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -59,9 +59,9 @@ var _ = Describe("Cluster creation with anti affined nodes", func() {
})

It("should create a cluster with anti-affined nodes", func() {
// Since the upstream CI has five nodes, worker node count is set to 5
// Since the upstream CI has four nodes, worker node count is set to 4.
VerifyAntiAffinity(ctx, AntiAffinitySpecInput{
WorkerNodeCount: 5,
WorkerNodeCount: 4,
Namespace: namespace,
InfraClients: InfraClients{
Client: vsphereClient,
Expand All @@ -88,7 +88,12 @@ func VerifyAntiAffinity(ctx context.Context, input AntiAffinitySpecInput) {
clusterName := fmt.Sprintf("anti-affinity-%s", util.RandomString(6))
Expect(namespace).NotTo(BeNil())

Byf("creating a workload cluster with")
By("checking if the target system has enough hosts")
hostSystems, err := input.Finder.HostSystemList(ctx, "*")
Expect(err).ToNot(HaveOccurred())
Expect(len(hostSystems) >= int(input.WorkerNodeCount)).To(BeTrue(), "This test requires more or equal number of hosts compared to the WorkerNodeCount. Expected at least %d but only got %d hosts.", input.WorkerNodeCount, len(hostSystems))

Byf("creating a workload cluster %s", clusterName)
configCluster := defaultConfigCluster(clusterName, namespace.Name, "", 1, input.WorkerNodeCount,
input.Global)

Expand Down

0 comments on commit 26ff4be

Please sign in to comment.