Skip to content

fix: Fix to resolve SFI-W18 policy issue#562

Merged
Prajwal-Microsoft merged 2 commits intomainfrom
psl-resolve-sfi-w18
Apr 23, 2026
Merged

fix: Fix to resolve SFI-W18 policy issue#562
Prajwal-Microsoft merged 2 commits intomainfrom
psl-resolve-sfi-w18

Conversation

@NirajC-Microsoft
Copy link
Copy Markdown
Contributor

Purpose

This pull request updates the infrastructure templates for the Content Processing Solution Accelerator, focusing on improving tag management, output variables, and resource dependency ordering. There are also minor fixes to resource scoping and generator metadata.

Key changes:

Tag and Metadata Improvements:

  • Updated the tags for the managed environment in both infra/main.bicep and infra/main_custom.bicep to include resource group tags, custom tags, template name, deployment type, creator, and deployment name, while removing the redundant location tag. This ensures richer and more consistent metadata across resources. [1] [2] [3]

Output and Variable Additions:

  • Added a new output variable CONTENT_UNDERSTANDING_ACCOUNT_NAME to infra/main.json to expose the Content Understanding AI Services account name for downstream use.

Resource Dependency and Ordering Fixes:

  • Corrected the order of dependencies in several dependsOn arrays in infra/main.json to ensure resources are created in the correct sequence, which can help avoid deployment issues. [1] [2] [3]

Resource Scope and Reference Fixes:

  • Updated several resource scope definitions in infra/main.json to use the correct string formatting for referencing Cognitive Services accounts, improving reliability and consistency. [1] [2] [3] [4] [5] [6]

Template Generator Metadata:

  • Updated the Bicep generator version and template hash metadata in multiple locations within infra/main.json, reflecting the use of a different Bicep compiler version. [1] [2] [3] [4] [5] [6] [7] [8] [9] [10] [11] [12]

Does this introduce a breaking change?

  • Yes
  • No

Golden Path Validation

  • I have tested the primary workflows (the "golden path") to ensure they function correctly without errors.

Deployment Validation

  • I have validated the deployment process successfully and all services are running as expected with this change.

What to Check

Verify that the following are valid

  • ...

Other Information

@Prajwal-Microsoft Prajwal-Microsoft merged commit a8592ce into main Apr 23, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants