Skip to content

Commit 61dde8d

Browse files
committed
fixup! feat: cluster per EL onboarding
1 parent f1dd511 commit 61dde8d

File tree

5 files changed

+16
-13
lines changed

5 files changed

+16
-13
lines changed

examples/onboarding-with-existing-aks-cluster/main.tf

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -17,9 +17,6 @@ module "castai_omni_cluster" {
1717
pod_cidr = data.azurerm_kubernetes_cluster.aks.network_profile[0].pod_cidr
1818
service_cidr = data.azurerm_kubernetes_cluster.aks.network_profile[0].service_cidr
1919

20-
storage_provider = ""
21-
loadbalancer_provider = ""
22-
2320
skip_helm = var.skip_helm
2421
}
2522

examples/onboarding-with-existing-eks-cluster/main.tf

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -22,9 +22,6 @@ module "castai_omni_cluster" {
2222
pod_cidr = data.aws_vpc.eks_vpc.cidr_block
2323
service_cidr = data.aws_eks_cluster.eks.kubernetes_network_config[0].service_ipv4_cidr
2424

25-
storage_provider = "gp3"
26-
loadbalancer_provider = "external" // External requires AWS Load Balancer Controller to be installed in the cluster. Set to "nlb" to use legacy in-tree LB provider.
27-
2825
skip_helm = var.skip_helm
2926
}
3027

examples/onboarding-with-existing-gke-cluster/main.tf

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -38,9 +38,6 @@ module "castai_omni_cluster" {
3838
service_cidr = data.google_container_cluster.gke.services_ipv4_cidr
3939
reserved_subnet_cidrs = [data.google_compute_subnetwork.gke_subnet.ip_cidr_range]
4040

41-
storage_provider = "premium-rwo"
42-
loadbalancer_provider = ""
43-
4441
skip_helm = var.skip_helm
4542
}
4643

main.tf

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -48,6 +48,9 @@ locals {
4848
castai_helm_repository = "https://castai.github.io/helm-charts"
4949
castai_agent_secret_ref = "castai-omni-agent-token"
5050

51+
storage_provider = var.storage_provider != null ? var.storage_provider : (var.k8s_provider == "gke" ? "premium-rwo" : (var.k8s_provider == "eks" ? "gp3" : ""))
52+
loadbalancer_provider = var.loadbalancer_provider != null ? var.loadbalancer_provider : (var.k8s_provider == "eks" ? "external" : "")
53+
5154
# Select the appropriate yaml_values based on k8s_provider
5255
liqo_yaml_values = merge(
5356
{ for v in module.liqo_helm_values_gke : "gke" => v.liqo_yaml_values },
@@ -60,15 +63,16 @@ locals {
6063
helm_yaml_values = {
6164
castai = {
6265
apiUrl = var.api_url
66+
grpcAddr = var.kvisor_grpc_url
6367
apiKeySecretRef = local.castai_agent_secret_ref
6468
organizationID = var.organization_id
6569
clusterID = var.cluster_id
6670
clusterName = var.cluster_name
6771
}
6872
liqo = local.liqo_yaml_values[var.k8s_provider].liqo
6973
edgeCluster = {
70-
storageProvider = var.storage_provider
71-
loadBalancerProvider = var.loadbalancer_provider
74+
storageProvider = local.storage_provider
75+
loadBalancerProvider = local.loadbalancer_provider
7276
}
7377
}
7478
}

variables.tf

Lines changed: 10 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,12 @@ variable "api_url" {
1313
default = "https://api.cast.ai"
1414
}
1515

16+
variable "kvisor_grpc_url" {
17+
description = "Kvisor gRPC URL"
18+
type = string
19+
default = "kvisor.prod-master.cast.ai:443"
20+
}
21+
1622
variable "api_token" {
1723
description = "CAST AI API token (key) for authentication"
1824
type = string
@@ -56,13 +62,15 @@ variable "reserved_subnet_cidrs" {
5662
}
5763

5864
variable "storage_provider" {
59-
description = "Storage provider for the edge clusters. Recommended are `premium-rwo` for GKE and `gp3` for EKS. Leave empty to use the default storageclass for your cluster."
65+
description = "Storage provider (storageclass) for the edge clusters. If empty, they will be defaulted to `premium-rwo` for GKE and `gp3` for EKS"
6066
type = string
67+
default = null
6168
}
6269

6370
variable "loadbalancer_provider" {
64-
description = "LoadBalancer provider for edge cluster. This setting is used only for EKS clusters: accepted values: `nlb` and `external`)."
71+
description = "LoadBalancer provider for edge cluster. This setting is used only for EKS clusters (accepted values are `nlb` and `external`). If empty, it will be defaulted to `external` for EKS"
6572
type = string
73+
default = null
6674
}
6775

6876
variable "omni_agent_chart_version" {

0 commit comments

Comments
 (0)