Skip to content

Integration branch for PR #5710: Expand OTel fallback exception handling#5718

Closed
Copilot wants to merge 2 commits intomainfrom
copilot/integratepr-5710-otel-fallback
Closed

Integration branch for PR #5710: Expand OTel fallback exception handling#5718
Copilot wants to merge 2 commits intomainfrom
copilot/integratepr-5710-otel-fallback

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Feb 5, 2026

Changes proposed in this request

Integration branch for external PR #5710 to enable CI testing. Expands exception handling when OpenTelemetry instrumentation fails to load.

Code changes:

  • AbstractPlatformProxy.cs: Catch TypeLoadException and TypeInitializationException in addition to FileNotFoundException when instantiating OTel instrumentation
  • Uses pattern matching with when clause for cleaner exception filtering

Context:
Observed in Azure Functions environments with older System.Diagnostics.DiagnosticSource versions. Original issue (#4456) identified FileNotFoundException, but contributor @fowl2 encountered TypeLoadException in production.

Testing

No new tests added. Change expands existing fallback behavior to handle additional exception types in the same code path.

Performance impact

None. Exception handling only triggered when OTel dependencies unavailable (existing failure path).

Documentation

  • All relevant documentation is updated.

Reference: #5710
Credit: Original contribution by @fowl2

Original prompt

Create an integration branch off main to allow CI tests to run for PR #5710 (external fork), and adjust the workflow so the changes can be merged safely.

Context:

Tasks:

  1. Create a new branch from main named something like integrate/pr-5710-otel-fallback.
  2. Bring the commits/changes from PR Fix fallback where OpenTelemetry instrumentation is not available #5710 onto this new branch (prefer merge PR into integration branch, or cherry-pick commits if required).
  3. Open a new PR (or retarget existing PR if appropriate) such that the PR head is the integration branch in the main repo and base is main, ensuring CI runs.
  4. Ensure the PR description references the original PR Fix fallback where OpenTelemetry instrumentation is not available #5710 and explains why the integration branch exists.
  5. Confirm the new PR triggers the expected CI checks and that it is merge-ready.

Acceptance criteria:

This pull request was created from Copilot chat.


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

…dling

This integrates the changes from external PR #5710 by @fowl2 which expands
exception handling for OpenTelemetry initialization to catch TypeLoadException
and TypeInitializationException in addition to FileNotFoundException.

Reference: #5710

Co-authored-by: neha-bhargava <61847233+neha-bhargava@users.noreply.github.com>
Copilot AI changed the title [WIP] Create integration branch for CI testing of PR #5710 Integration branch for PR #5710: Expand OTel fallback exception handling Feb 5, 2026
Copilot AI requested a review from neha-bhargava February 5, 2026 21:37
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.

2 participants