From 82c52098e7ffde330c557a748966f88030a745d9 Mon Sep 17 00:00:00 2001 From: Treehopper Date: Wed, 24 Nov 2021 15:56:20 +0000 Subject: [PATCH 1/6] Fix: "Values of identifierUris property must use a verified domain of the organization or its subdomain" Full error: Changing "aks-getting-started-sp" to a valid URI of "http://aks-getting-started-sp", which is the required format used for service principal names Values of identifierUris property must use a verified domain of the organization or its subdomain: 'http://aks-getting-started-sp' Replaced with: WARNING: 'name' property in the output is deprecated and will be removed in the future. Use 'appId' instead. --- kubernetes/cloud/azure/terraform/readme.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/kubernetes/cloud/azure/terraform/readme.md b/kubernetes/cloud/azure/terraform/readme.md index 222919e..79ab8ce 100644 --- a/kubernetes/cloud/azure/terraform/readme.md +++ b/kubernetes/cloud/azure/terraform/readme.md @@ -11,7 +11,7 @@ We'll need the Azure CLI to gather information so we can build our Terraform fil ``` # Run Azure CLI -docker run -it --rm -v ${PWD}:/work -w /work --entrypoint /bin/sh mcr.microsoft.com/azure-cli:2.6.0 +docker run -it --rm -v ${PWD}:/work -w /work --entrypoint /bin/sh mcr.microsoft.com/azure-cli:latest ``` From 12ebd066941d689d765b3a5946634c0bd85ed389 Mon Sep 17 00:00:00 2001 From: Treehopper Date: Wed, 24 Nov 2021 15:57:44 +0000 Subject: [PATCH 2/6] Update to Terraform 1.0.11. --- kubernetes/cloud/azure/terraform/modules/k8s/k8s.tf | 4 ++-- kubernetes/cloud/azure/terraform/readme.md | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/kubernetes/cloud/azure/terraform/modules/k8s/k8s.tf b/kubernetes/cloud/azure/terraform/modules/k8s/k8s.tf index 4adf09a..512aca5 100644 --- a/kubernetes/cloud/azure/terraform/modules/k8s/k8s.tf +++ b/kubernetes/cloud/azure/terraform/modules/k8s/k8s.tf @@ -38,11 +38,11 @@ resource "kubernetes_deployment" "example" { name = "example" resources { - limits { + limits = { cpu = "0.5" memory = "512Mi" } - requests { + requests = { cpu = "250m" memory = "50Mi" } diff --git a/kubernetes/cloud/azure/terraform/readme.md b/kubernetes/cloud/azure/terraform/readme.md index 79ab8ce..66936c6 100644 --- a/kubernetes/cloud/azure/terraform/readme.md +++ b/kubernetes/cloud/azure/terraform/readme.md @@ -62,7 +62,7 @@ For extra reference you can also take a look at the Microsoft Docs: [here](https ``` # Get Terraform -curl -o /tmp/terraform.zip -LO https://releases.hashicorp.com/terraform/0.12.28/terraform_0.12.28_linux_amd64.zip +curl -o /tmp/terraform.zip -LO https://releases.hashicorp.com/terraform/1.0.11/terraform_1.0.11_linux_amd64.zip unzip /tmp/terraform.zip chmod +x terraform && mv terraform /usr/local/bin/ From e6c40a7f8a165d5d68c8a18d2cf7ed4fbb1dba81 Mon Sep 17 00:00:00 2001 From: Treehopper Date: Wed, 24 Nov 2021 16:22:11 +0000 Subject: [PATCH 3/6] Fix: Version 1.16.10 is not supported in this region. Full: Error: creating Managed Kubernetes Cluster "aks-getting-started" (Resource Group "aks-getting-started"): containerservice.ManagedClustersClient#CreateOrUpdate: Failure sending request: StatusCode=400 -- Original Error: Code="AgentPoolK8sVersionNotSupported" Message="Version 1.16.10 is not supported in this region. Please use [az aks get-versions] command to get the supported version list in this region. For more information, please check https://aka.ms/supported-version-list" --- kubernetes/cloud/azure/terraform/variables.tf | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/kubernetes/cloud/azure/terraform/variables.tf b/kubernetes/cloud/azure/terraform/variables.tf index 27c6ee4..fe81512 100644 --- a/kubernetes/cloud/azure/terraform/variables.tf +++ b/kubernetes/cloud/azure/terraform/variables.tf @@ -19,5 +19,5 @@ variable "location" { } variable "kubernetes_version" { - default = "1.16.10" + default = "1.22.2" } \ No newline at end of file From ab38006fc8a77e52c25963adf70b2e8f24b3e8b2 Mon Sep 17 00:00:00 2001 From: Treehopper Date: Thu, 25 Nov 2021 09:29:21 +0000 Subject: [PATCH 4/6] Use fixed version for reproducibility. --- kubernetes/cloud/azure/terraform/readme.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/kubernetes/cloud/azure/terraform/readme.md b/kubernetes/cloud/azure/terraform/readme.md index 66936c6..19eb6fe 100644 --- a/kubernetes/cloud/azure/terraform/readme.md +++ b/kubernetes/cloud/azure/terraform/readme.md @@ -11,7 +11,7 @@ We'll need the Azure CLI to gather information so we can build our Terraform fil ``` # Run Azure CLI -docker run -it --rm -v ${PWD}:/work -w /work --entrypoint /bin/sh mcr.microsoft.com/azure-cli:latest +docker run -it --rm -v ${PWD}:/work -w /work --entrypoint /bin/sh mcr.microsoft.com/azure-cli:2.30.0 ``` From 99a7535702e227d4e2dcb45018853ac87274eb89 Mon Sep 17 00:00:00 2001 From: Treehopper Date: Thu, 25 Nov 2021 13:11:21 +0000 Subject: [PATCH 5/6] Fix terraform error: Unsupported argument option has been dropped in terraform 2.x --- kubernetes/cloud/azure/terraform/modules/k8s/k8s.tf | 1 - 1 file changed, 1 deletion(-) diff --git a/kubernetes/cloud/azure/terraform/modules/k8s/k8s.tf b/kubernetes/cloud/azure/terraform/modules/k8s/k8s.tf index 512aca5..8d53a5c 100644 --- a/kubernetes/cloud/azure/terraform/modules/k8s/k8s.tf +++ b/kubernetes/cloud/azure/terraform/modules/k8s/k8s.tf @@ -1,6 +1,5 @@ provider "kubernetes" { - load_config_file = "false" host = var.host client_certificate = var.client_certificate client_key = var.client_key From f88562ce1f64680f7c06b4f82b5569096b75eb88 Mon Sep 17 00:00:00 2001 From: Treehopper Date: Thu, 25 Nov 2021 13:19:07 +0000 Subject: [PATCH 6/6] Fix Warning: Version constraints inside provider configuration blocks are deprecated. --- kubernetes/cloud/azure/terraform/main.tf | 11 +++++++++-- 1 file changed, 9 insertions(+), 2 deletions(-) diff --git a/kubernetes/cloud/azure/terraform/main.tf b/kubernetes/cloud/azure/terraform/main.tf index a7ad7d5..52e7d12 100644 --- a/kubernetes/cloud/azure/terraform/main.tf +++ b/kubernetes/cloud/azure/terraform/main.tf @@ -1,6 +1,13 @@ -provider "azurerm" { - version = "=2.5.0" +terraform { + required_providers { + azurerm = { + source = "hashicorp/azurerm" + version = "=2.5.0" + } + } +} +provider "azurerm" { subscription_id = var.subscription_id client_id = var.serviceprinciple_id client_secret = var.serviceprinciple_key