Skip to content

Commit 47d7a85

Browse files
committed
support for multiple extra seclists
Signed-off-by: junior <junior@users.noreply.github.com>
1 parent a8c89e5 commit 47d7a85

File tree

4 files changed

+21
-11
lines changed

4 files changed

+21
-11
lines changed

defaults.tf

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -103,7 +103,7 @@ locals {
103103
# alternative_route_table = null # Optional, Name of the previously created route table
104104
# dhcp_options_id = module.vcn.default_dhcp_options_id # If null, the VCN's default set of DHCP options is used
105105
# security_list_ids = [module.security_lists["opensearch_security_list"].security_list_id] # If null, the VCN's default security list is used
106-
# alternative_security_list = null # Optional, Name of the previously created security list
106+
# extra_security_list_names = [] # Optional, Names of the previously created security lists
107107
# ipv6cidr_block = null # If null, no IPv6 CIDR block is assigned
108108
# },
109109
]

main.tf

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -208,7 +208,7 @@ locals {
208208
alternative_route_table = null
209209
dhcp_options_id = module.vcn.default_dhcp_options_id
210210
security_list_ids = [module.security_lists["oke_endpoint_security_list"].security_list_id]
211-
alternative_security_list = null
211+
extra_security_list_names = anytrue([(var.extra_security_list_name_for_api_endpoint == ""), (var.extra_security_list_name_for_api_endpoint == null)]) ? [] : [var.extra_security_list_name_for_api_endpoint]
212212
ipv6cidr_block = null
213213
},
214214
{
@@ -222,7 +222,7 @@ locals {
222222
alternative_route_table = null
223223
dhcp_options_id = module.vcn.default_dhcp_options_id
224224
security_list_ids = [module.security_lists["oke_nodes_security_list"].security_list_id]
225-
alternative_security_list = null
225+
extra_security_list_names = anytrue([(var.extra_security_list_name_for_nodes == ""), (var.extra_security_list_name_for_nodes == null)]) ? [] : [var.extra_security_list_name_for_nodes]
226226
ipv6cidr_block = null
227227
},
228228
{
@@ -236,7 +236,7 @@ locals {
236236
alternative_route_table = null
237237
dhcp_options_id = module.vcn.default_dhcp_options_id
238238
security_list_ids = [module.security_lists["oke_lb_security_list"].security_list_id]
239-
alternative_security_list = null
239+
extra_security_list_names = []
240240
ipv6cidr_block = null
241241
}
242242
]
@@ -252,7 +252,7 @@ locals {
252252
alternative_route_table = null
253253
dhcp_options_id = module.vcn.default_dhcp_options_id
254254
security_list_ids = [module.security_lists["oke_pod_network_security_list"].security_list_id]
255-
alternative_security_list = null
255+
extra_security_list_names = []
256256
ipv6cidr_block = null
257257
}] : []
258258
subnet_bastion = []

oci-networking.tf

Lines changed: 8 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -66,12 +66,14 @@ module "subnets" {
6666
prohibit_internet_ingress = each.value.prohibit_internet_ingress
6767
route_table_id = (anytrue([(each.value.alternative_route_table == ""), (each.value.alternative_route_table == null)])
6868
? each.value.route_table_id
69-
: module.route_tables[each.value.alternative_route_table].route_table_id) # If null, the VCN's default route table is used
70-
dhcp_options_id = each.value.dhcp_options_id # If null, the VCN's default set of DHCP options is used
71-
security_list_ids = (anytrue([(each.value.alternative_security_list == ""), (each.value.alternative_security_list == null)]) # If null, the VCN's default security list is used
72-
? each.value.security_list_ids
73-
: [module.security_lists[each.value.alternative_security_list].security_list_id])
74-
ipv6cidr_block = each.value.ipv6cidr_block # If null, no IPv6 CIDR block is assigned
69+
: module.route_tables[each.value.alternative_route_table].route_table_id) # If null, the VCN's default route table is used
70+
dhcp_options_id = each.value.dhcp_options_id # If null, the VCN's default set of DHCP options is used
71+
security_list_ids = concat(each.value.security_list_ids, [for v in each.value.extra_security_list_names : module.security_lists[v].security_list_id]) # If null, the VCN's default security list is used
72+
ipv6cidr_block = each.value.ipv6cidr_block # If null, no IPv6 CIDR block is assigned
73+
74+
# security_list_ids = (anytrue([(each.value.alternative_security_list == ""), (each.value.alternative_security_list == null)]) # If null, the VCN's default security list is used
75+
# ? each.value.security_list_ids
76+
# : [module.security_lists[each.value.alternative_security_list].security_list_id])
7577
}
7678

7779
################################################################################

variables.tf

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -107,6 +107,14 @@ variable "extra_security_lists" {
107107
default = []
108108
description = "Extra security lists to be created."
109109
}
110+
variable "extra_security_list_name_for_api_endpoint" {
111+
default = null
112+
description = "Extra security list name previosly created to be used by the K8s API Endpoint Subnet."
113+
}
114+
variable "extra_security_list_name_for_nodes" {
115+
default = null
116+
description = "Extra security list name previosly created to be used by the Nodes Subnet."
117+
}
110118

111119
################################################################################
112120
# Variables: OKE Network

0 commit comments

Comments
 (0)