Skip to content

chore: Remove unused parameters and definitions from main.parameters.json and main.waf.parameters.json#371

Open
Harmanpreet-Microsoft wants to merge 2 commits intodevfrom
psl-bicep-param
Open

chore: Remove unused parameters and definitions from main.parameters.json and main.waf.parameters.json#371
Harmanpreet-Microsoft wants to merge 2 commits intodevfrom
psl-bicep-param

Conversation

@Harmanpreet-Microsoft
Copy link
Contributor

Purpose

This pull request simplifies the Azure infrastructure parameter files by removing several parameters related to backend and frontend resource definitions, existence checks, and AI model deployments. The changes streamline the configuration, likely to reduce complexity and potential confusion in deployment.

Removed parameters from infrastructure configuration:

Backend and Frontend Resource Parameters:

  • Removed backendExists, frontendExists, backendDefinition, and frontendDefinition parameters from both main.parameters.json and main.waf.parameters.json. These parameters previously controlled the existence and configuration of backend and frontend resources. [1] [2] [3]

Identity and AI Model Deployment Parameters:

  • Removed principalId and aiModelDeployments parameters from both parameter files. These were used for Azure identity and specifying AI model deployment configurations. [1] [2]
  • ...

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

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