Skip to content

Commit

Permalink
Merge pull request #14 from babbel/ingress-security-groups-default
Browse files Browse the repository at this point in the history
Ingress security groups default
  • Loading branch information
jtsaito authored Nov 21, 2022
2 parents 1dd8c23 + 963d03e commit 3834f7c
Show file tree
Hide file tree
Showing 3 changed files with 9 additions and 3 deletions.
4 changes: 4 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,5 +1,9 @@
# Changelog

## v1.1.1

- [Add default value for `ingress_security_groups` attribute](https://github.com/babbel/terraform-aws-alb-for-vpc-internal-requests/pull/14)

## v1.1.0

- [Bump hashicorp/aws dependency](https://github.com/babbel/terraform-aws-alb-for-vpc-internal-requests/pull/10)
Expand Down
6 changes: 3 additions & 3 deletions _test/main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ provider "aws" {
}

module "alb" {
source = "./.."
source = "./.."

name = "example"

Expand All @@ -21,7 +21,7 @@ module "alb" {
target_group = { arn = "arn:aws:elasticloadbalancing:local:123456789012:targetgroup/some-service/0123456789abcdef" }

tags = {
app = "some-service"
env = "production"
app = "some-service"
env = "production"
}
}
2 changes: 2 additions & 0 deletions variables.tf
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,8 @@ variable "ingress_security_groups" {
id = string
})
)

default = {}
}

variable "name" {
Expand Down

0 comments on commit 3834f7c

Please sign in to comment.