Skip to content

Commit

Permalink
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Merge pull request #37 from zscaler/ZTW-4057
Browse files Browse the repository at this point in the history
feat: removing the assignment of Storage Blob Data Reader to the managed identity in terraform
rhartnett-zscaler authored May 14, 2024

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
2 parents 88c6fd1 + d57db06 commit 5bb18b6
Showing 1 changed file with 0 additions and 8 deletions.
8 changes: 0 additions & 8 deletions modules/terraform-zscc-function-app-azure/main.tf
Original file line number Diff line number Diff line change
@@ -43,14 +43,6 @@ resource "azurerm_storage_blob" "cc_function_storage_blob" {
content_md5 = filemd5("${path.module}/zscaler_cc_function_app.zip")
}

# Restrict storage account blob access to only CC/Function App Managed Identity
resource "azurerm_role_assignment" "cc_function_role_assignment_storage" {
count = var.upload_function_app_zip ? 1 : 0
scope = local.storage_account_id
role_definition_name = "Storage Blob Data Reader"
principal_id = var.managed_identity_principal_id
}

# Create App Service Plan
resource "azurerm_service_plan" "vmss_orchestration_app_service_plan" {
name = "${var.name_prefix}-ccvmss-${var.resource_tag}-app-service-plan"

0 comments on commit 5bb18b6

Please sign in to comment.