fix: Bicep fix for Failed Private Endpoint #1056
Open
Ashwal-Microsoft wants to merge 1 commit into
Open
Conversation
c344c92 to
4e506cc
Compare
4e506cc to
ad93608
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This pull request introduces several important infrastructure and documentation updates to improve deployment reliability, enforce newer tooling requirements, and simplify network configuration management. The most significant changes include raising the minimum required Azure Developer CLI (
azd) version, clarifying deployment prerequisites and troubleshooting steps, and refactoring the virtual network and network security group (NSG) logic in the Bicep infrastructure code. Additionally, deployment workflows now add richer resource tagging and timestamping for traceability.Infrastructure and Bicep refactoring:
azdversion is now 1.18.0 (was 1.15.0), enforced inazure.yamland all documentation references. [1] [2]infra/main.bicepto a new module (modules/virtualNetwork.bicep), greatly simplifyingmain.bicepand centralizing network configuration. NSG modules and related variables are removed frommain.bicep.createdBytag logic is improved to handle cases whereuserPrincipalNameis missing, defaulting toobjectId, and a newTypetag is added to distinguish WAF/non-WAF deployments.Deployment workflow improvements:
deploy.yml,deploy-waf.yml) now generate and apply aCreatedDatetag with a UTC timestamp, and add other descriptive tags to improve traceability and auditing. [1] [2] [3] [4]Documentation and troubleshooting enhancements:
azdversion (1.18.0), with download links and upgrade reminders in both theREADME.mdandDeploymentGuide.md. [1] [2]LogAnalyticsReplicationDisable.md) explains how to disable Log Analytics workspace replication before resource deletion, and the deployment guide links to it for users who enable redundancy. [1] [2]Workflow and environment configuration:
workflow_dispatch), no longer running on every push tomain.AZURE_DEV_COLLECT_TELEMETRYenvironment variable is now passed to the Azure template validation workflow for improved telemetry handling.These changes collectively improve deployment clarity, enforce up-to-date tooling, and make network configuration more maintainable and modular.## Purpose
Does this introduce a breaking change?
How to Test
What to Check
Verify that the following are valid
Other Information