Skip to content

Commit

Permalink
Remove AWS security rule allowing ICMP packets to nodes
Browse files Browse the repository at this point in the history
* Deny ICMP packets for consistency across Typhoon clusters on
various clouds and because there isn't much need to allow them
  • Loading branch information
dghubble committed Aug 22, 2018
1 parent da5d2c5 commit bbf2c13
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 40 deletions.
4 changes: 4 additions & 0 deletions CHANGES.md
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,10 @@ Notable changes between versions.

## Latest

#### AWS

* Remove firewall rule allowing ICMP packets to nodes

#### Bare-Metal

* Remove `controller_networkds` and `worker_networkds` variables. Use Container Linux Config snippets [#277](https://github.com/poseidon/typhoon/pull/277)
Expand Down
20 changes: 0 additions & 20 deletions aws/container-linux/kubernetes/security.tf
Original file line number Diff line number Diff line change
Expand Up @@ -11,16 +11,6 @@ resource "aws_security_group" "controller" {
tags = "${map("Name", "${var.cluster_name}-controller")}"
}

resource "aws_security_group_rule" "controller-icmp" {
security_group_id = "${aws_security_group.controller.id}"

type = "ingress"
protocol = "icmp"
from_port = 0
to_port = 0
cidr_blocks = ["0.0.0.0/0"]
}

resource "aws_security_group_rule" "controller-ssh" {
security_group_id = "${aws_security_group.controller.id}"

Expand Down Expand Up @@ -217,16 +207,6 @@ resource "aws_security_group" "worker" {
tags = "${map("Name", "${var.cluster_name}-worker")}"
}

resource "aws_security_group_rule" "worker-icmp" {
security_group_id = "${aws_security_group.worker.id}"

type = "ingress"
protocol = "icmp"
from_port = 0
to_port = 0
cidr_blocks = ["0.0.0.0/0"]
}

resource "aws_security_group_rule" "worker-ssh" {
security_group_id = "${aws_security_group.worker.id}"

Expand Down
20 changes: 0 additions & 20 deletions aws/fedora-atomic/kubernetes/security.tf
Original file line number Diff line number Diff line change
Expand Up @@ -11,16 +11,6 @@ resource "aws_security_group" "controller" {
tags = "${map("Name", "${var.cluster_name}-controller")}"
}

resource "aws_security_group_rule" "controller-icmp" {
security_group_id = "${aws_security_group.controller.id}"

type = "ingress"
protocol = "icmp"
from_port = 0
to_port = 0
cidr_blocks = ["0.0.0.0/0"]
}

resource "aws_security_group_rule" "controller-ssh" {
security_group_id = "${aws_security_group.controller.id}"

Expand Down Expand Up @@ -217,16 +207,6 @@ resource "aws_security_group" "worker" {
tags = "${map("Name", "${var.cluster_name}-worker")}"
}

resource "aws_security_group_rule" "worker-icmp" {
security_group_id = "${aws_security_group.worker.id}"

type = "ingress"
protocol = "icmp"
from_port = 0
to_port = 0
cidr_blocks = ["0.0.0.0/0"]
}

resource "aws_security_group_rule" "worker-ssh" {
security_group_id = "${aws_security_group.worker.id}"

Expand Down

0 comments on commit bbf2c13

Please sign in to comment.