From b8942f20c21e3a12b52439c636ffbeee92190e42 Mon Sep 17 00:00:00 2001
From: Gustavo Alves <112630064+gfariasalves-ionos@users.noreply.github.com>
Date: Thu, 13 Jun 2024 11:42:29 +0200
Subject: [PATCH] =?UTF-8?q?=F0=9F=90=9B=20Use=20correct=20machine=20counts?=
 =?UTF-8?q?=20for=20high=20availability=20test=20(#153)?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

**What is the purpose of this pull request/Why do we need it?**

I just noticed the high availability test creates only 1 control plane
node and 1 worker node. This mistake happened while testing the tests.
This PR fixes the incongruence.

**Issue #, if available:**

N/A

**Description of changes:**

High availability test now creates 3 and 2 CP and worker nodes,
respectively.

**Special notes for your reviewer:**

**Checklist:**
- [x] E2E Tests added
- [x] Includes
[emojis](https://github.com/kubernetes-sigs/kubebuilder-release-tools?tab=readme-ov-file#kubebuilder-project-versioning)
---
 test/e2e/capic_test.go | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/test/e2e/capic_test.go b/test/e2e/capic_test.go
index 54c451c4..1295cc11 100644
--- a/test/e2e/capic_test.go
+++ b/test/e2e/capic_test.go
@@ -40,8 +40,8 @@ var _ = Describe("Should be able to create a cluster with 3 control-plane and 2
 			BootstrapClusterProxy:    bootstrapClusterProxy,
 			ArtifactFolder:           artifactFolder,
 			SkipCleanup:              skipCleanup,
-			ControlPlaneMachineCount: ptr.To[int64](1),
-			WorkerMachineCount:       ptr.To[int64](1),
+			ControlPlaneMachineCount: ptr.To[int64](3),
+			WorkerMachineCount:       ptr.To[int64](2),
 			PostNamespaceCreated:     cloudEnv.createCredentialsSecretPNC,
 			PostMachinesProvisioned: func(proxy framework.ClusterProxy, namespace, clusterName string) {
 				// This check ensures that owner references are resilient - i.e. correctly re-reconciled - when removed.