Merge pull request #61 from MaterializeInc/nlb_details_include_instance_name

Include Materialize instance name in NLB details

Author: alex-hunt-materialize

main.tf

@@ -170,6 +170,7 @@ module "nlb" {
 
   for_each = { for idx, instance in local.instances : instance.name => instance if lookup(instance, "create_nlb", true) }
 
+  instance_name                    = each.value.name
   name_prefix                      = "${local.name_prefix}-${each.value.name}"
   namespace                        = each.value.namespace
   internal                         = each.value.internal_nlb

modules/nlb/outputs.tf

@@ -1,3 +1,8 @@
+output "instance_name" {
+  description = "The name of the Materialize instance."
+  value       = var.instance_name
+}
+
 output "nlb_dns_name" {
   description = "DNS name of the Network Load Balancer."
   value       = aws_lb.nlb.dns_name

modules/nlb/variables.tf

@@ -1,3 +1,8 @@
+variable "instance_name" {
+  description = "The name of the Materialize instance."
+  type        = string
+}
+
 variable "name_prefix" {
   description = "Prefix to use for NLB, Target Groups, Listeners, and TargetGroupBindings"
   type        = string

outputs.tf

@@ -89,10 +89,10 @@ output "operator_details" {
 
 output "nlb_details" {
   description = "Details of the Materialize instance NLBs."
-  value = [
-    for nlb in module.nlb : {
+  value = {
+    for nlb in module.nlb : nlb.instance_name => {
       arn      = nlb.nlb_arn
       dns_name = nlb.nlb_dns_name
     }
-  ]
+  }
 }