OPS-5596: quick fix

Flaconi · Sep 25, 2023 · aeb6234 · aeb6234
1 parent e87e4ce
commit aeb6234
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/main.tf b/main.tf
@@ -5,7 +5,7 @@ module "vault_cluster" {
   cluster_size  = var.vault_cluster_size
   instance_type = var.vault_instance_type
 
-  ami_id = var.ami_id == null ? data.aws_ami.vault_consul.image_id : var.ami_id
+  ami_id = var.ami_id == null ? data.aws_ami.vault_consul.id : var.ami_id
   user_data = templatefile("${path.module}/user-data/vault.sh.tftpl", {
     enable_s3_backend        = var.enable_s3_backend ? 1 : 0
     s3_bucket_region         = data.aws_region.current.name
@@ -81,7 +81,7 @@ module "consul_cluster" {
   cluster_size  = var.consul_cluster_size
   instance_type = var.consul_instance_type
 
-  ami_id = var.ami_id == null ? data.aws_ami.vault_consul.image_id : var.ami_id
+  ami_id = var.ami_id == null ? data.aws_ami.vault_consul.id : var.ami_id
   user_data = templatefile("${path.module}/user-data/consul.sh.tftpl", {
     consul_cluster_tag_key   = local.consul_cluster_tag_key
     consul_cluster_tag_value = local.consul_cluster_tag_val

diff --git a/variables.tf b/variables.tf
@@ -120,8 +120,8 @@ variable "kms_alias_name" {
 
 variable "ami_id" {
   description = "ID of the AMI to be used for the Consul and Vault instances."
-  type        = string
   default     = null
+  type        = string
 }
 
 variable "ami_owner" {