Include AzureML FQDN as a bundle output (#3450)

* Fix aml outputs * Update changelog and version
microsoft · Apr 20, 2023 · 617d5bf · 617d5bf
1 parent de5f662
commit 617d5bf
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 2 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -9,6 +9,7 @@ ENHANCEMENTS:
 * Update Guacamole to version 1.5.1 ([#3443](https://github.com/microsoft/AzureTRE/issues/3443))
 
 BUG FIXES:
+* AML workspace service fails to install and puts firewall into failed state ([#3448](https://github.com/microsoft/AzureTRE/issues/3448))
 
 COMPONENTS:
 

diff --git a/templates/workspace_services/azureml/porter.yaml b/templates/workspace_services/azureml/porter.yaml
@@ -1,7 +1,7 @@
 ---
 schemaVersion: 1.0.0
 name: tre-service-azureml
-version: 0.8.1
+version: 0.8.2
 description: "An Azure TRE service for Azure Machine Learning"
 registry: azuretre
 dockerfile: Dockerfile.tmpl
@@ -115,6 +115,11 @@ outputs:
     applyTo:
       - install
       - upgrade
+  - name: aml_fqdn
+    type: string
+    applyTo:
+      - install
+      - upgrade
 
 mixins:
   - terraform:
@@ -155,6 +160,7 @@ install:
         - name: storage_tag
         - name: batch_tag
         - name: mcr_tag
+        - name: aml_fqdn
 
 upgrade:
   - terraform:
@@ -188,6 +194,7 @@ upgrade:
         - name: storage_tag
         - name: batch_tag
         - name: mcr_tag
+        - name: aml_fqdn
 
 uninstall:
   - terraform:

diff --git a/templates/workspace_services/azureml/terraform/outputs.tf b/templates/workspace_services/azureml/terraform/outputs.tf
@@ -11,7 +11,7 @@ output "azureml_storage_account_id" {
 }
 
 output "aml_fqdn" {
-  value = module.terraform_azurerm_environment_configuration.aml_studio_endpoint
+  value = regex("(?:(?P<scheme>[^:/?#]+):)?(?://(?P<fqdn>[^/?#:]*))?", module.terraform_azurerm_environment_configuration.aml_studio_endpoint).fqdn
 }
 
 output "connection_uri" {