Merge branch 'main' into loadazdenv

Author: pamelafox

app/backend/app.py

@@ -448,6 +448,7 @@ async def setup_clients():
     # The managed identity is setup in the infra/ folder.
     azure_credential: Union[AzureDeveloperCliCredential, ManagedIdentityCredential]
     if RUNNING_ON_AZURE:
+        current_app.logger.info("Setting up Azure credential using ManagedIdentityCredential")
         if AZURE_CLIENT_ID := os.getenv("AZURE_CLIENT_ID"):
             # ManagedIdentityCredential should use AZURE_CLIENT_ID if set in env, but its not working for some reason,
             # so we explicitly pass it in as the client ID here. This is necessary for user-assigned managed identities.

azure.yaml

@@ -9,8 +9,6 @@ services:
     language: py
     # Please check docs/azure_container_apps.md for more information on how to deploy to Azure Container Apps
     # host: containerapp
-    docker:
-      remoteBuild: true
     host: appservice
     hooks:
       prebuild:

scripts/adlsgen2setup.py

@@ -139,8 +139,11 @@ async def create_or_get_group(self, group_name: str):
                 logger.info(f"Could not find group {group_name}, creating...")
                 group = {
                     "displayName": group_name,
-                    "groupTypes": ["Unified"],
                     "securityEnabled": self.security_enabled_groups,
+                    "groupTypes": ["Unified"],
+                    # If Unified does not work for you, then you may need the following settings instead:
+                    # "mailEnabled": False,
+                    # "mailNickname": group_name,
                 }
                 async with session.post("https://graph.microsoft.com/v1.0/groups", json=group) as response:
                     content = await response.json()