Bug fixes aligned to logged issues (#217)

* Fix #216 * Change scope of managed_by_module to fix #204 * Consider both scopes to fix #204 Co-authored-by: Matt White <[email protected]>
Azure · Nov 25, 2021 · 2185380 · 2185380
1 parent 25e2047
commit 2185380
Showing 1 changed file with 11 additions and 1 deletion.
diff --git a/modules/connectivity/locals.tf b/modules/connectivity/locals.tf
@@ -774,7 +774,7 @@ locals {
         {
           # Resource logic attributes
           resource_id       = "${zone.resource_id}/virtualNetworkLinks/${link_config.name}"
-          managed_by_module = zone.managed_by_module
+          managed_by_module = zone.managed_by_module && link_config.managed_by_module
           # Resource definition attributes
           name                  = link_config.name
           resource_group_name   = zone.resource_group_name
@@ -859,6 +859,16 @@ locals {
         Deploy-Private-DNS-Zones = local.deploy_dns
       }
     }
+    "${local.root_id}-landing-zones" = {
+      parameters = {
+        Enable-DDoS-VNET = {
+          ddosPlan = local.ddos_protection_plan_resource_id
+        }
+      }
+      enforcement_mode = {
+        Enable-DDoS-VNET = local.deploy_ddos_protection_plan
+      }
+    }
   }
 }