From 5f7cf81400e6602e0c7aec23ac16203b46a99922 Mon Sep 17 00:00:00 2001
From: Adam Rush <adamrushuk@gmail.com>
Date: Tue, 23 Feb 2021 08:29:20 +0000
Subject: [PATCH] Added load_balancer_sku, defaulted to Basic

---
 main.tf      |  8 ++++----
 test/main.tf |  2 +-
 variables.tf | 34 ++++++++++++++++++++--------------
 3 files changed, 25 insertions(+), 19 deletions(-)

diff --git a/main.tf b/main.tf
index 3f507c6..d286c8a 100644
--- a/main.tf
+++ b/main.tf
@@ -140,7 +140,7 @@ resource "azurerm_kubernetes_cluster" "aks" {
 
   # https://registry.terraform.io/providers/hashicorp/azurerm/latest/docs/resources/kubernetes_cluster#network_plugin
   network_profile {
-    load_balancer_sku  = "Standard"
+    load_balancer_sku  = var.load_balancer_sku
     outbound_type      = "loadBalancer"
     network_plugin     = "azure"
     network_policy     = "azure"
@@ -155,9 +155,9 @@ resource "azurerm_kubernetes_cluster" "aks" {
 # Add role to access AKS Resource View
 # https://docs.microsoft.com/en-us/azure/aks/kubernetes-portal
 resource "azurerm_role_assignment" "aks_portal_resource_view" {
-  principal_id                     = azuread_group.aks_admins[0].id
-  role_definition_name             = "Azure Kubernetes Service RBAC Cluster Admin"
-  scope                            = azurerm_kubernetes_cluster.aks.id
+  principal_id         = azuread_group.aks_admins[0].id
+  role_definition_name = "Azure Kubernetes Service RBAC Cluster Admin"
+  scope                = azurerm_kubernetes_cluster.aks.id
 }
 
 
diff --git a/test/main.tf b/test/main.tf
index b9d6c92..8d6e7d3 100644
--- a/test/main.tf
+++ b/test/main.tf
@@ -1,6 +1,6 @@
 # Test AKS module
 provider "azurerm" {
-  version = "2.46.1"
+  version = "2.47.0"
   features {}
 }
 
diff --git a/variables.tf b/variables.tf
index 5bb1f4e..c15e030 100644
--- a/variables.tf
+++ b/variables.tf
@@ -33,6 +33,12 @@ variable "sla_sku" {
   default     = "Free"
 }
 
+variable "load_balancer_sku" {
+  description = "Specifies the SKU of the Load Balancer used for this Kubernetes Cluster"
+  type        = string
+  default     = "Basic"
+}
+
 variable "tags" {
   description = "A map of the tags to use on the resources"
   type        = map(string)
@@ -57,20 +63,20 @@ variable "default_node_pool" {
 Default node pool configuration. Overrides/merges with locals.default_agent_profile:
 ```
 map(object({
-    name                  = string
-    count                 = number
-    vm_size               = string
-    os_type               = string
-    availability_zones    = list(number)
-    enable_auto_scaling   = bool
-    min_count             = number
-    max_count             = number
-    type                  = string
-    node_taints           = list(string)
-    vnet_subnet_id        = string
-    max_pods              = number
-    os_disk_size_gb       = number
-    enable_node_public_ip = bool
+  name                  = string
+  count                 = number
+  vm_size               = string
+  os_type               = string
+  availability_zones    = list(number)
+  enable_auto_scaling   = bool
+  min_count             = number
+  max_count             = number
+  type                  = string
+  node_taints           = list(string)
+  vnet_subnet_id        = string
+  max_pods              = number
+  os_disk_size_gb       = number
+  enable_node_public_ip = bool
 }))
 ```
 EOD