Skip to content
This repository was archived by the owner on Sep 4, 2025. It is now read-only.

[Deploy] Onboard code to cloud tools #626

Merged
merged 68 commits into from
Aug 19, 2025
Merged

[Deploy] Onboard code to cloud tools #626

merged 68 commits into from
Aug 19, 2025

Conversation

@qianwens
Copy link
Member

@qianwens qianwens commented Jul 16, 2025 •
edited by joshfree
Loading

What does this PR do?

Add a new command group "deploy" with the tools for application deployment

GitHub issue number?

https://github.com/Azure/azure-mcp-pr/issues/276

Pre-merge checklist

  • I have read the contribution guidelines covering pull request process, code style, and testing
  • PR title is clear and informative
  • Commit history is clean with informative messages (no previously merged commits appear in PR history). See cleanup guide
  • Added comprehensive tests for core features
  • Added CHANGELOG.md entry for user-impacting changes (bug fixes, new features, UI/UX changes)
  • Spelling check passes with .\eng\common\spelling\Invoke-Cspell.ps1
  • For MCP tool changes, updated:
    • Documentation in README.md
    • Command list in /docs/azmcp-commands.md
    • End-to-end test prompts in /e2eTests/e2eTestPrompts.md
  • Team member live testing:
    • Security review: Review PR for security vulnerabilities and malicious code before running tests (e.g., cryptocurrency mining, email spam, data exfiltration, or other harmful activities)
    • Test execution: Add comment /azp run azure - mcp to trigger pipeline

Copilot AI review requested due to automatic review settings July 16, 2025 09:03
@qianwens qianwens requested review from a team as code owners July 16, 2025 09:03
@qianwens qianwens requested review from hallipr and joshfree July 16, 2025 09:03

This comment was marked as outdated.

Sign in to view

jongio
jongio reviewed Jul 17, 2025
View reviewed changes
g2vinay
g2vinay reviewed Jul 18, 2025
View reviewed changes
g2vinay
g2vinay reviewed Jul 18, 2025
View reviewed changes
g2vinay
g2vinay reviewed Jul 18, 2025
View reviewed changes
g2vinay
g2vinay reviewed Jul 18, 2025
View reviewed changes
xfz11 added 2 commits July 18, 2025 13:57
@xfz11
rename iac command (#6)
56a2c39 
* rename iac command

* revert changes

* update
@xfz11
update deployment plan template
60df1dc
@jongio
Copy link
Member

jongio commented Jul 18, 2025

Let's get Wallace approval and manual testing complete with all tools enabled before merging.

wbreza
wbreza reviewed Jul 21, 2025
View reviewed changes
Copy link
Contributor

@wbreza wbreza left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There are some lots of great ideas in here. Many of them overlap a lot with azd and the work the azd team are working on.

The azd team is working on multiple agentic scenarios and also building more MCP tools. Some of these tools would be a great fit to extend azd MCP tools themselves.

The azd team is also working with the Code to Cloud team who is transitioning to MCP tool development as well.

Lets see what opportunities we have for all of us to work together on this to create a great cohesive experience.

qianwens
qianwens commented Jul 22, 2025
View reviewed changes
@Menghua1 Menghua1 mentioned this pull request Aug 18, 2025
Closed
@g2vinay
Copy link
Member

g2vinay commented Aug 18, 2025

/azp run azure-mcp

@azure-pipelines
Copy link

azure-pipelines bot commented Aug 18, 2025

No pipelines are associated with this pull request.

@qianwens qianwens marked this pull request as ready for review August 19, 2025 02:24
@qianwens qianwens requested review from anuchandy and jongio August 19, 2025 02:26
joshfree
joshfree approved these changes Aug 19, 2025
View reviewed changes
Copy link
Member

@joshfree joshfree left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed final rebase

@g2vinay
Update sign-and-pack.yml
5bfc795 
add YamlDotNet.dll to allow list
@g2vinay g2vinay dismissed stale reviews from anuchandy and jongio August 19, 2025 05:29

dismissing to unblock merging

@g2vinay g2vinay requested a review from joshfree August 19, 2025 05:40
@g2vinay
Copy link
Member

g2vinay commented Aug 19, 2025

/check-enforcer override

@g2vinay
Copy link
Member

g2vinay commented Aug 19, 2025

/azp run azure-mcp

@azure-pipelines
Copy link

azure-pipelines bot commented Aug 19, 2025

No pipelines are associated with this pull request.

@g2vinay
Copy link
Member

g2vinay commented Aug 19, 2025

/azp run azure - mcp

@azure-pipelines
Copy link

azure-pipelines bot commented Aug 19, 2025

Azure Pipelines successfully started running 1 pipeline(s).

@g2vinay g2vinay enabled auto-merge (squash) August 19, 2025 05:50
@g2vinay g2vinay merged commit bdce198 into Azure:main Aug 19, 2025
54 checks passed
@github-project-automation github-project-automation bot moved this from In Progress to Done in Azure MCP Server (OLD) Aug 19, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

Copilot code review Copilot Copilot left review comments

@hallipr hallipr Awaiting requested review from hallipr hallipr is a code owner automatically assigned from Azure/azure-mcp

@conniey conniey Awaiting requested review from conniey conniey is a code owner automatically assigned from Azure/azure-mcp

@xiangyan99 xiangyan99 Awaiting requested review from xiangyan99 xiangyan99 is a code owner automatically assigned from Azure/azure-mcp

@sandeep-sen sandeep-sen Awaiting requested review from sandeep-sen sandeep-sen is a code owner automatically assigned from Azure/azure-mcp

@anuchandy anuchandy Awaiting requested review from anuchandy

@jongio jongio Awaiting requested review from jongio

@joshfree joshfree Awaiting requested review from joshfree

+6 more reviewers

@g2vinay g2vinay g2vinay left review comments

@vcolin7 vcolin7 vcolin7 left review comments

@wbreza wbreza wbreza left review comments

@kvenkatrajan kvenkatrajan kvenkatrajan left review comments

@wchigit wchigit wchigit left review comments

@xfz11 xfz11 xfz11 approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@jongio jongio

Labels

None yet

Projects

Azure MCP Server (OLD)
Status: Done

Milestone

2025-08

Development

Successfully merging this pull request may close these issues.