From 891e10cd3080d7c4472ec813ac5b4120bbc72ca2 Mon Sep 17 00:00:00 2001
From: Rahul Sharma <rahsharm@akamai.com>
Date: Wed, 27 Nov 2024 18:47:29 +0000
Subject: [PATCH] address review comments

---
 Makefile | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/Makefile b/Makefile
index dea6dc43..17c5f76c 100644
--- a/Makefile
+++ b/Makefile
@@ -22,6 +22,7 @@ CONTROLPLANE_NODES      ?= 1
 WORKER_NODES            ?= 1
 LINODE_FIREWALL_ENABLED ?= true
 LINODE_REGION           ?= us-lax
+LINODE_OS               ?= linode/ubuntu22.04
 KUBECONFIG_PATH         ?= $(CURDIR)/test-cluster-kubeconfig.yaml
 
 # if the $DEVBOX_PACKAGES_DIR env variable exists that means we are within a devbox shell and can safely
@@ -144,9 +145,8 @@ capl-cluster: generate-capl-cluster-manifests create-capl-cluster patch-linode-c
 .PHONY: generate-capl-cluster-manifests
 generate-capl-cluster-manifests:
 	# Create the CAPL cluster manifests without any CSI driver stuff
-	LINODE_FIREWALL_ENABLED=$(LINODE_FIREWALL_ENABLED) clusterctl generate cluster $(CLUSTER_NAME) \
-		--kubernetes-version $(K8S_VERSION) \
-		--infrastructure linode-linode:$(CAPL_VERSION) \
+	LINODE_FIREWALL_ENABLED=$(LINODE_FIREWALL_ENABLED) LINODE_OS=$(LINODE_OS) clusterctl generate cluster $(CLUSTER_NAME) \
+		--kubernetes-version $(K8S_VERSION) --infrastructure linode-linode:$(CAPL_VERSION) \
 		--control-plane-machine-count $(CONTROLPLANE_NODES) --worker-machine-count $(WORKER_NODES) > capl-cluster-manifests.yaml
 
 .PHONY: create-capl-cluster
@@ -179,9 +179,9 @@ mgmt-cluster:
 
 .PHONY: cleanup-cluster
 cleanup-cluster:
-	kubectl delete cluster --all
-	kubectl delete linodefirewalls --all
-	kubectl delete lvpc --all
+	kubectl delete cluster -A --all --timeout=180s
+	kubectl delete linodefirewalls -A --all --timeout=60s
+	kubectl delete lvpc -A --all --timeout=60s
 	kind delete cluster -n caplccm
 
 .PHONY: e2e-test