Skip to content

Commit 0f5aaef

Browse files
Merge pull request #18 from hund030/main
stop assigning roles to non-user principal
2 parents 12ea9cc + 6155fa2 commit 0f5aaef

File tree

2 files changed

+6
-1
lines changed

2 files changed

+6
-1
lines changed

infra/main.bicep

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,6 +35,8 @@ param openAiUrl string = '' // Set in main.parameters.json
3535
param openAiApiVersion string // Set in main.parameters.json
3636

3737
param principalId string // Set in main.parameters.json
38+
@description('Flag to decide where to create OpenAI role for current user')
39+
param createRoleForUser bool = true
3840

3941
var finalOpenAiUrl = empty(openAiUrl) ? 'https://${openAi.outputs.name}.openai.azure.com' : openAiUrl
4042
var abbrs = loadJsonContent('abbreviations.json')
@@ -148,7 +150,7 @@ module openAi 'core/ai/cognitiveservices.bicep' = if (empty(openAiUrl)) {
148150
// Roles
149151

150152
// User roles
151-
module openAiRoleUser 'core/security/role.bicep' = {
153+
module openAiRoleUser 'core/security/role.bicep' = if (createRoleForUser) {
152154
scope: resourceGroup
153155
name: 'openai-role-user'
154156
params: {

infra/main.parameters.json

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,9 @@
1414
"principalId": {
1515
"value": "${AZURE_PRINCIPAL_ID}"
1616
},
17+
"createRoleForUser": {
18+
"value": "${CREATE_ROLE_FOR_USER=true}"
19+
},
1720
"openAiLocation": {
1821
"value": "${AZURE_OPENAI_LOCATION=swedencentral}"
1922
},

0 commit comments

Comments
 (0)