From 37ad816a696fa1f6c4f97a70e676d76cc55a2c9f Mon Sep 17 00:00:00 2001 From: "francesco.mangiacrapa" Date: Tue, 5 Dec 2023 10:06:23 +0100 Subject: [PATCH] Added server_groups_ids as input paramater and managed it --- openstack-tf/d4s-dev/geoportal/main.tf | 30 ++++++++++++++-- openstack-tf/d4s-dev/geoserver_new/main.tf | 27 +++++++++----- openstack-tf/d4s-dev/uri-resolver/main.tf | 27 +++++++++++--- .../modules/common_variables/outputs.tf | 36 ++++++++++--------- .../modules/common_variables/variables.tf | 23 ++++++++---- .../generic_smartgears_service.tf | 18 ++++++++-- .../generic_smartgears_service/outputs.tf | 4 +++ .../variables_generic_smartgears_service.tf | 12 +++++-- .../modules/geo_services/geo_service.tf | 12 ++++++- openstack-tf/modules/geo_services/inputs.tf | 7 ++-- 10 files changed, 150 insertions(+), 46 deletions(-) diff --git a/openstack-tf/d4s-dev/geoportal/main.tf b/openstack-tf/d4s-dev/geoportal/main.tf index 4755adb..c7ce5cd 100644 --- a/openstack-tf/d4s-dev/geoportal/main.tf +++ b/openstack-tf/d4s-dev/geoportal/main.tf @@ -18,17 +18,43 @@ data "terraform_remote_state" "privnet_dns_router" { } +# +# Uses common_variables as module +# +module "common_variables" { + source = "../../modules/common_variables" +} + + +# +# Creates the server group "geoportal" +# +resource "openstack_compute_servergroup_v2" "geoportal" { + name = "geoportal" + policies = [module.common_variables.policy_list.soft_anti_affinity] +} + + +# +# Uses the "generic_smartgears_service" as module +# module "smartgears_service_generic" { source = "../../modules/generic_smartgears_service" + # Here, you can overwrite the openstack_vm_data_scripts to run for ubuntuXYZ.sh + # user_data_var_file = { + # user_data_file = "../../openstack_vm_data_scripts/ubuntu2204.sh" + # } + smartgears_service_instances_map = { geoportal_service = { name = "geoportal-cms", description = "The Geoportal instance", flavor = "m1.medium", networks = ["d4s-dev-cloud-main", "postgresql-srv-net"], - security_groups = ["default", "http and https from the load balancers"] - block_device_uuid = "7ed6a2cd-2b07-482e-8ce4-f018dff16c89" #ubuntu_18_04.uuid of DEV + security_groups = ["default", "http and https from the load balancers"], + server_groups_ids = [openstack_compute_servergroup_v2.geoportal.id], + image_uuid = "7ed6a2cd-2b07-482e-8ce4-f018dff16c89" #ubuntu_18_04.uuid of DEV } } } diff --git a/openstack-tf/d4s-dev/geoserver_new/main.tf b/openstack-tf/d4s-dev/geoserver_new/main.tf index 65d6a74..e24cd14 100644 --- a/openstack-tf/d4s-dev/geoserver_new/main.tf +++ b/openstack-tf/d4s-dev/geoserver_new/main.tf @@ -33,7 +33,8 @@ module "geo_service" { flavor = "m1.xlarge", networks = ["d4s-dev-cloud-main", "postgresql-srv-net"], security_groups = ["default", "http and https from the load balancers"], - block_device_uuid = "7ed6a2cd-2b07-482e-8ce4-f018dff16c89" #ubuntu_18_04.uuid of DEV + server_groups_ids = [], + image_uuid = "7ed6a2cd-2b07-482e-8ce4-f018dff16c89" #ubuntu_18_04.uuid of DEV vol_data_name = "geoserver-geona_data_volume", vol_data_size = "50" }, @@ -43,7 +44,8 @@ module "geo_service" { flavor = "m2.small", networks = ["d4s-dev-cloud-main", "postgresql-srv-net"], security_groups = ["default", "http and https from the load balancers"], - block_device_uuid = "7ed6a2cd-2b07-482e-8ce4-f018dff16c89" #ubuntu_18_04.uuid of DEV + server_groups_ids = [], + image_uuid = "7ed6a2cd-2b07-482e-8ce4-f018dff16c89" #ubuntu_18_04.uuid of DEV vol_data_name = "geoserver-esquiline_data_volume", vol_data_size = "20" }, @@ -53,7 +55,8 @@ module "geo_service" { flavor = "m2.small", networks = ["d4s-dev-cloud-main", "postgresql-srv-net"], security_groups = ["default", "http and https from the load balancers"], - block_device_uuid = "7ed6a2cd-2b07-482e-8ce4-f018dff16c89" #ubuntu_18_04.uuid of DEV + server_groups_ids = [], + image_uuid = "7ed6a2cd-2b07-482e-8ce4-f018dff16c89" #ubuntu_18_04.uuid of DEV vol_data_name = "geoserver-ariadne_data_volume", vol_data_size = "20" }, @@ -63,7 +66,8 @@ module "geo_service" { flavor = "m1.large", networks = ["d4s-dev-cloud-main", "postgresql-srv-net"], security_groups = ["default", "http and https from the load balancers"], - block_device_uuid = "7ed6a2cd-2b07-482e-8ce4-f018dff16c89" #ubuntu_18_04.uuid of DEV + server_groups_ids = [], + image_uuid = "7ed6a2cd-2b07-482e-8ce4-f018dff16c89" #ubuntu_18_04.uuid of DEV vol_data_name = "geoserver-grsf_data_volume", vol_data_size = "40" }, @@ -73,7 +77,8 @@ module "geo_service" { flavor = "c1.large", networks = ["d4s-dev-cloud-main", "postgresql-srv-net"], security_groups = ["default", "http and https from the load balancers"], - block_device_uuid = "7ed6a2cd-2b07-482e-8ce4-f018dff16c89" #ubuntu_18_04.uuid of DEV + server_groups_ids = [], + image_uuid = "7ed6a2cd-2b07-482e-8ce4-f018dff16c89" #ubuntu_18_04.uuid of DEV vol_data_name = "geoserver-protectedareaimpactmaps_data_volume", vol_data_size = "70" } @@ -83,7 +88,8 @@ module "geo_service" { flavor = "m1.large", networks = ["d4s-dev-cloud-main", "postgresql-srv-net"], security_groups = ["default", "http and https from the load balancers"], - block_device_uuid = "7ed6a2cd-2b07-482e-8ce4-f018dff16c89" #ubuntu_18_04.uuid of DEV + server_groups_ids = [], + image_uuid = "7ed6a2cd-2b07-482e-8ce4-f018dff16c89" #ubuntu_18_04.uuid of DEV vol_data_name = "geoserver-sdi-lab_data_volume", vol_data_size = "100" } @@ -93,7 +99,8 @@ module "geo_service" { flavor = "m1.large", networks = ["d4s-dev-cloud-main", "postgresql-srv-net"], security_groups = ["default", "http and https from the load balancers"], - block_device_uuid = "7ed6a2cd-2b07-482e-8ce4-f018dff16c89" #ubuntu_18_04.uuid of DEV + server_groups_ids = [], + image_uuid = "7ed6a2cd-2b07-482e-8ce4-f018dff16c89" #ubuntu_18_04.uuid of DEV vol_data_name = "geoserver-tunaatlas_data_volume", vol_data_size = "60" } @@ -103,7 +110,8 @@ module "geo_service" { flavor = "m1.large", networks = ["d4s-dev-cloud-main", "postgresql-srv-net"], security_groups = ["default", "http and https from the load balancers"], - block_device_uuid = "7ed6a2cd-2b07-482e-8ce4-f018dff16c89" #ubuntu_18_04.uuid of DEV + server_groups_ids = [], + image_uuid = "7ed6a2cd-2b07-482e-8ce4-f018dff16c89" #ubuntu_18_04.uuid of DEV vol_data_name = "geoserver-wecafc-firms_data_volume", vol_data_size = "40" } @@ -113,7 +121,8 @@ module "geo_service" { flavor = "m1.large", networks = ["d4s-dev-cloud-main", "postgresql-srv-net"], security_groups = ["default", "http and https from the load balancers"], - block_device_uuid = "7ed6a2cd-2b07-482e-8ce4-f018dff16c89" #ubuntu_18_04.uuid of DEV + server_groups_ids = [], + image_uuid = "7ed6a2cd-2b07-482e-8ce4-f018dff16c89" #ubuntu_18_04.uuid of DEV vol_data_name = "geoserver-aquacultureatlas_data_volume", vol_data_size = "50" } diff --git a/openstack-tf/d4s-dev/uri-resolver/main.tf b/openstack-tf/d4s-dev/uri-resolver/main.tf index ac1cd99..414c97e 100644 --- a/openstack-tf/d4s-dev/uri-resolver/main.tf +++ b/openstack-tf/d4s-dev/uri-resolver/main.tf @@ -17,18 +17,36 @@ data "terraform_remote_state" "privnet_dns_router" { } } +# +# Uses common_variables as module +# +module "common_variables" { + source = "../../modules/common_variables" +} + + +# +# Creates the server group "uri-resolver" +# +resource "openstack_compute_servergroup_v2" "uri-resolver" { + name = "uri-resolver" + policies = [module.common_variables.policy_list.soft_anti_affinity] +} + module "smartgears_service_generic" { source = "../../modules/generic_smartgears_service" smartgears_service_instances_map = { + # URI-Resolver instance 1 uri_resolver_service_i1 = { name = "data", description = "The data instance", flavor = "m1.medium", networks = ["d4s-dev-cloud-main"], - security_groups = ["default", "http and https from the load balancers"] - block_device_uuid = "7ed6a2cd-2b07-482e-8ce4-f018dff16c89" #ubuntu_18_04.uuid of DEV + security_groups = ["default", "http and https from the load balancers"], + server_groups_ids = [openstack_compute_servergroup_v2.uri-resolver.id], + image_uuid = "7ed6a2cd-2b07-482e-8ce4-f018dff16c89" #ubuntu_18_04.uuid of DEV }, # URI-Resolver instance 2 uri_resolver_service_i2 = { @@ -36,8 +54,9 @@ module "smartgears_service_generic" { description = "The data1 instance", flavor = "m1.medium", networks = ["d4s-dev-cloud-main"], - security_groups = ["default", "http and https from the load balancers"] - block_device_uuid = "7ed6a2cd-2b07-482e-8ce4-f018dff16c89" #ubuntu_18_04.uuid of DEV + security_groups = ["default", "http and https from the load balancers"], + server_groups_ids = [openstack_compute_servergroup_v2.uri-resolver.id], + image_uuid = "7ed6a2cd-2b07-482e-8ce4-f018dff16c89" #ubuntu_18_04.uuid of DEV } } } diff --git a/openstack-tf/modules/common_variables/outputs.tf b/openstack-tf/modules/common_variables/outputs.tf index 3814d5a..7812033 100644 --- a/openstack-tf/modules/common_variables/outputs.tf +++ b/openstack-tf/modules/common_variables/outputs.tf @@ -4,20 +4,20 @@ output "main_region" { } output "external_network" { - value = var.external_network + value = var.external_network } output "external_network_id" { - value = var.external_network.id + value = var.external_network.id } output "floating_ip_pools" { - value = var.floating_ip_pools - + value = var.floating_ip_pools + } output "resolvers_ip" { - value = var.resolvers_ip + value = var.resolvers_ip } output "mtu_size" { @@ -29,15 +29,15 @@ output "availability_zones_names" { } output "availability_zone_no_gpu_name" { - value = var.availability_zones_names.availability_zone_no_gpu + value = var.availability_zones_names.availability_zone_no_gpu } output "availability_zone_with_gpu_name" { - value = var.availability_zones_names.availability_zone_with_gpu + value = var.availability_zones_names.availability_zone_with_gpu } output "ssh_sources" { - value = var.ssh_sources + value = var.ssh_sources } output "networks_with_d4s_services" { @@ -55,16 +55,16 @@ output "networks_with_d4s_services" { # } output "dns_zone" { - value = var.dns_zone + value = var.dns_zone } output "dns_zone_id" { - value = var.dns_zone_id + value = var.dns_zone_id } output "main_private_network" { value = { - name = var.main_private_network.name - description = var.main_private_network.description + name = var.main_private_network.name + description = var.main_private_network.description } } @@ -77,7 +77,7 @@ output "main_private_subnet" { } output "main_private_subnet_id" { - value = var.main_private_subnet_id + value = var.main_private_subnet_id } output "external_router" { @@ -101,15 +101,15 @@ output "almalinux_9" { } output "ubuntu1804_data_file" { - value = var.ubuntu1804_data_file + value = var.ubuntu1804_data_file } output "ubuntu2204_data_file" { - value = var.ubuntu2204_data_file + value = var.ubuntu2204_data_file } output "el7_data_file" { - value = var.el7_data_file + value = var.el7_data_file } output "ssh_jump_proxy" { @@ -160,3 +160,7 @@ output "octavia_information" { value = var.octavia_information } +output "policy_list" { + value = var.policy_list +} + diff --git a/openstack-tf/modules/common_variables/variables.tf b/openstack-tf/modules/common_variables/variables.tf index 89ea671..eadca81 100644 --- a/openstack-tf/modules/common_variables/variables.tf +++ b/openstack-tf/modules/common_variables/variables.tf @@ -354,14 +354,25 @@ variable "security_group_list" { } } +# Added by Francesco +variable "policy_list" { + type = map(string) + default = { + soft_anti_affinity = "soft-anti-affinity" + anti_affinity = "anti-affinity" + affinity = "affinity" + soft_affinity = "soft-affinity" + } +} + variable "networks_list" { type = map(string) default = { shared_postgresql = "postgresql-srv-net" - swarm = "swarm-nfs-net" - timescaledb = "timescaledb-net" - orientdb = "orientdb-net" - orientdb_se = "orientdb-se-net" + swarm = "swarm-nfs-net" + timescaledb = "timescaledb-net" + orientdb = "orientdb-net" + orientdb_se = "orientdb-se-net" } - -} \ No newline at end of file + +} diff --git a/openstack-tf/modules/generic_smartgears_service/generic_smartgears_service.tf b/openstack-tf/modules/generic_smartgears_service/generic_smartgears_service.tf index c04c39d..6f486aa 100644 --- a/openstack-tf/modules/generic_smartgears_service/generic_smartgears_service.tf +++ b/openstack-tf/modules/generic_smartgears_service/generic_smartgears_service.tf @@ -1,13 +1,15 @@ -# Generic smartgears_service instance +# Module used module "ssh_settings" { source = "../../modules/ssh-key-ref" } +# Module used module "common_variables" { source = "../../modules/common_variables" } +# Generic smartgears_service instance resource "openstack_compute_instance_v2" "smartgears_service" { for_each = var.smartgears_service_instances_map name = each.value.name @@ -16,7 +18,7 @@ resource "openstack_compute_instance_v2" "smartgears_service" { key_pair = module.ssh_settings.ssh_key_name security_groups = each.value.security_groups block_device { - uuid = each.value.block_device_uuid + uuid = each.value.image_uuid source_type = "image" volume_size = 10 boot_index = 0 @@ -24,6 +26,7 @@ resource "openstack_compute_instance_v2" "smartgears_service" { delete_on_termination = false } + # Creates the networks according to input networks dynamic "network" { for_each = each.value.networks content { @@ -31,5 +34,14 @@ resource "openstack_compute_instance_v2" "smartgears_service" { } } - user_data = file("${module.common_variables.ubuntu1804_data_file}") + # Creates the scheduler_hints (i.e. server groups) according to input server_groups_ids + dynamic "scheduler_hints" { + for_each = each.value.server_groups_ids + content { + group = scheduler_hints.value + } + } + + #user_data script used + user_data = file("${var.user_data_var_file.user_data_file}") #default is 18.04 } diff --git a/openstack-tf/modules/generic_smartgears_service/outputs.tf b/openstack-tf/modules/generic_smartgears_service/outputs.tf index 5c320ee..0d2d444 100644 --- a/openstack-tf/modules/generic_smartgears_service/outputs.tf +++ b/openstack-tf/modules/generic_smartgears_service/outputs.tf @@ -2,3 +2,7 @@ output "smartgears_service_instances_map" { value = var.smartgears_service_instances_map } + +output "user_data_var_file" { + value = var.user_data_var_file +} diff --git a/openstack-tf/modules/generic_smartgears_service/variables_generic_smartgears_service.tf b/openstack-tf/modules/generic_smartgears_service/variables_generic_smartgears_service.tf index dd761c8..8a89233 100644 --- a/openstack-tf/modules/generic_smartgears_service/variables_generic_smartgears_service.tf +++ b/openstack-tf/modules/generic_smartgears_service/variables_generic_smartgears_service.tf @@ -1,4 +1,11 @@ +variable "user_data_var_file" { + type = map(string) + default = { + user_data_file = "../../openstack_vm_data_scripts/ubuntu1804.sh" #default is 18.04 + } +} + #Default smartgears_service is EMPTY. Override it to create a proper smartegears plan variable "smartgears_service_instances_map" { type = map(object({ @@ -7,10 +14,11 @@ variable "smartgears_service_instances_map" { flavor = string networks = list(string) security_groups = list(string) - block_device_uuid = string + server_groups_ids = list(string) + image_uuid = string })) default = { - smartgears_service = { name = "", description = "", flavor = "", networks = [], security_groups = [], block_device_uuid = "" } + smartgears_service = { name = "", description = "", flavor = "", networks = [], security_groups = [], server_groups_ids = [], image_uuid = "" } } } diff --git a/openstack-tf/modules/geo_services/geo_service.tf b/openstack-tf/modules/geo_services/geo_service.tf index 63e22d6..0c12dd1 100644 --- a/openstack-tf/modules/geo_services/geo_service.tf +++ b/openstack-tf/modules/geo_services/geo_service.tf @@ -22,19 +22,29 @@ resource "openstack_compute_instance_v2" "geo_service" { key_pair = module.ssh_settings.ssh_key_name security_groups = each.value.security_groups block_device { - uuid = each.value.block_device_uuid + uuid = each.value.image_uuid source_type = "image" volume_size = 10 boot_index = 0 destination_type = "volume" delete_on_termination = false } + + # Creates the networks according to input networks dynamic "network" { for_each = each.value.networks content { name = network.value } } + + # Creates the scheduler_hints (i.e. server groups) according to input server_groups_ids + dynamic "scheduler_hints" { + for_each = each.value.server_groups_ids + content { + group = scheduler_hints.value + } + } user_data = file("${var.user_data_var_file.user_data_file}") #default is 18.04 } diff --git a/openstack-tf/modules/geo_services/inputs.tf b/openstack-tf/modules/geo_services/inputs.tf index df29485..ddf1736 100644 --- a/openstack-tf/modules/geo_services/inputs.tf +++ b/openstack-tf/modules/geo_services/inputs.tf @@ -10,7 +10,7 @@ variable "geo_instance_basic_data" { variable "user_data_var_file" { type = map(string) default = { - user_data_file = "../../openstack_vm_data_scripts/ubuntu1804.sh" + user_data_file = "../../openstack_vm_data_scripts/ubuntu1804.sh" #default is 18.04 } } @@ -22,12 +22,13 @@ variable "geo_service_instances_map" { flavor = string networks = list(string) security_groups = list(string) - block_device_uuid = string + server_groups_ids = list(string) + image_uuid = string vol_data_name = string vol_data_size = string })) default = { - geoserver = { name = "", description = "", flavor = "", networks = [], security_groups = [], block_device_uuid = "", vol_data_name = "", vol_data_size = "" } + geoserver = { name = "", description = "", flavor = "", networks = [], security_groups = [], server_groups_ids = [], image_uuid = "", vol_data_name = "", vol_data_size = "" } } }