terraform: Fix random VM image deployment errors

Signed-off-by: Henri Rosten <[email protected]>
tiiuae · Dec 9, 2024 · 3a52328 · 3a52328
1 parent 4070ca0
commit 3a52328
Show file tree

Hide file tree

Showing 5 changed files with 15 additions and 1 deletion.
diff --git a/terraform/binary-cache.tf b/terraform/binary-cache.tf
@@ -16,6 +16,7 @@ module "binary_cache_image" {
   location               = azurerm_resource_group.infra.location
   storage_account_name   = azurerm_storage_account.vm_images.name
   storage_container_name = azurerm_storage_container.vm_images.name
+  depends_on             = [azurerm_storage_container.vm_images]
 }
 
 module "binary_cache_vm" {

diff --git a/terraform/builder.tf b/terraform/builder.tf
@@ -17,6 +17,7 @@ module "builder_image" {
   location               = azurerm_resource_group.infra.location
   storage_account_name   = azurerm_storage_account.vm_images.name
   storage_container_name = azurerm_storage_container.vm_images.name
+  depends_on             = [azurerm_storage_container.vm_images]
 }
 
 locals {

diff --git a/terraform/jenkins-controller.tf b/terraform/jenkins-controller.tf
@@ -17,6 +17,7 @@ module "jenkins_controller_image" {
   location               = azurerm_resource_group.infra.location
   storage_account_name   = azurerm_storage_account.vm_images.name
   storage_container_name = azurerm_storage_container.vm_images.name
+  depends_on             = [azurerm_storage_container.vm_images]
 }
 
 # Create a machine using this image

diff --git a/terraform/main.tf b/terraform/main.tf
@@ -216,9 +216,16 @@ resource "azurerm_subnet" "builders" {
   address_prefixes     = ["10.0.4.0/28"]
 }
 
+# https://github.com/hashicorp/terraform-provider-azurerm/issues/15609
+resource "random_string" "id" {
+  length  = 8
+  upper   = false
+  special = false
+}
+
 # Storage account and storage container used to store VM images
 resource "azurerm_storage_account" "vm_images" {
-  name                            = "img${local.ws}${local.shortloc}"
+  name                            = "img${random_string.id.result}"
   resource_group_name             = azurerm_resource_group.infra.name
   location                        = azurerm_resource_group.infra.location
   account_tier                    = "Standard"

diff --git a/terraform/modules/azurerm-nix-vm-image/main.tf b/terraform/modules/azurerm-nix-vm-image/main.tf
@@ -38,6 +38,10 @@ resource "azurerm_image" "default" {
     os_state = "Generalized"
     os_type  = "Linux"
   }
+  depends_on = [
+    azurerm_storage_blob.default,
+    data.external.nix_build
+  ]
 }
 
 output "image_id" {