Skip to content

Commit

Permalink
Merge pull request #420 from imperva/cpapke/fix-azure-for-sonar
Browse files Browse the repository at this point in the history
re-add role skipping
  • Loading branch information
jsonar-cpapke authored Jun 24, 2024
2 parents 93e2914 + 9966b5e commit a6268c8
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions modules/azurerm/sonar-base-instance/main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -104,6 +104,7 @@ resource "azurerm_user_assigned_identity" "dsf_base" {
data "azurerm_subscription" "current" {}

resource "azurerm_role_assignment" "dsf_base_storage_role_assignment" {
count = var.binaries_location.az_resource_group != "" ? 1 : 0
scope = "${data.azurerm_subscription.current.id}/resourceGroups/${var.binaries_location.az_resource_group}/providers/Microsoft.Storage/storageAccounts/${var.binaries_location.az_storage_account}/blobServices/default/containers/${var.binaries_location.az_container}"
role_definition_name = "Storage Blob Data Reader"
principal_id = azurerm_user_assigned_identity.dsf_base.principal_id
Expand Down

0 comments on commit a6268c8

Please sign in to comment.