fix(module): updating default group creation for virtualMachine

modules/virtualMachine/locals.tf

@@ -0,0 +1,6 @@
+locals {
+   fqdn = "${var.vm_shortname}.${var.vm_domain}"
+   system_image_name = "system.${local.fqdn}"
+   system_image_source = var.system_image_source
+   data_image_name = "data.${local.fqdn}"
+}

modules/virtualMachine/main.tf

@@ -1,13 +1,11 @@
-// Dependencies management
+
-resource "null_resource" "depends_on" {
+resource "opennebula_group" "main" {
-  triggers = {
+  name = var.group
-    depends_on = join("", var.depends)
-  }
 }
 
 // OpenNebula disk image !
 resource "opennebula_image" "system" {
-    name = var.system_image_name
+    name = local.system_image_name
     description = "System disk image"
     datastore_id = var.one_prod_ds 
     persistent = false
@@ -21,7 +19,7 @@ resource "opennebula_image" "system" {
 }
 
 resource "opennebula_image" "data" {
-    name = var.data_image_name
+    name = local.data_image_name
     description = "Data disk image"
     datastore_id = var.one_prod_ds 
     persistent = true
@@ -31,7 +29,7 @@ resource "opennebula_image" "data" {
     driver = "qcow2"
     format = "qcow2"
     permissions = "660"
-    group = "production"  
+    group = var.group 
 }
 
 // OpenNebula (VM) Template definition
@@ -59,6 +57,10 @@ resource "opennebula_template" "vm_template" {
     image_id = opennebula_image.data.id
   }
 
+  os {
+    arch = "x86_64"
+    boot = "disk0,nic0"
+  }
   //labels = var.tag_labels
 
   dynamic "nic" {
@@ -78,12 +80,10 @@ resource "opennebula_template" "vm_template" {
 
 // OpenNebula Virtual Machine
 resource "opennebula_virtual_machine" "vm" {
-  name = var.vm_fqdn
+  name = local.fqdn
   group = var.group
-  timeout = 10
   template_id = opennebula_template.vm_template.id
-  depends_on = [ opennebula_template.vm_template,
+  depends_on = [ opennebula_template.vm_template ]
-                 null_resource.depends_on ]
 }
 
 resource "dns_a_record_set" "vm_dns_a_record" {    
@@ -102,7 +102,7 @@ resource "dns_cname_record" "vm_dns_cname_record" {
 
   zone  = var.vm_domain 
   name  = each.value
-  cname = "${var.vm_fqdn}."
+  cname = "${local.fqdn}."
   ttl   = 300
 
   depends_on = [ opennebula_virtual_machine.vm ]

modules/virtualMachine/outputs.tf

@@ -1,9 +1,13 @@
-output "virtual_machine_shortname" {
+output "shortname" {
   value = opennebula_virtual_machine.vm.name
 }
 
-output "virtual_machine_nic0_mac" {
+output "fqdn" {
-  value = opennebula_virtual_machine.vm.nic[0].mac
+  value = local.fqdn 
+}
+
+output "first_nic_mac" {
+  value = opennebula_virtual_machine.vm.template_nic[0].computed_mac
 }
 
 output "network_interfaces" {

modules/virtualMachine/variables.tf

@@ -8,11 +8,6 @@ variable "one_prod_ds" {
   type = string
 }
 
-variable "vm_fqdn" {
-  description = "Virtual Machine FQDN Name"
-  type = string
-}
-
 variable "vm_shortname" {
   description = "Virtual machine short name"
   type = string
@@ -23,24 +18,16 @@ variable "vm_domain" {
   type = string
 }
 
-variable "system_image_name" {
-  description = "System image disk name"
-  type = string
-}
-
 variable "system_image_source" {
   description = "Virtual Machine system image source"
   type = string
-}
+  default = "https://marketplace.opennebula.io//appliance/251b0578-6c47-4ba4-b184-dc6e873201a2/download/0"
-
-variable "data_image_name" {
-  description = "Data image disk name"
-  type = string
 }
 
 variable "data_image_source" {
   description = "Virtual Machine data disk image source"
   type = string
+  default = "https://marketplace.opennebula.io//appliance/251b0578-6c47-4ba4-b184-dc6e873201a2/download/0"
 }
 
 variable "tag_labels" {
@@ -56,16 +43,19 @@ variable "network_interfaces" {
 variable "cpu" {
   description = "VM CPU"
   type = string
+  default = "0.2"
 }
 
 variable "vcpu" {
   description = "VM vCPU"
   type = string
+  default = "2"
 }
 
 variable "ram" {
   description = "VM RAM"
   type = string
+  default = "512"
 }
 
 variable "depends" {
@@ -105,5 +95,5 @@ variable "graphics" {
 variable "imagemode" {
   description = "Disk image mode"
   type = string
-  default = "0600"
+  default = "600"
 }