Skip to content

Commit

Permalink
Use current TF+provider versions in WAF module.
Browse files Browse the repository at this point in the history
Also remove a reference to another deleted load balancer, which was
breaking the ability to apply the module in prod.
  • Loading branch information
sengi committed Dec 14, 2023
1 parent cfe31b0 commit d1ff2f5
Show file tree
Hide file tree
Showing 4 changed files with 6 additions and 13 deletions.
2 changes: 1 addition & 1 deletion terraform/projects/infra-public-wafs/.terraform-version
Original file line number Diff line number Diff line change
@@ -1 +1 @@
1.1.7
1.6.6
7 changes: 3 additions & 4 deletions terraform/projects/infra-public-wafs/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -2,14 +2,14 @@

| Name | Version |
|------|---------|
| <a name="requirement_terraform"></a> [terraform](#requirement\_terraform) | = 1.1.7 |
| <a name="requirement_aws"></a> [aws](#requirement\_aws) | ~> 3.0 |
| <a name="requirement_terraform"></a> [terraform](#requirement\_terraform) | ~> 1.6 |
| <a name="requirement_aws"></a> [aws](#requirement\_aws) | ~> 5.0 |

## Providers

| Name | Version |
|------|---------|
| <a name="provider_aws"></a> [aws](#provider\_aws) | ~> 3.0 |
| <a name="provider_aws"></a> [aws](#provider\_aws) | ~> 5.0 |
| <a name="provider_terraform"></a> [terraform](#provider\_terraform) | n/a |

## Modules
Expand All @@ -30,7 +30,6 @@ No modules.
| [aws_shield_protection.licensify_backend_public_lb](https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/shield_protection) | resource |
| [aws_shield_protection.licensify_frontend_public_lb](https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/shield_protection) | resource |
| [aws_shield_protection.monitoring_public_lb](https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/shield_protection) | resource |
| [aws_shield_protection.prometheus_public_lb](https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/shield_protection) | resource |
| [aws_wafv2_ip_set.govuk_requesting_ips](https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/wafv2_ip_set) | resource |
| [aws_wafv2_ip_set.high_request_rate](https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/wafv2_ip_set) | resource |
| [aws_wafv2_ip_set.nat_gateway_ips](https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/wafv2_ip_set) | resource |
Expand Down
5 changes: 2 additions & 3 deletions terraform/projects/infra-public-wafs/main.tf
Original file line number Diff line number Diff line change
@@ -1,11 +1,10 @@
terraform {
backend "s3" {}
required_version = "= 1.1.7"

required_version = "~> 1.6"
required_providers {
aws = {
source = "hashicorp/aws"
version = "~> 3.0"
version = "~> 5.0"
}
}
}
Expand Down
5 changes: 0 additions & 5 deletions terraform/projects/infra-public-wafs/standard_config.tf
Original file line number Diff line number Diff line change
Expand Up @@ -49,8 +49,3 @@ resource "aws_wafv2_web_acl_association" "monitoring_public_web_acl" {
resource_arn = data.terraform_remote_state.infra_public_services.outputs.monitoring_public_lb_id
web_acl_arn = aws_wafv2_web_acl.default.arn
}

resource "aws_shield_protection" "prometheus_public_lb" {
name = "${var.stackname}-prometheus-public-lb_shield"
resource_arn = data.terraform_remote_state.infra_public_services.outputs.prometheus_public_lb_id
}

0 comments on commit d1ff2f5

Please sign in to comment.