Merge pull request #50 from microsoft/psl-fix-validate-template

fix: add createdBy param in ci.yml

Author: Roopan-Microsoft

.github/workflows/ci.yml

@@ -5,6 +5,9 @@ on:
       - main
       - dev
       - demo
+  pull_request:
+    branches:
+      - dev
   schedule:
     - cron: '0 9,21 * * *'  # Runs at 9:00 AM and 9:00 PM GMT
   workflow_dispatch:  # Allow manual triggering
@@ -113,7 +116,8 @@ jobs:
             --parameters solutionName=${{env.SOLUTION_PREFIX}} \
             --parameters location=${{ env.AZURE_LOCATION }} \
             --parameters aiDeploymentLocation=${{ env.AZURE_LOCATION }} \
-            --parameters azureAiServiceLocation=${{ env.AZURE_LOCATION }} 
+            --parameters azureAiServiceLocation=${{ env.AZURE_LOCATION }} \
+            --parameters createdBy="pipeline" \
 
 
       - name: Extract AI Services and Key Vault Names

infra/main.bicep

@@ -100,14 +100,12 @@ param cosmosLocation string = 'eastus2'
 param existingLogAnalyticsWorkspaceId string = ''
 
 @description('Tag, Created by user name')
-param createdBy string = contains(deployer(), 'userPrincipalName')
-  ? split(deployer().userPrincipalName, '@')[0]
-  : deployer().objectId
+param createdBy string = deployer().objectId
 
 // Get the current deployer's information for local debugging permissions
 var deployerInfo = deployer()
 var deployingUserPrincipalId = deployerInfo.objectId
-var deployingUserType = !empty(deployerInfo.userPrincipalName) ? 'User' : 'ServicePrincipal'
+var deployingUserType = contains(deployerInfo, 'userPrincipalName') ? 'User' : 'ServicePrincipal'
 
 @description('Optional. Resource ID of an existing Foundry project')
 param existingFoundryProjectResourceId string = ''