Add telemetry #18
Draft
Add telemetry #18
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
Author
|
This needs Azure#935 merged. |
* onboard code to cloud tools * fix source analysis errors * rename iac command (Azure#6) * rename iac command * revert changes * update * update deployment plan template * fix comments * fix the build error and refine the plan template * refine the template * add quota group (Azure#9) * add quota group * add unit test * update region checker return message * refactor * add unit test * update * revert * [GenerateArchitectureDiagramCommand] Fix and AKS support (Azure#8) * init aks diagram support * fix and test * fix * prompt fix encoding issue * fix by replacing + with - * fix encoding * fix test * resolve conflicts * cspell and format * fix the iac-rules-get tool name in e2etestprompt.md * Refactor: extract prompt to md file (Azure#10) * fix * extract prompt to md file * rename available-region-list * clean code (Azure#11) * clean code * update * add deploy command in md and fix analyze error * Some fix according to e2e test (Azure#12) * fix prompt for test * format * update * fix * update * update * add rules * Refactor deploy"a to project * fix test failure * Add sub command description to goup description * update description based on comment * fix live test (Azure#13) * fix live test * update quota test * update * update cspell (Azure#14) * force to save plan to file (Azure#15) * update plan command description * Add PR guidance document * Add test scenarios * aot safe refactoring and update template md (Azure#16) * aot safe refactoring * update template md * move app topology definition to resource file and update the command extension implementation * update test result - not completed * update test result * reorganize file folders (Azure#17) * reorganize file folders * update test structure * update pipeline template * format * remove comments * fix * update the command names in md file * Add code review report for PR Azure#626 on Deploy and Quota commands * fix the comments in code review report * remove code review doc * fix test failure * Add code review report for PR Azure#626 addressing deploy and quota commands - Summarized findings and compliance with architectural guidelines - Documented command structure, integration issues, and AOT safety - Provided targeted recommendations and next steps for improvements - Included exhaustive merge-readiness checklist and quick quality gates snapshot - Suggested documentation deltas and compliance matrix against final recommendations * Add manual testing plan and update documentation checklist for PR Azure#626 * update action plan and add invalid json test case * feat: Add comprehensive unit tests for quota commands (Azure#19) - Add edge case tests for whitespace-only resource types - Add tests for mixed casing in resource types with proper case preservation - Add tests for very long resource types lists (50+ items) - Enhance test coverage for AvailabilityListCommand and CheckCommand - Ensure proper validation and error handling for edge cases * lint (Azure#20) * Chentony/mermaid response refactor (Azure#21) * Fix diagram prompting * Small change * Reminder * Fix * Remove Mermaid Encode/Decode --------- Co-authored-by: Tony Chen (DevDiv) <[email protected]> * fix cspell errors * Mermaid generation: Fix </br> hallucination, fix copilotmd file target, fix extension installation reminder (Azure#22) * Fix diagram prompting * Small change * Reminder * Fix * Remove Mermaid Encode/Decode * Fix </br> hallucination, fix copilotmd file target, fix extension installation reminder --------- Co-authored-by: Tony Chen (DevDiv) <[email protected]> * feat: Add remaining work items for Deploy & Quota command areas * feat: Add test execution log section to Manual Testing Plan for Copilot prompts * Xf/updatecomm2 (Azure#23) * add logger * Hard-coded endpoints * update GetQuotaByUrlAsync * update Error handling * update doc * HTTP usage pattern * doc * format * format * add test * update * aot check * update document * update the remaining work * fix parser (Azure#24) * update remaining work * Adds document describing tool organization tasks * Updated organization task priorities * Add 32 character max for resource names (Azure#25) Co-authored-by: Tony Chen (DevDiv) <[email protected]> * fix build error and comments * remove image * Update Directory.Packages.props update ARM versions * Update cspell.json fix cspell json * Update sign-and-pack.yml add YamlDotNet.dll to allow list --------- Co-authored-by: xfz11 <[email protected]> Co-authored-by: wchigit <[email protected]> Co-authored-by: Vinay Gera <[email protected]> Co-authored-by: Jon Gallant <[email protected]> Co-authored-by: Tonychen0227 <[email protected]> Co-authored-by: Tony Chen (DevDiv) <[email protected]> Co-authored-by: Wallace Breza <[email protected]>
* Add azmcp-storage-blob-upload command * Update return of upload tool and update tests * Result uses full file path * etag -> eTag * Change tool name
* Stop showing stack trace as it is not helpful to LLM * update
* Add command to list repositories in Azure Container Registries and update related files * Refactor ACR image import logic to include resource module checks and enhance error handling * Enhance Import-AcrImageAz function to include TargetTag parameter and improve repository verification logic * Add support for listing repositories in Azure Container Registries via `azmcp-acr-registry-repository-list` command * Update areas/acr/src/AzureMcp.Acr/Services/AcrService.cs Co-authored-by: Copilot <[email protected]> * Fix exception handling in AddRepositoriesForRegistryAsync method to log warnings correctly * Update areas/acr/src/AzureMcp.Acr/Services/AcrService.cs Co-authored-by: Copilot <[email protected]> * Fix exception handling in AddRepositoriesForRegistryAsync method to log warnings correctly * Update CHANGELOG.md Co-authored-by: vcolin7 <[email protected]> * Update areas/acr/src/AzureMcp.Acr/Services/AcrService.cs Co-authored-by: Copilot <[email protected]> * Fix exception handling in AddRepositoriesForRegistryAsync method to log warnings correctly * Update areas/acr/src/AzureMcp.Acr/Services/AcrService.cs Co-authored-by: Copilot <[email protected]> * Fix exception handling in AddRepositoriesForRegistryAsync method to log warnings correctly * Add commands and prompts for listing container registry repositories * Add command to list repositories in Azure Container Registries and update related files * Update areas/acr/src/AzureMcp.Acr/Services/AcrService.cs Co-authored-by: Copilot <[email protected]> * Fix exception handling in AddRepositoriesForRegistryAsync method to log warnings correctly * Add command to list repositories in Azure Container Registries and update related files * Update areas/acr/src/AzureMcp.Acr/Services/AcrService.cs Co-authored-by: Copilot <[email protected]> * Fix exception handling in AddRepositoriesForRegistryAsync method to log warnings correctly * Add command to list repositories in Azure Container Registries and update related files * Refactor ACR image import logic to include resource module checks and enhance error handling * Update areas/acr/src/AzureMcp.Acr/Services/AcrService.cs Co-authored-by: Copilot <[email protected]> * Fix exception handling in AddRepositoriesForRegistryAsync method to log warnings correctly * Update areas/acr/src/AzureMcp.Acr/Services/AcrService.cs Co-authored-by: Copilot <[email protected]> * Fix exception handling in AddRepositoriesForRegistryAsync method to log warnings correctly * Update areas/acr/src/AzureMcp.Acr/Services/AcrService.cs Co-authored-by: Copilot <[email protected]> * Fix exception handling in AddRepositoriesForRegistryAsync method to log warnings correctly * Update areas/acr/src/AzureMcp.Acr/Services/AcrService.cs Co-authored-by: Copilot <[email protected]> * Fix exception handling in AddRepositoriesForRegistryAsync method to log warnings correctly * Update areas/acr/src/AzureMcp.Acr/Services/AcrService.cs Co-authored-by: Copilot <[email protected]> * Fix exception handling in AddRepositoriesForRegistryAsync method to log warnings correctly * Update areas/acr/src/AzureMcp.Acr/Services/AcrService.cs Co-authored-by: Copilot <[email protected]> * Fix exception handling in AddRepositoriesForRegistryAsync method to log warnings correctly * Took cspell through hooked on phonics * Cast a spell on cspell * Fix spelling dictionary entries in cspell configuration --------- Co-authored-by: Copilot <[email protected]> Co-authored-by: vcolin7 <[email protected]>
* Add server instructions * Add PR number * Remove unused assembly reference * Address copilot review feedback * Fix format * Use azure rules instead of the content of bestpractices tools * Remove unnecessary files * Remove unused library reference * Fix merge error
* Updated base CHANGELOG * Updated README to add missing commands * Updated VS Code CHANGELOG
* Increment package version after release * Update CHANGELOG.md --------- Co-authored-by: Arthur Ma <[email protected]>
…libc6:arm64 / libgcc-s1 conflicts during cross-compilation setup (Azure#1019) * Ensure version alignment of multi-arch GCC base libraries to prevent libc6:arm64 / libgcc-s1 conflicts during cross-compilation setup * enable script invocation from build.tml * regex comment
* Using ARG query to list & get Sql data * update * update * update doc * update * update * update * update * update * update * update * update * update * revert vscode/README.md * udpate * update * update changelog * update changelog * update checklist * Update the identify unused properteis check * update * fix typo * update changelog
…e#1015) * Standardize Storage descriptions and option and parameter names * Fix serialization * Fix tests * Add CHANGELOG entries
…Azure#1003) * Add support for Azure Managed Lustre List and Subnet size * Fix description of list command * Add documentation for AzureManagedLustre List / RequiredSubnetSize and additional unittests * Fix spell checks for Azure Managed Lustre area commands list and requiredsubnetsize * Exclude AzureManagedLustre from native builds because of StorageCache SDK dependencies * Fix Azure Managed Lustre area after review comments - Moving LustreFileSystem to record, adding missing JSONPropertyNames, removing not required GetError overrides. * Adjust typos in FileSystemListCommandTests.cs * Remove duplicated call in Program.cs of AzureManagedLustreSetup * Add NAT gateway for outbound connectivity in Azure Managed Lustre Live Tests * Add installation of Az.StorageCache module to the test-resources-post * Specify repository in install command of Az.StorageCache * Add Force to Az.StorageCache module install * Remove mcp.json file * Fix duplicated lines in sln and SQL mention in Azure Managed Lustre area
* Move to .NET 10 preview 7 to preview for .NET tool * Revert unnecessary change
* added cloud architect tool * updates * updates * updates * updates * update Tests * update documentation * update cspell * updates * added more prompt examples * removed unnecessary service classes * fixed formatting, added unit tests for proper escape handling * update cspell value * update description * updated unit test to use new description * add e2e prompt for cloud architect tool * cleaned up unused models * binded and used architectureTier and architectureDesignToolState options * added parameter validations and removed unnecessary classes * removed unnecessary base class and JsonPropertyNames * fixed CHANGELOG link formatting * removed unused json context * fixed spelling error * Include ResponseObject of parameters in response for design command * update description to be more concise to be under tool length limit * updated description to use multiline string * added validation * update tests * remove attribute
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
13 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.
What does this PR do?
Add telemetry.
GitHub issue number?
Pre-merge checklist
CHANGELOG.mdentry for user-impacting changes (bug fixes, new features, UI/UX changes).\eng\common\spelling\Invoke-Cspell.ps1README.md/docs/azmcp-commands.md/e2eTests/e2eTestPrompts.md/azp run azure - mcpto trigger pipeline