This repository was archived by the owner on Sep 4, 2025. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 357
[Deploy] Onboard code to cloud tools #626
Merged
Merged
Changes from 4 commits
Commits
Show all changes
68 commits
Select commit
Hold shift + click to select a range
e1e3c8b
onboard code to cloud tools
qianwens a9215a5
fix source analysis errors
qianwens 56a2c39
rename iac command (#6)
xfz11 60df1dc
update deployment plan template
xfz11 31f564b
Merge branch 'main' of https://github.com/Azure/azure-mcp into qianwe…
qianwens 9ad1104
Merge branch 'qianwen/deploy' of https://github.com/qianwens/azure-mc…
qianwens 3043492
fix comments
qianwens 0d0fd57
Merge branch 'main' of https://github.com/Azure/azure-mcp into qianwe…
qianwens efd49fd
fix the build error and refine the plan template
qianwens 5132545
refine the template
qianwens b4884c6
add quota group (#9)
xfz11 969c18e
[GenerateArchitectureDiagramCommand] Fix and AKS support (#8)
wchigit 23aa942
fix the iac-rules-get tool name in e2etestprompt.md
qianwens a144fe8
Refactor: extract prompt to md file (#10)
xfz11 458cec4
clean code (#11)
xfz11 52853c5
add deploy command in md and fix analyze error
qianwens 8dda05b
Some fix according to e2e test (#12)
xfz11 daaaaf0
Merge branch 'main' of https://github.com/Azure/azure-mcp into qianwe…
qianwens d8e05ff
Refactor deploy"a to project
qianwens 413e18c
fix test failure
qianwens b7a04d8
Add sub command description to goup description
qianwens 45277a3
Merge branch 'main' into qianwen/deploy
g2vinay 85a14e8
update description based on comment
qianwens 985757f
fix live test (#13)
xfz11 62b66fc
update cspell (#14)
xfz11 05c498f
Merge branch 'main' of https://github.com/Azure/azure-mcp into qianwe…
qianwens c1e5f7c
force to save plan to file (#15)
xfz11 a818e5c
Merge branch 'main' of https://github.com/Azure/azure-mcp into qianwe…
qianwens b6780fd
update plan command description
qianwens 0215c92
Add PR guidance document
jongio 8cc169c
Add test scenarios
jongio 7516979
aot safe refactoring and update template md (#16)
xfz11 eac6693
move app topology definition to resource file and update the command …
qianwens fe1f8e2
update test result - not completed
qianwens 5b1ef33
update test result
qianwens 83ddd29
reorganize file folders (#17)
xfz11 a7c9190
update the command names in md file
qianwens 3f5030c
Merge branch 'main' of https://github.com/Azure/azure-mcp into qianwe…
qianwens d675ae0
Add code review report for PR #626 on Deploy and Quota commands
jongio 0d2b738
fix the comments in code review report
qianwens b8e2afd
Merge branch 'main' of https://github.com/Azure/azure-mcp into qianwe…
qianwens 0083cc6
remove code review doc
qianwens f3cd8a6
fix test failure
qianwens 4c69f5a
Add code review report for PR #626 addressing deploy and quota commands
jongio c1a2e05
Add manual testing plan and update documentation checklist for PR #626
jongio d1077f4
update action plan and add invalid json test case
qianwens a0667a4
feat: Add comprehensive unit tests for quota commands (#19)
xfz11 2066752
lint (#20)
xfz11 04759ed
Chentony/mermaid response refactor (#21)
Tonychen0227 4c76942
fix cspell errors
qianwens 9e42001
Mermaid generation: Fix </br> hallucination, fix copilotmd file targe…
Tonychen0227 3f49516
feat: Add remaining work items for Deploy & Quota command areas
jongio 7e5748b
feat: Add test execution log section to Manual Testing Plan for Copil…
jongio 113c657
Xf/updatecomm2 (#23)
xfz11 1eeee74
update the remaining work
qianwens 916181b
fix parser (#24)
wchigit 945c17b
update remaining work
qianwens 768c480
Adds document describing tool organization tasks
wbreza 902ac6e
Updated organization task priorities
wbreza c622064
Add 32 character max for resource names (#25)
Tonychen0227 b18b209
Merge branch 'qianwen/deploy' of https://github.com/qianwens/azure-mc…
qianwens 2f6cb70
fix build error and comments
qianwens a0fa189
remove image
qianwens 6e8ef4b
Merge branch 'main' into qianwen/deploy
jongio 6c9edd0
Update Directory.Packages.props
g2vinay 43b70f7
Update cspell.json
g2vinay 58104c0
Merge branch 'main' of https://github.com/Azure/azure-mcp into qianwe…
qianwens 5bfc795
Update sign-and-pack.yml
g2vinay File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
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
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
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
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
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,81 @@ | ||
| // Copyright (c) Microsoft Corporation. | ||
| // Licensed under the MIT License. | ||
|
|
||
| using AzureMcp.Areas.Deploy.Options; | ||
| using AzureMcp.Areas.Deploy.Services; | ||
| using AzureMcp.Commands.Subscription; | ||
| using AzureMcp.Services.Telemetry; | ||
| using Microsoft.Extensions.Logging; | ||
|
|
||
| namespace AzureMcp.Areas.Deploy.Commands; | ||
|
|
||
| public sealed class AzdAppLogGetCommand(ILogger<AzdAppLogGetCommand> logger) : SubscriptionCommand<AzdAppLogOptions>() | ||
qianwens marked this conversation as resolved.
Outdated
Show resolved
Hide resolved
|
||
| { | ||
| private const string CommandTitle = "Get AZD deployed App Logs"; | ||
| private readonly ILogger<AzdAppLogGetCommand> _logger = logger; | ||
|
|
||
| private readonly Option<string> _workspaceFolderOption = DeployOptionDefinitions.AzdAppLogOptions.WorkspaceFolder; | ||
| private readonly Option<string> _azdEnvNameOption = DeployOptionDefinitions.AzdAppLogOptions.AzdEnvName; | ||
| private readonly Option<int> _limitOption = DeployOptionDefinitions.AzdAppLogOptions.Limit; | ||
|
|
||
| public override string Name => "azd-app-log-get"; | ||
|
|
||
| public override string Description => | ||
| """ | ||
| This tool helps fetch logs from log analytics workspace for Container Apps, App Services, function apps that were deployed through azd. Invoke this tool directly after a successful `azd up` or when user prompts to check the app's status or provide errors in the deployed apps. | ||
qianwens marked this conversation as resolved.
Outdated
Show resolved
Hide resolved
|
||
| """; | ||
|
|
||
| public override string Title => CommandTitle; | ||
|
|
||
| protected override void RegisterOptions(Command command) | ||
| { | ||
| base.RegisterOptions(command); | ||
| command.AddOption(_workspaceFolderOption); | ||
| command.AddOption(_azdEnvNameOption); | ||
| command.AddOption(_limitOption); | ||
| } | ||
|
|
||
| protected override AzdAppLogOptions BindOptions(ParseResult parseResult) | ||
| { | ||
| var options = base.BindOptions(parseResult); | ||
| options.WorkspaceFolder = parseResult.GetValueForOption(_workspaceFolderOption)!; | ||
| options.AzdEnvName = parseResult.GetValueForOption(_azdEnvNameOption)!; | ||
| options.Limit = parseResult.GetValueForOption(_limitOption); | ||
| return options; | ||
| } | ||
|
|
||
| [McpServerTool(Destructive = false, ReadOnly = true, Title = CommandTitle)] | ||
| public override async Task<CommandResponse> ExecuteAsync(CommandContext context, ParseResult parseResult) | ||
| { | ||
| var options = BindOptions(parseResult); | ||
|
|
||
| try | ||
| { | ||
| if (!Validate(parseResult.CommandResult, context.Response).IsValid) | ||
| { | ||
| return context.Response; | ||
| } | ||
|
|
||
| context.Activity?.WithSubscriptionTag(options); | ||
|
|
||
| // Parse optional date parameters | ||
qianwens marked this conversation as resolved.
Outdated
Show resolved
Hide resolved
|
||
|
|
||
| var deployService = context.GetService<IDeployService>(); | ||
| string result = await deployService.GetAzdResourceLogsAsync( | ||
| options.WorkspaceFolder!, | ||
| options.AzdEnvName!, | ||
| options.Subscription!, | ||
| options.Limit); | ||
|
|
||
| context.Response.Message = result; | ||
| } | ||
| catch (Exception ex) | ||
| { | ||
| _logger.LogError(ex, "An exception occurred getting azd app logs."); | ||
| HandleException(context, ex); | ||
| } | ||
|
|
||
| return context.Response; | ||
| } | ||
|
|
||
| } | ||
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,40 @@ | ||
| // Copyright (c) Microsoft Corporation. | ||
| // Licensed under the MIT License. | ||
|
|
||
| namespace AzureMcp.Areas.Deploy.Commands; | ||
|
|
||
| public static class AzureServiceConstants | ||
| { | ||
| public enum AzureComputeServiceType | ||
| { | ||
| AppService, | ||
| FunctionApp, | ||
| ContainerApp, | ||
| StaticWebApp | ||
| } | ||
|
|
||
| public enum AzureServiceType | ||
| { | ||
| AzureAISearch, | ||
| AzureAIServices, | ||
| AppService, | ||
| AzureApplicationInsights, | ||
| AzureBotService, | ||
| AzureContainerApp, | ||
| AzureCosmosDB, | ||
| AzureFunctionApp, | ||
| AzureKeyVault, | ||
| AzureDatabaseForMySQL, | ||
| AzureOpenAI, | ||
| AzureDatabaseForPostgreSQL, | ||
| AzurePrivateEndpoint, | ||
| AzureRedisCache, | ||
| AzureSQLDatabase, | ||
| AzureStorageAccount, | ||
| StaticWebApp, | ||
| AzureServiceBus, | ||
| AzureSignalRService, | ||
| AzureVirtualNetwork, | ||
| AzureWebPubSub | ||
| } | ||
| } |
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,28 @@ | ||
| // Copyright (c) Microsoft Corporation. | ||
| // Licensed under the MIT License. | ||
|
|
||
| using System.Text.Json.Serialization; | ||
| using Areas.Deploy.Services.Util; | ||
| using AzureMcp.Areas.Deploy.Commands.Quota; | ||
| using AzureMcp.Areas.Deploy.Commands.Region; | ||
| using AzureMcp.Areas.Deploy.Models; | ||
| using AzureMcp.Areas.Deploy.Options; | ||
|
|
||
| namespace AzureMcp.Areas.Deploy.Commands; | ||
|
|
||
| [JsonSourceGenerationOptions( | ||
| PropertyNamingPolicy = JsonKnownNamingPolicy.CamelCase, | ||
| PropertyNameCaseInsensitive = true, | ||
| DefaultIgnoreCondition = JsonIgnoreCondition.WhenWritingNull | ||
| )] | ||
| [JsonSerializable(typeof(AppTopology))] | ||
| [JsonSerializable(typeof(MermaidData))] | ||
| [JsonSerializable(typeof(MermaidConfig))] | ||
| [JsonSerializable(typeof(QuotaCheckCommand.QuotaCheckCommandResult))] | ||
| [JsonSerializable(typeof(QuotaInfo))] | ||
| [JsonSerializable(typeof(Dictionary<string, List<QuotaInfo>>))] | ||
| [JsonSerializable(typeof(RegionCheckCommand.RegionCheckCommandResult))] | ||
| [JsonSerializable(typeof(List<string>))] | ||
| internal sealed partial class DeployJsonContext : JsonSerializerContext | ||
| { | ||
| } |
Oops, something went wrong.
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.
Uh oh!
There was an error while loading. Please reload this page.