Skip to content

Commit

Permalink
fix: remove redundant security groups
Browse files Browse the repository at this point in the history
  • Loading branch information
Isobel Daley committed Apr 24, 2024
1 parent bf7a784 commit 0734940
Showing 1 changed file with 0 additions and 24 deletions.
24 changes: 0 additions & 24 deletions infra/security_groups.tf
Original file line number Diff line number Diff line change
Expand Up @@ -436,18 +436,6 @@ resource "aws_security_group_rule" "notebooks_ingress_https_from_admin" {
protocol = "tcp"
}

resource "aws_security_group_rule" "notebooks_ingress_https_from_arango" {
description = "ingress-https-from-arango"

security_group_id = aws_security_group.notebooks.id
source_security_group_id = aws_security_group.arango_service.id

type = "ingress"
from_port = local.arango_container_port
to_port = local.arango_container_port
protocol = "tcp"
}

resource "aws_security_group_rule" "notebooks_ingress_https_from_arango_lb" {
description = "ingress-https-from-arango-lb"

Expand Down Expand Up @@ -2026,18 +2014,6 @@ resource "aws_security_group_rule" "arango_lb_ingress_https_from_whitelist" {
protocol = "tcp"
}

resource "aws_security_group_rule" "arangoo_service_egress_http_to_arango_lb" {
description = "egress-http-to-arango-lb"

security_group_id = aws_security_group.arango_service.id
source_security_group_id = aws_security_group.arango_lb.id

type = "egress"
from_port = "80"
to_port = "80"
protocol = "tcp"
}

resource "aws_security_group_rule" "arango_lb_egress_https_to_arango_service" {
description = "egress-https-to-arango-service"

Expand Down

0 comments on commit 0734940

Please sign in to comment.