fix: merging dev to main changes #661

Roopan-Microsoft · 2025-09-08T06:12:24Z

Purpose

This pull request introduces an optional CreatedBy tag to resource group deployments, improves documentation clarity, and fixes a minor typo in troubleshooting steps. The main changes are grouped below:

Infrastructure and Deployment:

Added a new createdBy parameter to infra/main.bicep, which defaults to the deployer's username and is used to set the CreatedBy tag on resource groups.
Updated the deployment workflow in .github/workflows/CAdeploy.yml to pass the createdBy parameter with the value "Pipeline" during deployments.

Documentation:

Corrected a typo in the troubleshooting documentation, changing ParentResourceNotfound to ParentResourceNotFound in docs/TroubleShootingSteps.md.

Does this introduce a breaking change?

Yes
No

Golden Path Validation

I have tested the primary workflows (the "golden path") to ensure they function correctly without errors.

Deployment Validation

I have validated the deployment process successfully and all services are running as expected with this change.

What to Check

Verify that the following are valid

...

Other Information

…tingpr fix: model deployment when reusing existing foundry

* fixed opent telemetry issue CustomDomainInUse, FlagMustBeSetForRestore (#618) (#619) Co-authored-by: VishalS-Microsoft <[email protected]> * directory update in dependabot template (#634) --------- Co-authored-by: NirajC-Microsoft <[email protected]> Co-authored-by: VishalS-Microsoft <[email protected]> Co-authored-by: Prajwal-Microsoft <[email protected]>

* Agent deletion handled successfully * use get azure credentials * remove unused import --------- Co-authored-by: Shreyas-Microsoft <[email protected]>

…ilot-Solution-Accelerator into PSLRJ-TS-21657

… azd up

feat: Add required environment variables for local debugging in .env generated by azd up

…ur-own-copilot-Solution-Accelerator into psl-bug-21342

…ilot-Solution-Accelerator into PSLRJ-TS-21657

fix: Sample data processing script fails when using cross-subscription existing AI project resource ID

chore: Down merge from main to dev

…ilot-Solution-Accelerator into PSLRJ-TS-21657

docs: Azure Region Selection Updated in Deployment.Md Docs

Fixed typo error. Co-authored-by: Copilot <[email protected]>

Updated the finally block for function "get_SQL_Response" Co-authored-by: Copilot <[email protected]>

Removed Commented-out code Co-authored-by: Copilot <[email protected]>

`project_client` is always initialized before thread creation, so `if thread:` is sufficient.

* Added Troubleshoot.md files for BYOC-Client Advisor * fixed Check Markdown Broken Links * updated troubleshotingsteps file * Changed MACAE Github issues link with BYOC issue link --------- Co-authored-by: NirajC-Microsoft <[email protected]>

Co-authored-by: Copilot <[email protected]>

…teps.md Co-authored-by: Copilot <[email protected]>

Co-authored-by: Copilot <[email protected]>

…entifier for better formatting and readability. Co-authored-by: Copilot <[email protected]>

…y, and the total value matches the asset value displayed in the left panel. (#651)

Co-authored-by: Copilot <[email protected]>

…to prevent validation pipeline failures caused by the deployer function in the Bicep file. (#659) * Adding a hardcoded createdBy parameter to the CAdeploy.yml file to prevent validation pipeline failures caused by the deployer function in the Bicep file. * Consolidate parameters in CAdeploy.yml

Copilot

Pull Request Overview

This pull request adds an optional CreatedBy tag to infrastructure deployments and includes documentation fixes. The changes enhance resource tagging capabilities while correcting minor documentation errors.

Adds a createdBy parameter to Bicep templates that defaults to the deployer's username
Updates CI/CD pipeline to set the CreatedBy tag to "Pipeline" for automated deployments
Fixes a typo in troubleshooting documentation

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File	Description
infra/main.bicep	Adds `createdBy` parameter and `CreatedBy` tag to resource group deployments
.github/workflows/CAdeploy.yml	Updates deployment command to include `createdBy` parameter
docs/TroubleShootingSteps.md	Corrects typo from `ParentResourceNotfound` to `ParentResourceNotFound`

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

github-actions · 2025-09-08T06:42:21Z

🎉 This PR is included in version 1.8.2 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

* chore: Dev to Main changes merge (#657) * model deployment when reusing existing foundry * change model deployment name * removed system managed identity code * seggregated the files * fix: Updated file names * chore: down merging main into dev (#638) * fixed opent telemetry issue CustomDomainInUse, FlagMustBeSetForRestore (#618) (#619) Co-authored-by: VishalS-Microsoft <[email protected]> * directory update in dependabot template (#634) --------- Co-authored-by: NirajC-Microsoft <[email protected]> Co-authored-by: VishalS-Microsoft <[email protected]> Co-authored-by: Prajwal-Microsoft <[email protected]> * added for cross-subscription existing AI project resource ID * update * fix: agent cleanup (#639) * Agent deletion handled successfully * use get azure credentials * remove unused import --------- Co-authored-by: Shreyas-Microsoft <[email protected]> * TS( 21657) Bicep Standard code changes * Added required env variables for local debugging in .env generated by azd up * Added "Required/Optional" and changed aiProjectName suffix. * removed location tag * foundry project documentation * corrected name * updated CAdeploy.yml * update CAdeploy.yml * changed pipeline variable * updated main.json file * updated yml file * test * test * update * update * test * update CAdeploy.yml * CustomizingAzdparamMd * deploymentMdUpdate * Updated * Update infra/main.bicep Fixed typo error. Co-authored-by: Copilot <[email protected]> * Update src/App/backend/plugins/chat_with_data_plugin.py Updated the finally block for function "get_SQL_Response" Co-authored-by: Copilot <[email protected]> * Update infra/main.bicep Removed Commented-out code Co-authored-by: Copilot <[email protected]> * Revert Copilot suggestion: `and project_client` is unnecessary. `project_client` is always initialized before thread creation, so `if thread:` is sufficient. * docs: Added Troubleshoot.md files for BYOC-Client Advisor (#650) * Added Troubleshoot.md files for BYOC-Client Advisor * fixed Check Markdown Broken Links * updated troubleshotingsteps file * Changed MACAE Github issues link with BYOC issue link --------- Co-authored-by: NirajC-Microsoft <[email protected]> * added colon Co-authored-by: Copilot <[email protected]> * Replace 'u' with 'you' for proper grammarUpdate docs/TroubleShootingSteps.md Co-authored-by: Copilot <[email protected]> * extra space removed Co-authored-by: Copilot <[email protected]> * The command block is not properly closed Co-authored-by: Copilot <[email protected]> * The command should be wrapped in a proper code block with language identifier for better formatting and readability. Co-authored-by: Copilot <[email protected]> * The total asset value and current asset value are responding correctly, and the total value matches the asset value displayed in the left panel. (#651) * Bicep file changes to add CreatedBy tag (#655) * Update docs/TroubleShootingSteps.md Co-authored-by: Copilot <[email protected]> * fix: Adding a hardcoded createdBy parameter to the CAdeploy.yml file to prevent validation pipeline failures caused by the deployer function in the Bicep file. (#659) * Adding a hardcoded createdBy parameter to the CAdeploy.yml file to prevent validation pipeline failures caused by the deployer function in the Bicep file. * Consolidate parameters in CAdeploy.yml --------- Co-authored-by: Priyanka-Microsoft <[email protected]> Co-authored-by: Prajwal D C <[email protected]> Co-authored-by: Rafi-Microsoft <[email protected]> Co-authored-by: VishalS-Microsoft <[email protected]> Co-authored-by: Kanchan Nagshetti (Persistent Systems Inc) <[email protected]> Co-authored-by: Shreyas-Microsoft <[email protected]> Co-authored-by: Shreyas-Microsoft <[email protected]> Co-authored-by: Bangarraju-Microsoft <[email protected]> Co-authored-by: Prasanjeet-Microsoft <[email protected]> Co-authored-by: UtkarshMishra-Microsoft <[email protected]> Co-authored-by: Copilot <[email protected]> Co-authored-by: Roopan P M <[email protected]> Co-authored-by: Prekshith-Microsoft <[email protected]> Co-authored-by: Roopan-Microsoft <[email protected]> * fix: merging dev to main changes (#661) * model deployment when reusing existing foundry * change model deployment name * removed system managed identity code * seggregated the files * fix: Updated file names * chore: down merging main into dev (#638) * fixed opent telemetry issue CustomDomainInUse, FlagMustBeSetForRestore (#618) (#619) Co-authored-by: VishalS-Microsoft <[email protected]> * directory update in dependabot template (#634) --------- Co-authored-by: NirajC-Microsoft <[email protected]> Co-authored-by: VishalS-Microsoft <[email protected]> Co-authored-by: Prajwal-Microsoft <[email protected]> * added for cross-subscription existing AI project resource ID * update * fix: agent cleanup (#639) * Agent deletion handled successfully * use get azure credentials * remove unused import --------- Co-authored-by: Shreyas-Microsoft <[email protected]> * TS( 21657) Bicep Standard code changes * Added required env variables for local debugging in .env generated by azd up * Added "Required/Optional" and changed aiProjectName suffix. * removed location tag * foundry project documentation * corrected name * updated CAdeploy.yml * update CAdeploy.yml * changed pipeline variable * updated main.json file * updated yml file * test * test * update * update * test * update CAdeploy.yml * CustomizingAzdparamMd * deploymentMdUpdate * Updated * Update infra/main.bicep Fixed typo error. Co-authored-by: Copilot <[email protected]> * Update src/App/backend/plugins/chat_with_data_plugin.py Updated the finally block for function "get_SQL_Response" Co-authored-by: Copilot <[email protected]> * Update infra/main.bicep Removed Commented-out code Co-authored-by: Copilot <[email protected]> * Revert Copilot suggestion: `and project_client` is unnecessary. `project_client` is always initialized before thread creation, so `if thread:` is sufficient. * docs: Added Troubleshoot.md files for BYOC-Client Advisor (#650) * Added Troubleshoot.md files for BYOC-Client Advisor * fixed Check Markdown Broken Links * updated troubleshotingsteps file * Changed MACAE Github issues link with BYOC issue link --------- Co-authored-by: NirajC-Microsoft <[email protected]> * added colon Co-authored-by: Copilot <[email protected]> * Replace 'u' with 'you' for proper grammarUpdate docs/TroubleShootingSteps.md Co-authored-by: Copilot <[email protected]> * extra space removed Co-authored-by: Copilot <[email protected]> * The command block is not properly closed Co-authored-by: Copilot <[email protected]> * The command should be wrapped in a proper code block with language identifier for better formatting and readability. Co-authored-by: Copilot <[email protected]> * The total asset value and current asset value are responding correctly, and the total value matches the asset value displayed in the left panel. (#651) * Bicep file changes to add CreatedBy tag (#655) * Update docs/TroubleShootingSteps.md Co-authored-by: Copilot <[email protected]> * fix: Adding a hardcoded createdBy parameter to the CAdeploy.yml file to prevent validation pipeline failures caused by the deployer function in the Bicep file. (#659) * Adding a hardcoded createdBy parameter to the CAdeploy.yml file to prevent validation pipeline failures caused by the deployer function in the Bicep file. * Consolidate parameters in CAdeploy.yml --------- Co-authored-by: Priyanka-Microsoft <[email protected]> Co-authored-by: Prajwal D C <[email protected]> Co-authored-by: Rafi-Microsoft <[email protected]> Co-authored-by: NirajC-Microsoft <[email protected]> Co-authored-by: VishalS-Microsoft <[email protected]> Co-authored-by: Kanchan Nagshetti (Persistent Systems Inc) <[email protected]> Co-authored-by: Shreyas-Microsoft <[email protected]> Co-authored-by: Shreyas-Microsoft <[email protected]> Co-authored-by: Bangarraju-Microsoft <[email protected]> Co-authored-by: Prasanjeet-Microsoft <[email protected]> Co-authored-by: UtkarshMishra-Microsoft <[email protected]> Co-authored-by: Copilot <[email protected]> Co-authored-by: Prekshith-Microsoft <[email protected]> * fixed vulnerabilities * removed duplicate variable --------- Co-authored-by: NirajC-Microsoft <[email protected]> Co-authored-by: Priyanka-Microsoft <[email protected]> Co-authored-by: Prajwal D C <[email protected]> Co-authored-by: Rafi-Microsoft <[email protected]> Co-authored-by: VishalS-Microsoft <[email protected]> Co-authored-by: Shreyas-Microsoft <[email protected]> Co-authored-by: Shreyas-Microsoft <[email protected]> Co-authored-by: Bangarraju-Microsoft <[email protected]> Co-authored-by: Prasanjeet-Microsoft <[email protected]> Co-authored-by: UtkarshMishra-Microsoft <[email protected]> Co-authored-by: Copilot <[email protected]> Co-authored-by: Roopan P M <[email protected]> Co-authored-by: Prekshith-Microsoft <[email protected]> Co-authored-by: Roopan-Microsoft <[email protected]>

github-actions · 2025-09-18T08:59:45Z

🎉 This PR is included in version 1.9.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Priyanka-Microsoft and others added 30 commits July 25, 2025 13:41

model deployment when reusing existing foundry

1a8347f

change model deployment name

383706b

removed system managed identity code

5ccc7c1

seggregated the files

99ad169

fix: Updated file names

e0747b2

Resolved main json commit

d604cea

Merge pull request #621 from microsoft/model-deployment-resusing-exis…

d4065d5

…tingpr fix: model deployment when reusing existing foundry

added for cross-subscription existing AI project resource ID

c509549

update

ad9ba63

fix: agent cleanup (#639)

18f3e78

* Agent deletion handled successfully * use get azure credentials * remove unused import --------- Co-authored-by: Shreyas-Microsoft <[email protected]>

TS( 21657) Bicep Standard code changes

0f8ccc5

Merge branch 'dev' of https://github.com/microsoft/Build-your-own-cop…

d499936

…ilot-Solution-Accelerator into PSLRJ-TS-21657

Added required env variables for local debugging in .env generated by…

e8a113f

… azd up

Merge pull request #642 from microsoft/PSL-US-21463

12b99bb

feat: Add required environment variables for local debugging in .env generated by azd up

Added "Required/Optional" and changed aiProjectName suffix.

2c17025

removed location tag

51df269

Merge branch 'dev' into psl-bug-21342

f9dbf41

foundry project documentation

f202b60

Merge branch 'psl-bug-21342' of https://github.com/microsoft/Build-yo…

313f6e3

…ur-own-copilot-Solution-Accelerator into psl-bug-21342

corrected name

8b24bca

Merge branch 'dev' of https://github.com/microsoft/Build-your-own-cop…

b0e5161

…ilot-Solution-Accelerator into PSLRJ-TS-21657

updated CAdeploy.yml

d455118

update CAdeploy.yml

a8e25dd

Merge pull request #640 from microsoft/psl-bug-21342

2f89864

fix: Sample data processing script fails when using cross-subscription existing AI project resource ID

changed pipeline variable

1df9fc6

Merge pull request #644 from microsoft/main

1166aa8

chore: Down merge from main to dev

Merge branch 'dev' of https://github.com/microsoft/Build-your-own-cop…

d9c999d

…ilot-Solution-Accelerator into PSLRJ-TS-21657

updated main.json file

a911f20

updated yml file

d3ed0ee

Prajwal-Microsoft and others added 17 commits August 14, 2025 12:28

Merge pull request #648 from microsoft/deploymentMd

45fd259

docs: Azure Region Selection Updated in Deployment.Md Docs

Update infra/main.bicep

39e0085

Fixed typo error. Co-authored-by: Copilot <[email protected]>

Update src/App/backend/plugins/chat_with_data_plugin.py

a1b363d

Updated the finally block for function "get_SQL_Response" Co-authored-by: Copilot <[email protected]>

Update infra/main.bicep

4f81d2a

Removed Commented-out code Co-authored-by: Copilot <[email protected]>

Revert Copilot suggestion: and project_client is unnecessary.

12ca750

`project_client` is always initialized before thread creation, so `if thread:` is sufficient.

Merge branch 'main' into dev

7232d77

added colon

9ed413c

Co-authored-by: Copilot <[email protected]>

Replace 'u' with 'you' for proper grammarUpdate docs/TroubleShootingS…

f9e0c08

…teps.md Co-authored-by: Copilot <[email protected]>

extra space removed

d31d0e8

Co-authored-by: Copilot <[email protected]>

The command block is not properly closed

3c2645f

Co-authored-by: Copilot <[email protected]>

The command should be wrapped in a proper code block with language id…

3f092e4

…entifier for better formatting and readability. Co-authored-by: Copilot <[email protected]>

The total asset value and current asset value are responding correctl…

ccc16de

…y, and the total value matches the asset value displayed in the left panel. (#651)

Bicep file changes to add CreatedBy tag (#655)

a0f3944

Update docs/TroubleShootingSteps.md

c900215

Co-authored-by: Copilot <[email protected]>

Merge branch 'main' into dev

3df581e

Copilot AI review requested due to automatic review settings September 8, 2025 06:12

Roopan-Microsoft requested review from Avijit-Microsoft, Prajwal-Microsoft, Vinay-Microsoft and aniaroramsft as code owners September 8, 2025 06:12

Copilot AI reviewed Sep 8, 2025

View reviewed changes

microsoft deleted a comment from Copilot AI Sep 8, 2025

Prajwal-Microsoft approved these changes Sep 8, 2025

View reviewed changes

Roopan-Microsoft merged commit 064a340 into main Sep 8, 2025
22 checks passed

github-actions bot added the released label Sep 8, 2025

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

fix: merging dev to main changes #661

fix: merging dev to main changes #661

Uh oh!

Roopan-Microsoft commented Sep 8, 2025 •

edited

Loading

Uh oh!

Copilot AI left a comment

Uh oh!

Uh oh!

github-actions bot commented Sep 8, 2025

Uh oh!

github-actions bot commented Sep 18, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

12 participants

fix: merging dev to main changes #661

fix: merging dev to main changes #661

Uh oh!

Conversation

Roopan-Microsoft commented Sep 8, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Purpose

Does this introduce a breaking change?

Golden Path Validation

Deployment Validation

What to Check

Other Information

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Reviewed Changes

Uh oh!

Uh oh!

github-actions bot commented Sep 8, 2025

Uh oh!

github-actions bot commented Sep 18, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

12 participants

Roopan-Microsoft commented Sep 8, 2025 •

edited

Loading