-
Notifications
You must be signed in to change notification settings - Fork 371
feat: Move changes of Dev branch to Main #310
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
Merged
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
fix: Bug fixes of #19515,#19330,#19278
fix: Bugs fixing for 19597,19553,19524,19330,19278,19238
fix: Consistent Date Formatting Across Product, HR, and GroupChat Responses Based on User Locale
chore: Reuse Ai Foundry for MACAE
docs: added missing parameter
…based on existing and new deployment
fix: Bug fix # 20359
fix: Fix to avoid task creation for violent and meaning less prompts when deployment reuse existing AI Foundry Project.
fix: Macae bug fixes
Roopan-Microsoft
approved these changes
Jul 10, 2025
fix: Macae bug fixes for tooltip
Prajwal-Microsoft
approved these changes
Jul 10, 2025
docs: added required azd version in azure.yml file and updated DeploymentGuide.md file
Roopan-Microsoft
approved these changes
Jul 10, 2025
Prajwal-Microsoft
approved these changes
Jul 10, 2025
🎉 This PR is included in version 2.1.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
feat: Move changes of Dev branch to Main
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This pull request introduces several updates to the Azure infrastructure deployment templates and documentation. The changes focus on enhancing configurability, improving resource management, and cleaning up unused or outdated configurations. Key updates include the addition of new parameters for reusing existing AI Foundry projects, adjustments to AI services deployment logic, and removal of deprecated code.
Documentation Updates:
docs/CustomizingAzdParameters.md
anddocs/DeploymentGuide.md
to include descriptions forAZURE_ENV_FOUNDRY_PROJECT_ID
(reuse existing AI Foundry projects) andAZURE_ENV_MODEL_CAPACITY
(set GPT model capacity).Infrastructure Enhancements:
existingFoundryProjectResourceId
ininfra/main.bicep
to allow reusing existing AI Foundry projects. Adjusted deployment logic to handle both new and existing projects. Updated AI services module to usemodules/account/main.bicep
instead of the previous path.infra/modules/account/modules/keyVaultExport.bicep
to support exporting secrets to Azure Key Vault.Cleanup and Simplification:
infra/main.bicep
file, including telemetry resources, outbound access settings, and unused role assignments.Parameter Updates:
infra/main.parameters.json
: IncludedexistingFoundryProjectResourceId
for AI Foundry project reuse.## PurposeDoes this introduce a breaking change?
How to Test
What to Check
Verify that the following are valid
Other Information