From a3d470c51570404a70bf3a4357d39214cdb6073c Mon Sep 17 00:00:00 2001
From: Tero Tervala <tero.tervala@unikie.com>
Date: Tue, 27 Feb 2024 10:50:37 +0200
Subject: [PATCH] Make osdisk size configurable

Default osdisk sizes:
- jenkins-controller: 150 GB
- builder: 150 GB
- binary-cache: 50 GB

Signed-off-by: Tero Tervala <tero.tervala@unikie.com>
---
 terraform/binary-cache.tf                             | 4 ++--
 terraform/builder.tf                                  | 4 ++--
 terraform/jenkins-controller.tf                       | 4 ++--
 terraform/modules/azurerm-linux-vm/variables.tf       | 7 +++++--
 terraform/modules/azurerm-linux-vm/virtual_machine.tf | 5 ++---
 5 files changed, 13 insertions(+), 11 deletions(-)

diff --git a/terraform/binary-cache.tf b/terraform/binary-cache.tf
index 485c443b..cd1a4524 100644
--- a/terraform/binary-cache.tf
+++ b/terraform/binary-cache.tf
@@ -1,5 +1,4 @@
-# SPDX-FileCopyrightText: 2023 Technology Innovation Institute (TII)
-#
+# SPDX-FileCopyrightText: 2022-2024 Technology Innovation Institute (TII)
 # SPDX-License-Identifier: Apache-2.0
 
 module "binary_cache_image" {
@@ -22,6 +21,7 @@ module "binary_cache_vm" {
   location                     = azurerm_resource_group.infra.location
   virtual_machine_name         = "ghaf-binary-cache-${local.env}"
   virtual_machine_size         = local.opts[local.conf].vm_size_binarycache
+  virtual_machine_osdisk_size  = "50"
   virtual_machine_source_image = module.binary_cache_image.image_id
 
   virtual_machine_custom_data = join("\n", ["#cloud-config", yamlencode({
diff --git a/terraform/builder.tf b/terraform/builder.tf
index 99deddd0..ae7f2e90 100644
--- a/terraform/builder.tf
+++ b/terraform/builder.tf
@@ -1,5 +1,4 @@
-# SPDX-FileCopyrightText: 2023 Technology Innovation Institute (TII)
-#
+# SPDX-FileCopyrightText: 2022-2024 Technology Innovation Institute (TII)
 # SPDX-License-Identifier: Apache-2.0
 
 module "builder_image" {
@@ -28,6 +27,7 @@ module "builder_vm" {
   location                     = azurerm_resource_group.infra.location
   virtual_machine_name         = "ghaf-builder-${count.index}-${local.env}"
   virtual_machine_size         = local.opts[local.conf].vm_size_builder
+  virtual_machine_osdisk_size  = "150"
   virtual_machine_source_image = module.builder_image.image_id
 
   virtual_machine_custom_data = join("\n", ["#cloud-config", yamlencode({
diff --git a/terraform/jenkins-controller.tf b/terraform/jenkins-controller.tf
index 9ce4461e..25ae9f13 100644
--- a/terraform/jenkins-controller.tf
+++ b/terraform/jenkins-controller.tf
@@ -1,5 +1,4 @@
-# SPDX-FileCopyrightText: 2023 Technology Innovation Institute (TII)
-#
+# SPDX-FileCopyrightText: 2022-2024 Technology Innovation Institute (TII)
 # SPDX-License-Identifier: Apache-2.0
 
 # Build the Jenkins controller image
@@ -24,6 +23,7 @@ module "jenkins_controller_vm" {
   location                     = azurerm_resource_group.infra.location
   virtual_machine_name         = "ghaf-jenkins-controller-${local.env}"
   virtual_machine_size         = local.opts[local.conf].vm_size_controller
+  virtual_machine_osdisk_size  = "150"
   virtual_machine_source_image = module.jenkins_controller_image.image_id
 
   virtual_machine_custom_data = join("\n", ["#cloud-config", yamlencode({
diff --git a/terraform/modules/azurerm-linux-vm/variables.tf b/terraform/modules/azurerm-linux-vm/variables.tf
index 88be7e64..5deb6c79 100644
--- a/terraform/modules/azurerm-linux-vm/variables.tf
+++ b/terraform/modules/azurerm-linux-vm/variables.tf
@@ -1,5 +1,4 @@
-# SPDX-FileCopyrightText: 2023 Technology Innovation Institute (TII)
-#
+# SPDX-FileCopyrightText: 2022-2024 Technology Innovation Institute (TII)
 # SPDX-License-Identifier: Apache-2.0
 
 variable "resource_group_name" {
@@ -18,6 +17,10 @@ variable "virtual_machine_size" {
   type = string
 }
 
+variable "virtual_machine_osdisk_size" {
+  type = string
+}
+
 variable "virtual_machine_source_image" {
   type = string
 }
diff --git a/terraform/modules/azurerm-linux-vm/virtual_machine.tf b/terraform/modules/azurerm-linux-vm/virtual_machine.tf
index de7f7c2e..01d779ed 100644
--- a/terraform/modules/azurerm-linux-vm/virtual_machine.tf
+++ b/terraform/modules/azurerm-linux-vm/virtual_machine.tf
@@ -1,5 +1,4 @@
-# SPDX-FileCopyrightText: 2023 Technology Innovation Institute (TII)
-#
+# SPDX-FileCopyrightText: 2022-2024 Technology Innovation Institute (TII)
 # SPDX-License-Identifier: Apache-2.0
 
 resource "azurerm_virtual_machine" "main" {
@@ -56,7 +55,7 @@ resource "azurerm_virtual_machine" "main" {
     caching           = "ReadWrite"
     create_option     = "FromImage"
     managed_disk_type = "Standard_LRS"
-    disk_size_gb      = "100"
+    disk_size_gb      = var.virtual_machine_osdisk_size
   }
 
   dynamic "storage_data_disk" {