Skip to content

fix: Update parameters for AI services and remove secondary location references#142

Open
Prachig-Microsoft wants to merge 2 commits intodevfrom
psl-prachi-params
Open

fix: Update parameters for AI services and remove secondary location references#142
Prachig-Microsoft wants to merge 2 commits intodevfrom
psl-prachi-params

Conversation

@Prachig-Microsoft
Copy link

Purpose

This pull request updates environment parameter documentation and infrastructure parameter files to better align with current Azure resource requirements and simplify configuration. The main changes are the removal of unused parameters and the addition of new, optional parameters for advanced scenarios.

Parameter documentation updates:

  • Removed the AZURE_SECONDARY_LOCATION parameter and its references, as it is no longer required. [1] [2] [3]
  • Added new optional parameters to the documentation:
    • AZURE_AI_SERVICE_LOCATION for specifying the Azure region for AI services (OpenAI/AI Foundry).
    • AZURE_ENV_LOG_ANALYTICS_WORKSPACE_ID for using an existing Log Analytics workspace.
    • AZURE_EXISTING_AI_PROJECT_RESOURCE_ID for using an existing AI Foundry project.

Infrastructure parameter file simplification:

  • Removed secondaryLocation and azureAiServiceLocation from infra/main.parameters.json and infra/main.waf.parameters.json to match the updated documentation and streamline deployment parameters. [1] [2] [3] [4]

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.

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