Skip to content

Commit

Permalink
Some tfsec fixes
Browse files Browse the repository at this point in the history
  • Loading branch information
manics committed Jun 30, 2023
1 parent dd98ead commit b3e6818
Show file tree
Hide file tree
Showing 3 changed files with 17 additions and 13 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/terraform.yml
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ jobs:
- name: Install tflint
uses: terraform-linters/[email protected]
with:
tflint_version: v0.46.1
tflint_version: v0.47.0

- name: Run terraform pre-commit
run: pre-commit run --all --config .pre-commit-config-terraform.yaml
2 changes: 1 addition & 1 deletion .pre-commit-config-terraform.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ files: "^terraform/aws/curvenote/"
repos:
# We can't run any CI tests on production Terraform code, so use as many static linters as possible
- repo: https://github.com/antonbabenko/pre-commit-terraform
rev: v1.80.0
rev: v1.81.0
hooks:
- id: terraform_fmt
- id: terraform_tflint
Expand Down
26 changes: 15 additions & 11 deletions terraform/aws/curvenote/security-groups.tf
Original file line number Diff line number Diff line change
Expand Up @@ -3,18 +3,20 @@
resource "aws_security_group" "worker_group_all" {
name_prefix = "worker_group_all_ports"
vpc_id = module.vpc.vpc_id

description = "Allow all ports for worker group"

ingress {
protocol = "-1"
from_port = 0
to_port = 0
self = true
description = "Allow all inbound traffic"
protocol = "-1"
from_port = 0
to_port = 0
self = true
}
egress {
protocol = "-1"
from_port = 0
to_port = 0
description = "Allow all outbound traffic"
protocol = "-1"
from_port = 0
to_port = 0
# self = true
cidr_blocks = ["0.0.0.0/0"]
}
Expand All @@ -24,11 +26,13 @@ resource "aws_security_group" "worker_group_all" {
resource "aws_security_group" "all_worker_mgmt" {
name_prefix = "all_worker_management"
vpc_id = module.vpc.vpc_id
description = "Worker nodes internal access"

ingress {
from_port = 22
to_port = 22
protocol = "tcp"
description = "SSH"
from_port = 22
to_port = 22
protocol = "tcp"

cidr_blocks = [
"10.0.0.0/8",
Expand Down

0 comments on commit b3e6818

Please sign in to comment.