New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Feat: add security groups options #53
Feat: add security groups options #53
Conversation
- Change the default value of the "security_group_enabled" variable from true to false Signed-off-by: Quentin BERTRAND <contact@quentinbertrand.fr>
- Modify the `resource "aws_security_group_rule" "default_ingress"` block in `main.tf` to use `var.default_listener_ingress_cidr_blocks` and `var.default_listener_ingress_prefix_list_ids` - Add a new variable `variable "security_group_ids"` in `variables.tf` - Add a new variable `variable "default_listener_ingress_cidr_blocks"` in `variables.tf` - Add a new variable `variable "default_listener_ingress_prefix_list_ids"` in `variables.tf` Signed-off-by: Quentin BERTRAND <contact@quentinbertrand.fr>
- Add an output for the security group ID of the NLB Signed-off-by: Quentin BERTRAND <contact@quentinbertrand.fr>
- Add `security_group_enabled = true` to `fixtures.us-east-2.tfvars` - Add `security_group_enabled` variable to the `nlb` module in `main.tf` Signed-off-by: Quentin BERTRAND <contact@quentinbertrand.fr>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@QuentinBtd thanks for the PR
Please address:
README.md is outdated. Please run the following commands locally and push the files:
make init
make github/init
make readme
- Add `aws_security_group.default` resource - Add `aws_security_group_rule.default_ingress` resource - Add `aws_security_group_rule.egress` resource - Add `aws_security_group_rule.tls_ingress` resource - Add `input_default_listener_ingress_cidr_blocks` variable - Add `input_default_listener_ingress_prefix_list_ids` variable - Add `input_security_group_enabled` variable - Add `input_security_group_ids` variable - Add `input_tls_ingress_cidr_blocks` variable - Add `input_tls_ingress_prefix_list_ids` variable - Add `output_security_group_id` output Signed-off-by: Quentin BERTRAND <contact@quentinbertrand.fr>
@QuentinBtd sorry, please do it again, see https://github.com/cloudposse/terraform-aws-nlb/actions/runs/7699308430/job/20982416956?pr=53 |
I hadn't paid attention the first time, but;
|
Hello @aknysh |
/terratest |
- Add a complete license in the README file Signed-off-by: Quentin BERTRAND <contact@quentinbertrand.fr>
|
/terratest |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/terratest
/terratest |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks @QuentinBtd
what
Add security groups options
why
We can associate NLB with security groups (only at NLB creation)
references
https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/lb#security_groups