mcp icon indicating copy to clipboard operation
mcp copied to clipboard

Users/saars/add app insights list app traces

Open xiaomi7732 opened this issue 2 months ago • 4 comments

What does this PR do?

Add a tool to ApplicationInsights for query the trace.

This is the first piece, and simple query tool to query user's application insights trace data by running KQL queries.

GitHub issue number?

#671

Pre-merge Checklist

  • [x] Required for All PRs
    • [x] Read contribution guidelines
    • [x] PR title clearly describes the change
    • [x] Commit history is clean with descriptive messages (cleanup guide)
    • [x] Added comprehensive tests for new/modified functionality
    • [x] Updated servers/Azure.Mcp.Server/CHANGELOG.md and/or servers/Fabric.Mcp.Server/CHANGELOG.md for product changes (features, bug fixes, UI/UX, updated dependencies)
  • [x] For MCP tool changes:
    • [x] One tool per PR: This PR adds or modifies only one MCP tool for faster review cycles
    • [x] Updated servers/Azure.Mcp.Server/README.md and/or servers/Fabric.Mcp.Server/README.md documentation
    • [x] Updated command list in /docs/azmcp-commands.md and/or /docs/fabric-commands.md
    • [x] For new or modified tool descriptions, ran ToolDescriptionEvaluator and obtained a score of 0.4 or more and a top 3 ranking for all related test prompts
    • [x] For new tools associated with Azure services or publicly available tools/APIs/products, add URL to documentation in the PR description
  • [x] Extra steps for Azure MCP Server tool changes:
    • [x] Updated test prompts in /docs/e2eTestPrompts.md
    • [ ] 👉 For Community (non-Microsoft team member) PRs:
      • [ ] Security review: Reviewed code for security vulnerabilities, malicious code, or suspicious activities before running tests (crypto mining, spam, data exfiltration, etc.)
      • [ ] Manual tests run: added comment /azp run mcp - pullrequest - live to run Live Test Pipeline

xiaomi7732 avatar Oct 01 '25 23:10 xiaomi7732

@jongio do you mind give me another approval? Rebased to address the merge conflicts. Thanks!

xiaomi7732 avatar Oct 10 '25 17:10 xiaomi7732

@jongio / @hallipr Mind give it another approval, please.

xiaomi7732 avatar Oct 13 '25 21:10 xiaomi7732

@srnagar it looks like this missed the October release window. Please help ensure this gets merged for the November release on day-1

joshfree avatar Oct 14 '25 15:10 joshfree

Make sure to update consolidated-tools.json](https://github.com/microsoft/mcp/blob/main/core/Azure.Mcp.Core/src/Areas/Server/Resources/consolidated-tools.json). You can find instructions in https://github.com/microsoft/mcp/blob/main/CONTRIBUTING.md

fanyang-mono avatar Nov 14 '25 21:11 fanyang-mono