Skip to content

Conversation

@harsimar
Copy link
Member

@harsimar harsimar commented Jan 6, 2026

This adds documentation for how to set azure monitor exporter options from the base and web modules.

See: #3058 & #3062

Copilot AI review requested due to automatic review settings January 6, 2026 01:01
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adds comprehensive documentation for Azure Monitor exporter options that can be configured through TelemetryConfiguration in the BASE and WEB modules. The changes provide developers with clear guidance on sampling, offline storage, and feature configuration options.

Key Changes

  • Added detailed Azure Monitor Exporter Options documentation table in BASE/README.md with property descriptions and defaults
  • Updated WEB/Src/Web/README.md to reference the new BASE documentation
  • Replaced minimal AAD authentication example in Global.asax.cs with comprehensive configuration example showing all exporter options

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 4 comments.

File Description
BASE/README.md Added new "Azure Monitor Exporter Options" section with configuration table, usage examples, and sampling guidance
WEB/Src/Web/README.md Added cross-reference to BASE documentation for Azure Monitor Exporter Options
examples/ClassicAspNetWebApp/Global.asax.cs Expanded example code to demonstrate all Azure Monitor exporter configuration options

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Co-authored-by: Rajkumar Rangaraj <rajrang@microsoft.com>
@harsimar harsimar merged commit b9776ed into main Jan 6, 2026
22 checks passed
@harsimar harsimar deleted the harskaur/webExporterOptions branch January 6, 2026 19:42
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.

3 participants