Fix: instance_count + ad_number == null distributes instances per AD (#94)

kral2 · web-flow · commit 6c81f58b48a5 · 2022-01-29T06:19:31.000+01:00
when var.ad_number == null data.oci_core_shapes.current_ad cannot be created anymore. We currently rely on var.ad_number == null to provision instances in each AD in a round robin manner when var.instance_count is used. Fixes: #93
diff --git a/main.tf b/main.tf
@@ -46,7 +46,7 @@ data "oci_core_subnet" "instance_subnet" {
 // This will not check quota and limits for AD requested at resource creation
 data "oci_core_shapes" "current_ad" {
   compartment_id      = var.compartment_ocid
-  availability_domain = local.ADs[(var.ad_number - 1)]
+  availability_domain = var.ad_number == null ? element(local.ADs, 0) : element(local.ADs, var.ad_number - 1)
 }
 
 locals {

Original file line number	Diff line number	Diff line change
`@@ -46,7 +46,7 @@ data "oci_core_subnet" "instance_subnet" {`
`46`	`46`	`// This will not check quota and limits for AD requested at resource creation`
`47`	`47`	`data "oci_core_shapes" "current_ad" {`
`48`	`48`	`compartment_id = var.compartment_ocid`
`49`		`- availability_domain = local.ADs[(var.ad_number - 1)]`
	`49`	`+ availability_domain = var.ad_number == null ? element(local.ADs, 0) : element(local.ADs, var.ad_number - 1)`
`50`	`50`	`}`
`51`	`51`
`52`	`52`	`locals {`