Skip to content

Add Application Insights telemetry example - #1778

Merged
jeffhandley merged 1 commit into
modelcontextprotocol:mainfrom
lntutor:docs/application-insights-1026
Aug 4, 2026
Merged

Add Application Insights telemetry example#1778
jeffhandley merged 1 commit into
modelcontextprotocol:mainfrom
lntutor:docs/application-insights-1026

Conversation

@lntutor

@lntutor lntutor commented Jul 30, 2026

Copy link
Copy Markdown
Contributor

Summary

  • add the Azure Monitor OpenTelemetry distro to the compiled ASP.NET Core MCP server sample
  • select Application Insights when APPLICATIONINSIGHTS_CONNECTION_STRING is configured while retaining OTLP as the default
  • document setup, exported signals, MCP attributes, and a sample Application Insights query

Closes #1026

Test plan

  • dotnet build samples/AspNetCoreMcpServer/AspNetCoreMcpServer.csproj --configuration Debug --no-incremental
  • dotnet build samples/AspNetCoreMcpServer/AspNetCoreMcpServer.csproj --configuration Release --no-incremental
  • OTLP-default startup smoke test and MCP endpoint request
  • Azure Monitor startup smoke test with a non-secret placeholder connection string
  • dotnet build ModelContextProtocol.slnx --configuration Debug --no-incremental
  • dotnet test ModelContextProtocol.slnx --configuration Debug --no-build --no-restore --filter "(Execution!=Manual)" (8,862 passed)
  • focused dotnet format verification for the modified C# file; the repository-wide formatter currently reports 74 pre-existing files on main

@lntutor

lntutor commented Aug 1, 2026

Copy link
Copy Markdown
Contributor Author

Current head 780b835370116b01cebce0cc3c592005f8e4f567 is mergeable and the visible CI surface on this exact SHA is fully green: markdown-link-check, CodeQL (Analyze (actions) and Analyze (csharp)), all Debug/Release build-and-test jobs on Ubuntu, macOS, and Windows, publish-coverage, and the aggregate CodeQL check all completed successfully. There are no review threads or review-body comments open on the current PR state, so if this Application Insights telemetry example still matches the intended direction, it looks ready for maintainer review/merge when convenient.

@jeffhandley jeffhandley left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Great addition here, @lntutor; thanks!

@jeffhandley
jeffhandley merged commit f5d54b5 into modelcontextprotocol:main Aug 4, 2026
11 checks passed
@jeffhandley jeffhandley mentioned this pull request Aug 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Provide an example of how to connect telemetry with Application Insights

2 participants