Skip to content

Commit 2c080ce

Browse files
authored
Merge pull request #416 from ComputeCanada/allow_all
Replace allowed protocol list by all for os and gcp
2 parents 936f9d3 + 92bbb95 commit 2c080ce

File tree

2 files changed

+3
-31
lines changed

2 files changed

+3
-31
lines changed

gcp/network.tf

Lines changed: 1 addition & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -33,17 +33,8 @@ resource "google_compute_firewall" "allow_all_internal" {
3333
source_ranges = [google_compute_subnetwork.subnet.ip_cidr_range]
3434

3535
allow {
36-
protocol = "tcp"
36+
protocol = "all"
3737
}
38-
39-
allow {
40-
protocol = "udp"
41-
}
42-
43-
allow {
44-
protocol = "icmp"
45-
}
46-
4738
}
4839

4940
locals {

openstack/network-2.tf

Lines changed: 2 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -3,29 +3,10 @@ resource "openstack_networking_secgroup_v2" "global" {
33
description = "${var.cluster_name} global security group"
44
}
55

6-
resource "openstack_networking_secgroup_rule_v2" "icmp" {
6+
resource "openstack_networking_secgroup_rule_v2" "all" {
77
direction = "ingress"
88
ethertype = "IPv4"
9-
protocol = "icmp"
10-
description = "internal allow all icmp"
11-
security_group_id = openstack_networking_secgroup_v2.global.id
12-
remote_group_id = openstack_networking_secgroup_v2.global.id
13-
}
14-
15-
resource "openstack_networking_secgroup_rule_v2" "tcp" {
16-
direction = "ingress"
17-
ethertype = "IPv4"
18-
protocol = "tcp"
19-
description = "internal allow all tcp"
20-
security_group_id = openstack_networking_secgroup_v2.global.id
21-
remote_group_id = openstack_networking_secgroup_v2.global.id
22-
}
23-
24-
resource "openstack_networking_secgroup_rule_v2" "udp" {
25-
direction = "ingress"
26-
ethertype = "IPv4"
27-
protocol = "udp"
28-
description = "internal allow all udp"
9+
description = "internal allow all"
2910
security_group_id = openstack_networking_secgroup_v2.global.id
3011
remote_group_id = openstack_networking_secgroup_v2.global.id
3112
}

0 commit comments

Comments
 (0)