-
Notifications
You must be signed in to change notification settings - Fork 530
feat: New Pattern Module avm/ptn/sa/customer-chatbot #6457
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
… customer-chatbot
|
Note for the core team's initial review: the MAR record PR for this module has been submitted, awaiting approval. |
| @@ -0,0 +1,26 @@ | |||
| param aifSearchConnectionName string | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add description for all params
| properties: { | ||
| category: 'CognitiveSearch' | ||
| target: 'https://${searchServiceName}.search.windows.net' | ||
| authType: 'ApiKey' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cant we use Identity based connection here ?
| } | ||
| } | ||
|
|
||
| output name string = virtualNetwork.outputs.name |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add description for output
| ))) | ||
|
|
||
| // Region pairs list based on article in [Azure Database for MySQL Flexible Server - Azure Regions](https://learn.microsoft.com/azure/mysql/flexible-server/overview#azure-regions) for supported high availability regions for CosmosDB. | ||
| var cosmosDbZoneRedundantHaRegionPairs = { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No need to have the dependency on regions, this can be removed and have a param for Cosmos Replica location
This pull request introduces a new "customer-chatbot" AVM module, including all necessary infrastructure, deployment automation, and repository configuration. The main changes add the module's Bicep templates, CI workflow, code ownership, and issue template support.
Key changes include:
1. New Module Addition:
avm/ptn/sa/customer-chatbotmodule, including its Bicep templates for AI project provisioning, AI Foundry connections, and virtual network infrastructure. [1] [2] [3]CHANGELOG.mdfor the new module with an initial entry.2. CI/CD and Repository Integration:
.github/workflows/avm.ptn.sa.customer-chatbot.yml) for building, validating, and deploying the new module..github/CODEOWNERSto assign ownership of the new module to the appropriate teams.Pipeline Reference
Checklist
Set-AVMModulelocally to generate the supporting module files.