Merge pull request #2255 from alan-turing-institute/minor-logging-update

Standardise subscription logging
alan-turing-institute · Oct 23, 2024 · fa7122e · fa7122e
2 parents 6e6e7e5 + e8eb8de
commit fa7122e
Showing 1 changed file with 8 additions and 2 deletions.
diff --git a/data_safe_haven/commands/sre.py b/data_safe_haven/commands/sre.py
@@ -92,7 +92,7 @@ def deploy(
         )
         logger.info(
             f"SRE will be deployed to subscription '[green]{sre_subscription_name}[/]'"
-            f" ('[bold]{sre_config.azure.subscription_id}[/]')"
+            f" ({sre_config.azure.subscription_id})"
         )
         # Set Entra options
         application = graph_api.get_application_by_name(context.entra_application_name)
@@ -136,7 +136,13 @@ def deploy(
             replace=True,
         )
         logger.info(f"SRE will be registered in SHM '[green]{shm_config.shm.fqdn}[/]'")
-        logger.info(f"SHM subscription '[green]{shm_config.azure.subscription_id}[/]'")
+        shm_subscription_name = azure_sdk.get_subscription_name(
+            shm_config.azure.subscription_id
+        )
+        logger.info(
+            f"SHM is deployed to subscription '[green]{shm_subscription_name}[/]'"
+            f" ({shm_config.azure.subscription_id})"
+        )
 
         # Deploy Azure infrastructure with Pulumi
         try: