Skip to content

Commit

Permalink
πŸ› fix(outputs): update outputs to incluude the dns_name and zone_id
Browse files Browse the repository at this point in the history
  • Loading branch information
morga471 committed Oct 17, 2024
1 parent a2ad2ff commit 6aff50a
Show file tree
Hide file tree
Showing 4 changed files with 15 additions and 4 deletions.
6 changes: 5 additions & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -79,13 +79,15 @@ have a istio proxy configured, prevent communication with that pod.)
| Name | Version |
|------|---------|
| <a name="requirement_terraform"></a> [terraform](#requirement\_terraform) | >= 0.13 |
| <a name="requirement_aws"></a> [aws](#requirement\_aws) | >= 5.14.0 |
| <a name="requirement_helm"></a> [helm](#requirement\_helm) | >= 2.11.0 |
| <a name="requirement_kubernetes"></a> [kubernetes](#requirement\_kubernetes) | >= 2.23.0 |

## Providers

| Name | Version |
|------|---------|
| <a name="provider_aws"></a> [aws](#provider\_aws) | 5.61.0 |
| <a name="provider_helm"></a> [helm](#provider\_helm) | 2.14.0 |
| <a name="provider_kubernetes"></a> [kubernetes](#provider\_kubernetes) | 2.31.0 |

Expand All @@ -104,6 +106,8 @@ have a istio proxy configured, prevent communication with that pod.)
| [helm_release.ingress](https://registry.terraform.io/providers/hashicorp/helm/latest/docs/resources/release) | resource |
| [helm_release.istiod](https://registry.terraform.io/providers/hashicorp/helm/latest/docs/resources/release) | resource |
| [kubernetes_namespace.ns](https://registry.terraform.io/providers/hashicorp/kubernetes/latest/docs/resources/namespace) | resource |
| [aws_lb.lb](https://registry.terraform.io/providers/hashicorp/aws/latest/docs/data-sources/lb) | data source |
| [kubernetes_service.istio_ingressgateway](https://registry.terraform.io/providers/hashicorp/kubernetes/latest/docs/data-sources/service) | data source |

## Inputs

Expand All @@ -125,7 +129,7 @@ have a istio proxy configured, prevent communication with that pod.)

| Name | Description |
|------|-------------|
| <a name="output_istio_namespace"></a> [istio\_namespace](#output\_istio\_namespace) | The namespace used by istio. |
| <a name="output_istio_ingress_lb"></a> [istio\_ingress\_lb](#output\_istio\_ingress\_lb) | The istio ingress load balancer dns\_name and zone\_id. |
| <a name="output_module_name"></a> [module\_name](#output\_module\_name) | The name of this module. |
| <a name="output_module_version"></a> [module\_version](#output\_module\_version) | The version of this module. |
<!-- END_TF_DOCS -->
2 changes: 1 addition & 1 deletion aws_data.tf
Original file line number Diff line number Diff line change
Expand Up @@ -9,4 +9,4 @@ data "kubernetes_service" "istio_ingressgateway" {
data "aws_lb" "lb" {
depends_on = [helm_release.ingress]
name = split("-", data.kubernetes_service.istio_ingressgateway.status[0].load_balancer[0].ingress[0].hostname)[0]
}
}
7 changes: 5 additions & 2 deletions outputs.tf
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,9 @@ output "module_version" {
}

output "istio_ingress_lb" {
description = "The istio ingress load balancer DNS."
value = data.aws_lb.lb.dns_name
description = "The istio ingress load balancer dns_name and zone_id."
value = {
dns_name = data.aws_lb.lb.dns_name
zone_id = data.aws_lb.lb.zone_id
}
}
4 changes: 4 additions & 0 deletions versions.tf β†’ requirements.tf
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,10 @@ terraform {
required_version = ">= 0.13"

required_providers {
aws = {
source = "hashicorp/aws"
version = ">= 5.14.0"
}
helm = {
source = "hashicorp/helm"
version = ">= 2.11.0"
Expand Down

0 comments on commit 6aff50a

Please sign in to comment.