Skip to content

Conversation

@Abdul-Microsoft
Copy link
Contributor

Purpose

This pull request makes changes to the infrastructure configuration files to support a new database setup and improve parameter flexibility. The most significant updates include renaming the Cosmos DB database from autogen to macae, introducing a new parameter for the Azure OpenAI location, and updating dependencies and references accordingly.

Database Configuration Updates:

  • Renamed the Cosmos DB database from autogen to macae in both infra/main.bicep and infra/main.json files, including all associated references, such as database IDs, container names, and dependency lists. [1] [2] [3] [4] [5] [6] [7]

Parameter Enhancements:

  • Removed the default value for the azureOpenAILocation parameter in infra/main.bicep to allow greater flexibility.
  • Added a new parameter azureOpenAILocation in infra/main.bicepparam, which reads its value from the environment variable AZURE_OPENAI_LOCATION.

Template Adjustments:

  • Updated the templateHash in infra/main.json to reflect the changes made to the template.escribe the intention of the changes being proposed. What problem does it solve or functionality does it add? -->

Does this introduce a breaking change?

  • Yes
  • No

How to Test

  • Get the code
git clone [repo-address]
cd [repo-name]
git checkout [branch-name]
npm install
  • Test the code

What to Check

Verify that the following are valid

  • ...

Other Information

@Roopan-Microsoft Roopan-Microsoft merged commit d31def5 into dev May 19, 2025
7 of 8 checks passed
@Roopan-Microsoft Roopan-Microsoft deleted the psl-remove-autogenref branch May 19, 2025 08:36
@github-actions
Copy link

🎉 This PR is included in version 1.5.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

blessing-msft pushed a commit that referenced this pull request Aug 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants