Skip to content

Conversation

@AjitPadhi-Microsoft
Copy link
Contributor

@AjitPadhi-Microsoft AjitPadhi-Microsoft commented Nov 10, 2025

Description

This pull request removes three Bicep modules related to AI Foundry resource provisioning and key management. The modules previously managed the deployment and configuration of AI Hub, AI Services, Search Services, and related Azure resources, as well as the handling of secrets in Azure Key Vault. The removal simplifies the codebase and likely reflects a shift to newer, more secure, or standardized deployment patterns.

The most important changes are:

Removal of AI Foundry resource provisioning modules:

  • Deleted the ai-foundry.bicep module, which orchestrated the deployment of multiple Azure resources (AI Hub, AI Project, AI Services, Search, Storage, Container Registry, Application Insights) and managed their configuration and interconnections.
  • Deleted the ai-foundry-ai-hub.bicep module, responsible for provisioning and configuring the AI Hub resource, including its connections to Cognitive Services and Search, and passing resource keys.

Removal of Key Vault secret management:

  • Deleted the ai-foundry-kv-secret-cog-services-key.bicep module, which handled storing Cognitive Services keys as secrets in Azure Key Vault.

Pipeline Reference

Pipeline
avm.ptn.sa.conversation-knowledge-mining

Type of Change

  • Azure Verified Module updates:
    • Bugfix containing backwards-compatible bug fixes, and I have NOT bumped the MAJOR or MINOR version in version.json:
    • Feature update backwards compatible feature updates, and I have bumped the MINOR version in version.json.
    • Breaking changes and I have bumped the MAJOR version in version.json.
    • Update to documentation
  • Update to CI Environment or utilities (Non-module affecting changes)

Checklist

  • I'm sure there are no other open Pull Requests for the same update/change
  • I have run Set-AVMModule locally to generate the supporting module files.
  • My corresponding pipelines / checks run clean and green without any errors or warnings
  • I have updated the module's CHANGELOG.md file with an entry for the next version

@AjitPadhi-Microsoft AjitPadhi-Microsoft requested review from a team as code owners November 10, 2025 09:18
@microsoft-github-policy-service microsoft-github-policy-service bot added the Needs: Triage 🔍 Maintainers need to triage still label Nov 10, 2025
@avm-organizer avm-organizer bot added the Needs: Module Owner 📣 This module needs an owner to develop or maintain it label Nov 10, 2025
@microsoft-github-policy-service microsoft-github-policy-service bot added the Type: AVM 🅰️ ✌️ Ⓜ️ This is an AVM related issue label Nov 10, 2025
@avm-organizer avm-organizer bot requested a review from alguadam November 10, 2025 09:18
@AjitPadhi-Microsoft AjitPadhi-Microsoft changed the title avm module update for ckm feat: avm module update for ckm Nov 10, 2025
@AjitPadhi-Microsoft AjitPadhi-Microsoft changed the title feat: avm module update for ckm feat: avm module upgrace for CKM Nov 10, 2025
@AjitPadhi-Microsoft AjitPadhi-Microsoft changed the title feat: avm module upgrace for CKM feat: avm module upgrade for CKM Nov 11, 2025
@Prajwal-Microsoft
Copy link
Contributor

@AjitPadhi-Microsoft ,

Also changelog & version.json needs to be updated.

@AjitPadhi-Microsoft AjitPadhi-Microsoft requested a review from a team as a code owner November 12, 2025 15:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Needs: Module Owner 📣 This module needs an owner to develop or maintain it Needs: Triage 🔍 Maintainers need to triage still Type: AVM 🅰️ ✌️ Ⓜ️ This is an AVM related issue

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants