Feature/tas27 bug fix#1085
Open
Fr4nc3 wants to merge 33 commits into
Open
Conversation
…ompipeline fix: Remove Createdby Parameter from deploy.yml and change logic in bicep
fix: CI Pipeline Validate Deployment - MACAE
docs: Updated README, azure.yml for minimum azd version 1.18.0
fix: optimize the network module for Macae
chore: Add AZURE_DEV_COLLECT_TELEMETRY variable in azure-dev.yml file for MACAE-v2
fix: [Revert] MACAE-v3-Golden path Script
fix: troubleshooting.md portal link
Adds a full research packet under .copilot-tracking for validating MACAE WAF private-networking wording. Includes one consolidated report plus four subagent deep-dives covering IaC networking mechanisms, source-KB claim verification, wording-origin tracing, and isolation posture. The notes conclude that “via VNet integration” is imprecise as an umbrella statement and provide source-accurate replacement wording grounded in infra and docs evidence.
Ignore the .copilot-tracking directory.
Replace gpt-4.1/gpt-4.1-mini/o4-mini defaults with gpt-5-mini across content packs, infra bicep, docs, quota scripts, and backend/MCP config (API version bumped to 2025-04-01-preview, model version 2025-08-07). Add a configurable image-generation quality parameter (gptImageQuality, default "high") wired through infra bicep, parameters, MCP settings, and the image service. Remove the deprecated backend image_agent.py. Fix MCP server test paths and conftest sys.path ordering, clean up repo-root conftest dead code, and add test_image_service.py covering the quality parameter.
Fr4nc3
requested review from
Avijit-Microsoft and
Roopan-Microsoft
as code owners
Fr4nc3
requested review from
aniaroramsft,
dgp10801,
nchandhi and
toherman-msft
as code owners
Bump default GPT-5-mini quota from 150k to 250k tokens in deployment docs and quota check. Update model name examples and validation bypass list to reference gpt-5-mini/gpt-5 instead of legacy gpt-4o/gpt-4 models. Also removes obsolete .copilot-tracking WAF private-networking research notes.
Reverts the deployment stack from the AVM/gpt-5-mini configuration back to the previous gpt-4o vanilla Bicep setup: - Rework `infra/main.bicep` from a deployment router (bicep/avm/avm-waf flavors) back to an inlined single-flavor template deploying AI Foundry, Cosmos DB, Container Apps, App Service, and optional WAF/private-networking resources. - Re-add `infra/modules/virtualNetwork.bicep` with NSGs, VNet, and subnets. - Update GitHub workflows (`azure-dev.yml`, `deploy.yml`, `deploy-waf.yml`) to use service-principal login, older action versions, `checkquota.sh` paths, and gpt-4o parameters; convert `azure-dev.yml` to template validation. - Simplify `azure.yaml` by removing post-deploy hooks. - Revert `README.md` and `docs/DeploymentGuide.md`/`TroubleShootingSteps.md` to the prior Azure OpenAI / Semantic Kernel wording and structure.
Bump the default GPT model from gpt-5-mini (and gpt-4o in the base infra template) to gpt-5.4-mini with version 2026-03-17. Updates deployment workflow, all content pack agent teams, Bicep infrastructure templates, quota check scripts, knowledge base seeding, backend env sample, and app config defaults.
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
9 participants
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.
Purpose
Does this introduce a breaking change?
How to Test
What to Check
Verify that the following are valid
Other Information