PR for getting e2e deployment working [WIP] (#398)

* Adding EventHub creation to the deployment script * Upgrading API versions and fixing the if clause of KV property creation for EH * Adding SQL connection string and couple other minor fixes
feathr-ai · Jul 5, 2022 · a70afed · a70afed
1 parent 2854b51
commit a70afed
Show file tree

Hide file tree

Showing 2 changed files with 94 additions and 76 deletions.
diff --git a/docs/how-to-guides/azure-deployment.md b/docs/how-to-guides/azure-deployment.md
@@ -32,8 +32,8 @@ az ad app create --display-name $sitename --sign-in-audience AzureADMyOrg --web-
 
 #Fetch the ClientId, TenantId and ObjectId for the created app
 aad_clientId=$(az ad app list --display-name $sitename --query [].appId -o tsv)
-aad_objectId=$(az ad app list --display-name $sitename --query [].id -o tsv)
 aad_tenantId=$(az account tenant list --query [].tenantId -o tsv)
+aad_objectId=$(az ad app list --display-name $sitename --query [].id -o tsv)
 
 # Updating the SPA app created above, currently there is no CLI support to add redirectUris to a SPA, so we have to patch manually via az rest
 az rest --method PATCH --uri "https://graph.microsoft.com/v1.0/applications/$aad_objectId" --headers "Content-Type=application/json" --body "{spa:{redirectUris:['https://$sitename.azurewebsites.net/.auth/login/aad/callback']}}"