Skip to content

Commit 0625909

Browse files
committed
removed a few more roles
1 parent 488aa77 commit 0625909

File tree

2 files changed

+4
-29
lines changed

2 files changed

+4
-29
lines changed

infra/main.search_configuration.tf

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -241,14 +241,14 @@ resource "time_sleep" "wait_for_rbac" {
241241
# Main storage permissions (write access needed for upload_data.py to upload data files)
242242
# azurerm_role_assignment.script_main_storage_queue_contributor,
243243
azurerm_role_assignment.script_main_storage_blob_owner,
244-
azurerm_role_assignment.script_main_storage_file_contributor,
244+
# azurerm_role_assignment.script_main_storage_file_contributor,
245245
# AI Search permissions
246246
azurerm_role_assignment.script_search_service_contributor,
247247
# azurerm_role_assignment.script_search_index_data_contributor,
248248
# Azure OpenAI permissions
249249
azurerm_role_assignment.script_cognitive_services_openai_user,
250250
# Other permissions
251-
azurerm_role_assignment.script_container_apps_contributor
251+
# azurerm_role_assignment.script_container_apps_contributor
252252
]
253253
create_duration = "30s"
254254
}
@@ -457,7 +457,7 @@ resource "null_resource" "verify_rbac_propagation" {
457457
# Storage permissions
458458
# azurerm_role_assignment.script_main_storage_queue_contributor,
459459
azurerm_role_assignment.script_main_storage_blob_owner,
460-
azurerm_role_assignment.script_main_storage_file_contributor,
460+
# azurerm_role_assignment.script_main_storage_file_contributor,
461461
# azurerm_role_assignment.script_deployment_container_storage_contributor,
462462
azurerm_role_assignment.script_deployment_container_blob_contributor,
463463
azurerm_role_assignment.script_deployment_container_file_owner,

infra/main.security.tf

Lines changed: 1 addition & 26 deletions
Original file line numberDiff line numberDiff line change
@@ -52,25 +52,7 @@ resource "azurerm_role_assignment" "script_search_service_contributor" {
5252
resource "azurerm_role_assignment" "script_main_storage_blob_owner" {
5353
principal_id = azurerm_user_assigned_identity.script_identity.principal_id
5454
scope = module.storage_account_and_container.resource_id
55-
role_definition_name = "Storage Blob Data Owner"
56-
}
57-
58-
resource "azurerm_role_assignment" "script_main_storage_file_contributor" {
59-
principal_id = azurerm_user_assigned_identity.script_identity.principal_id
60-
scope = module.storage_account_and_container.resource_id
61-
role_definition_name = "Storage File Data Privileged Contributor"
62-
}
63-
64-
resource "azurerm_role_assignment" "script_main_storage_reader" {
65-
principal_id = azurerm_user_assigned_identity.script_identity.principal_id
66-
scope = module.storage_account_and_container.resource_id
67-
role_definition_name = "Reader"
68-
}
69-
70-
resource "azurerm_role_assignment" "script_main_storage_account_contributor" {
71-
principal_id = azurerm_user_assigned_identity.script_identity.principal_id
72-
scope = module.storage_account_and_container.resource_id
73-
role_definition_name = "Storage Account Contributor"
55+
role_definition_name = "Storage Blob Data Contributor"
7456
}
7557

7658
# --- Deployment Container Storage Account ---
@@ -86,13 +68,6 @@ resource "azurerm_role_assignment" "script_deployment_container_file_owner" {
8668
role_definition_name = "Storage File Data Privileged Contributor"
8769
}
8870

89-
# --- Other Permissions ---
90-
resource "azurerm_role_assignment" "script_container_apps_contributor" {
91-
principal_id = azurerm_user_assigned_identity.script_identity.principal_id
92-
scope = azurerm_resource_group.this.id
93-
role_definition_name = "Container Apps Contributor"
94-
}
95-
9671
# ============================================================================
9772
# TERRAFORM PRINCIPAL PERMISSIONS (for deployment-time operations)
9873
# ============================================================================

0 commit comments

Comments
 (0)