Allow specific reservation for node-group in slurm-gcp v5

community/modules/compute/schedmd-slurm-gcp-v5-node-group/README.md

@@ -96,13 +96,13 @@ limitations under the License.
 | Name | Version |
 |------|---------|
 | <a name="requirement_terraform"></a> [terraform](#requirement\_terraform) | >= 1.1 |
-| <a name="requirement_google"></a> [google](#requirement\_google) | >= 3.83 |
+| <a name="requirement_google"></a> [google](#requirement\_google) | >= 5.11 |
 
 ## Providers
 
 | Name | Version |
 |------|---------|
-| <a name="provider_google"></a> [google](#provider\_google) | >= 3.83 |
+| <a name="provider_google"></a> [google](#provider\_google) | >= 5.11 |
 
 ## Modules
 

community/modules/compute/schedmd-slurm-gcp-v5-node-group/versions.tf

@@ -18,7 +18,7 @@ terraform {
   required_providers {
     google = {
       source  = "hashicorp/google"
-      version = ">= 3.83"
+      version = ">= 5.11"
     }
   }
   provider_meta "google" {

community/modules/compute/schedmd-slurm-gcp-v5-partition/README.md

@@ -134,13 +134,13 @@ limitations under the License.
 | Name | Version |
 |------|---------|
 | <a name="requirement_terraform"></a> [terraform](#requirement\_terraform) | >= 0.13.0 |
-| <a name="requirement_google"></a> [google](#requirement\_google) | >= 3.83 |
+| <a name="requirement_google"></a> [google](#requirement\_google) | >= 5.11 |
 
 ## Providers
 
 | Name | Version |
 |------|---------|
-| <a name="provider_google"></a> [google](#provider\_google) | >= 3.83 |
+| <a name="provider_google"></a> [google](#provider\_google) | >= 5.11 |
 
 ## Modules
 
@@ -152,6 +152,7 @@ limitations under the License.
 
 | Name | Type |
 |------|------|
+| [google_compute_reservation.reservation](https://registry.terraform.io/providers/hashicorp/google/latest/docs/data-sources/compute_reservation) | data source |
 | [google_compute_zones.available](https://registry.terraform.io/providers/hashicorp/google/latest/docs/data-sources/compute_zones) | data source |
 
 ## Inputs

community/modules/compute/schedmd-slurm-gcp-v5-partition/main.tf

@@ -30,6 +30,8 @@ locals {
 
   all_zones      = toset(concat([var.zone], tolist(var.zones)))
   excluded_zones = [for z in data.google_compute_zones.available.names : z if !contains(local.all_zones, z)]
+
+  reservation_map = { for x in var.node_groups : x.reservation_name => x }
 }
 
 data "google_compute_zones" "available" {
@@ -58,3 +60,29 @@ module "slurm_partition" {
   partition_startup_scripts         = local.ghpc_startup_script
   partition_startup_scripts_timeout = var.partition_startup_scripts_timeout
 }
+
+# tflint-ignore: terraform_unused_declarations
+data "google_compute_reservation" "reservation" {
+  project = var.project_id
+  zone    = var.zone
+
+  for_each = local.reservation_map
+  name     = each.value.reservation_name
+
+  lifecycle {
+    postcondition {
+      condition     = self.self_link != null
+      error_message = "couldn't find the reservation ${each.value.reservation_name}}"
+    }
+
+    postcondition {
+      condition     = coalesce(self.specific_reservation_required, true)
+      error_message = <<EOT
+      your reservation has to be specific,
+      see https://cloud.google.com/compute/docs/instances/reservations-overview#how-reservations-work
+      for more information. if it's intentionally automatic, don't specify
+      it in the blueprint.
+      EOT
+    }
+  }
+}

community/modules/compute/schedmd-slurm-gcp-v5-partition/versions.tf

@@ -18,7 +18,7 @@ terraform {
   required_providers {
     google = {
       source  = "hashicorp/google"
-      version = ">= 3.83"
+      version = ">= 5.11"
     }
   }
   provider_meta "google" {