Merge pull request #614 from microsoft/psl-fixPrincipalTypeIssue

Prajwal-Microsoft · web-flow · commit a8af9fd51041 · 2025-07-22T14:44:28.000+05:30
fix: add principalType as 'ServicePrincipal' for role assignments
diff --git a/infra/deploy_foundry_role_assignment.bicep b/infra/deploy_foundry_role_assignment.bicep
@@ -19,6 +19,7 @@ resource roleAssignmentToFoundry 'Microsoft.Authorization/roleAssignments@2022-0
   properties: {
     roleDefinitionId: roleDefinitionId
     principalId: principalId
+    principalType: 'ServicePrincipal'
   }
 }
 
diff --git a/infra/main.json b/infra/main.json
@@ -1489,7 +1489,8 @@
                       "name": "[parameters('roleAssignmentName')]",
                       "properties": {
                         "roleDefinitionId": "[parameters('roleDefinitionId')]",
-                        "principalId": "[parameters('principalId')]"
+                        "principalId": "[parameters('principalId')]",
+                        "principalType": "ServicePrincipal"
                       }
                     }
                   ],
@@ -2924,7 +2925,8 @@
                       "name": "[parameters('roleAssignmentName')]",
                       "properties": {
                         "roleDefinitionId": "[parameters('roleDefinitionId')]",
-                        "principalId": "[parameters('principalId')]"
+                        "principalId": "[parameters('principalId')]",
+                        "principalType": "ServicePrincipal"
                       }
                     }
                   ],

Original file line number	Diff line number	Diff line change
`@@ -19,6 +19,7 @@ resource roleAssignmentToFoundry 'Microsoft.Authorization/roleAssignments@2022-0`
`19`	`19`	`properties: {`
`20`	`20`	`roleDefinitionId: roleDefinitionId`
`21`	`21`	`principalId: principalId`
	`22`	`+ principalType: 'ServicePrincipal'`
`22`	`23`	`}`
`23`	`24`	`}`
`24`	`25`
Original file line number	Diff line number	Diff line change
`@@ -1489,7 +1489,8 @@`
`1489`	`1489`	`"name": "[parameters('roleAssignmentName')]",`
`1490`	`1490`	`"properties": {`
`1491`	`1491`	`"roleDefinitionId": "[parameters('roleDefinitionId')]",`
`1492`		`- "principalId": "[parameters('principalId')]"`
	`1492`	`+ "principalId": "[parameters('principalId')]",`
	`1493`	`+ "principalType": "ServicePrincipal"`
`1493`	`1494`	`}`
`1494`	`1495`	`}`
`1495`	`1496`	`],`
`@@ -2924,7 +2925,8 @@`
`2924`	`2925`	`"name": "[parameters('roleAssignmentName')]",`
`2925`	`2926`	`"properties": {`
`2926`	`2927`	`"roleDefinitionId": "[parameters('roleDefinitionId')]",`
`2927`		`- "principalId": "[parameters('principalId')]"`
	`2928`	`+ "principalId": "[parameters('principalId')]",`
	`2929`	`+ "principalType": "ServicePrincipal"`
`2928`	`2930`	`}`
`2929`	`2931`	`}`
`2930`	`2932`	`],`