Revert "fix(bicep): resolve ARM eager-eval index-out-of-bounds on EventHub namespace vars"

This reverts commit 50f93ec.

Author: Yamini-Microsoft

infra/main.bicep

@@ -75,18 +75,17 @@ param existingFabricCapacityName string = ''
 
 var useExistingEventHubNamespace = !empty(existingEventHubNamespaceId)
 // ARM eagerly evaluates both ternary branches at validation time, causing split('', '/')[n]
-// to throw index-out-of-bounds when the param is empty. Normalise to a structurally valid
-// ARM resource ID using the nil GUID (00000000-...) as subscription — a GUID Azure never
-// issues — so scope: resourceGroup(...) passes validation but can never target anything real.
-// The module is still gated by if(useExistingEventHubNamespace) and never deploys unless
-// a real existing namespace ID is provided.
+// to throw index-out-of-bounds when the param is empty. Provide a structurally valid
+// placeholder so the split is always safe. The nil GUID can never match a real subscription.
 var _safeEventHubNamespaceId = empty(existingEventHubNamespaceId)
   ? '/subscriptions/00000000-0000-0000-0000-000000000000/resourceGroups/placeholder/providers/Microsoft.EventHub/namespaces/placeholder'
   : existingEventHubNamespaceId
 // Extract name, subscription ID, and resource group from the resource ID.
-var eventHubNamespaceNameFromId = last(split(_safeEventHubNamespaceId, '/'))
-var eventHubNamespaceSubscriptionId = split(_safeEventHubNamespaceId, '/')[2]
-var eventHubNamespaceResourceGroup = split(_safeEventHubNamespaceId, '/')[4]
+// Supports cross-subscription/cross-resource-group deployments
+// (e.g. namespace in sub-123/rg-shared while deploying to sub-456/rg-demo).
+var eventHubNamespaceNameFromId = useExistingEventHubNamespace ? last(split(_safeEventHubNamespaceId, '/')) : ''
+var eventHubNamespaceSubscriptionId = useExistingEventHubNamespace ? split(_safeEventHubNamespaceId, '/')[2] : ''
+var eventHubNamespaceResourceGroup = useExistingEventHubNamespace ? split(_safeEventHubNamespaceId, '/')[4] : ''
 var useExistingFabricCapacity = !empty(existingFabricCapacityName)
 
 var solutionSuffix = toLower(trim(replace(

infra/scripts/fabric/fabric_data_ingester.py

@@ -274,7 +274,7 @@ def load_data_to_fabric(
     args = parser.parse_args()
 
     try:
-        load_data_to_fabric(
+        _results = load_data_to_fabric(
             cluster_uri=args.cluster_uri,
             database_name=args.database,
             data_path=args.data_path,

infra/scripts/fabric/fabric_environment.py

@@ -22,7 +22,7 @@
 import os
 import base64
 from typing import Optional, Dict, Any
-from fabric_api import FabricWorkspaceApiClient
+from fabric_api import FabricWorkspaceApiClient, FabricApiError
 
 
 def read_environment_yml(file_path: str) -> str:

src/sample_data.py

@@ -318,7 +318,7 @@ def generate_sample_data(
 
 
 if __name__ == "__main__":
-    generate_sample_data(
+    _result = generate_sample_data(
         num_sites=2,
         num_assets_per_site=1,
         num_products=20,