Skip to content

Commit

Permalink
Merge branch 'joselcaguilar-bug/agw-private-subnet-cidr-optional'
Browse files Browse the repository at this point in the history
  • Loading branch information
arnaudlh committed Aug 11, 2022
2 parents edb6459 + dbbe1f6 commit ea0363d
Show file tree
Hide file tree
Showing 6 changed files with 23 additions and 14 deletions.
2 changes: 1 addition & 1 deletion .devcontainer/docker-compose.yml
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@
version: '3.7'
services:
rover:
image: aztfmod/rover:1.2.5-2207.2909
image: aztfmod/rover:1.2.5-2208.0208
user: vscode

labels:
Expand Down
4 changes: 2 additions & 2 deletions .github/workflows/standalone-regressor-tf100.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -55,7 +55,7 @@ jobs:
matrix: ${{fromJSON(needs.load_scenarios.outputs.matrix)}}

container:
image: aztfmod/rover:1.2.5-2207.2909
image: aztfmod/rover:1.2.5-2208.0208
options: --user 0

steps:
Expand Down Expand Up @@ -165,7 +165,7 @@ jobs:
needs: [testcases]

container:
image: aztfmod/rover:1.2.5-2207.2909
image: aztfmod/rover:1.2.5-2208.0208
options: --user 0

steps:
Expand Down
4 changes: 2 additions & 2 deletions .github/workflows/standalone-tf100.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -51,7 +51,7 @@ jobs:
matrix: ${{fromJSON(needs.load_scenarios.outputs.matrix)}}

container:
image: aztfmod/rover:1.2.5-2207.2909
image: aztfmod/rover:1.2.5-2208.0208
options: --user 0

steps:
Expand Down Expand Up @@ -136,7 +136,7 @@ jobs:
needs: [testcases]

container:
image: aztfmod/rover:1.2.5-2207.2909
image: aztfmod/rover:1.2.5-2208.0208
options: --user 0

steps:
Expand Down
9 changes: 9 additions & 0 deletions examples/app_gateway/103-public-only/application.tfvars
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,15 @@ application_gateway_applications = {
application_gateway_key = "agw1"
name = "demoapp1"

listeners = {
public = {
name = "demo-app1-80-public"
front_end_ip_configuration_key = "public"
front_end_port_key = "80"
host_name = "cafdemo.com"
}
}

request_routing_rules = {
default = {
rule_type = "Basic"
Expand Down
16 changes: 8 additions & 8 deletions modules/networking/application_gateway/locals.networking.tf
Original file line number Diff line number Diff line change
Expand Up @@ -28,14 +28,14 @@ locals {
)
}
private = {
subnet_id = coalesce(
subnet_id = try(coalesce(
try(local.private_vnet.subnets[var.settings.front_end_ip_configurations.private.subnet_key].id, null),
try(var.settings.front_end_ip_configurations.private.subnet_id, null)
)
cidr = coalesce(
), null)
cidr = try(coalesce(
try(local.private_vnet.subnets[var.settings.front_end_ip_configurations.private.subnet_key].cidr, null),
try(var.settings.front_end_ip_configurations.private.subnet_cidr, null)
)
), null)
}
public = {
subnet_id = try(
Expand All @@ -44,17 +44,17 @@ locals {
null
)

ip_address_id = coalesce(
ip_address_id = try(coalesce(
try(var.public_ip_addresses[var.client_config.landingzone_key][var.settings.front_end_ip_configurations.public.public_ip_key].id, var.public_ip_addresses[var.settings.front_end_ip_configurations.public.lz_key][var.settings.front_end_ip_configurations.public.public_ip_key].id, null),
try(var.settings.front_end_ip_configurations.public.public_ip_id, null)
)
), null)
}
}

private_cidr = coalesce(
private_cidr = try(coalesce(
try(local.ip_configuration.private.cidr[var.settings.front_end_ip_configurations.private.subnet_cidr_index], null),
try(var.settings.front_end_ip_configurations.private.subnet_cidr, null)
)
), null)
private_ip_address = try(cidrhost(local.private_cidr, var.settings.front_end_ip_configurations.private.private_ip_offset), null)

}
2 changes: 1 addition & 1 deletion rover_on_ssh_host.yml
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@
version: '3.7'
services:
rover:
image: aztfmod/rover:1.2.5-2207.2909
image: aztfmod/rover:1.2.5-2208.0208

user: vscode

Expand Down

0 comments on commit ea0363d

Please sign in to comment.