Add OpenTelemetry sample and update deps by torosent · Pull Request #637 · microsoft/durabletask-dotnet

torosent · 2026-02-07T03:20:29Z

This pull request updates several package dependencies in the Directory.Packages.props file to newer versions. The changes mainly focus on upgrading Microsoft.Extensions, gRPC/Protobuf, and other supporting libraries to improve compatibility, security, and stability.

Summary

What changed?

Dependency upgrades:

Microsoft.Extensions.* packages: Upgraded all related packages (such as Microsoft.Extensions.Caching.Memory, Microsoft.Extensions.DependencyInjection, Microsoft.Extensions.Hosting, etc.) from version 10.0.x to 10.0.2 for consistency and potential bug fixes.
gRPC and Protobuf packages: Updated Google.Protobuf to 3.33.5, Grpc.Net.Client to 2.76.0, Grpc.Tools to 2.78.0, and Grpc.AspNetCore.Server to 2.76.0, ensuring use of the latest stable versions.

Supporting library updates:

Microsoft.Bcl.AsyncInterfaces and System.Text.Json: Upgraded to 10.0.2 for improved async support and JSON serialization.

Testing and analysis packages:

Removed duplicate BenchmarkDotNet entry and ensured version 0.15.8 is used.…ing sample

Updated Google.Protobuf package version from 3.33.2 to 3.33.5 in AzureFunctionsApp and Worker.Grpc projects.
Removed outdated package references in Directory.Packages.props for Functions sample.
Added OpenTelemetry packages for distributed tracing in Directory.Packages.props.
Introduced a new DistributedTracingSample project demonstrating OpenTelemetry integration with Durable Task SDK.
Implemented a fan-out/fan-in orchestration pattern in the DistributedTracingSample.
Created a README.md for the DistributedTracingSample with setup instructions and usage details.
Added a docker-compose.yml for running Jaeger and the DTS emulator for testing the sample.

Why is this change needed?

Adding sample

Issues / work items

Resolves #
Related #

Project checklist

Release notes are not required for the next release
- Otherwise: Notes added to release_notes.md
Backport is not required
- Otherwise: Backport tracked by issue/PR #issue_or_pr
All required tests have been added/updated (unit tests, E2E tests)
Breaking change?
- If yes:
  - Impact:
  - Migration guidance:

AI-assisted code disclosure (required)

Was an AI tool used? (select one)

No
Yes, AI helped write parts of this PR (e.g., GitHub Copilot)
Yes, an AI agent generated most of this PR

If AI was used:

Tool(s): Copilot CLI - Opus 4.6
AI-assisted areas/files: All

AI verification (required if AI was used):

I understand the code and can explain it
I verified referenced APIs/types exist and are correct
I reviewed edge cases/failure paths (timeouts, retries, cancellation, exceptions)
I reviewed concurrency/async behavior
I checked for unintended breaking or behavior changes

Testing

Automated tests

Result: Passed

Manual validation (only if runtime/behavior changed)

Environment (OS, .NET version, components): OSX .NET 10
Evidence (optional):

Notes for reviewers

N/A

Copilot

Pull request overview

This PR updates centrally-managed dependencies (Microsoft.Extensions, gRPC/Protobuf, System.Text.Json, etc.) and adds a new sample demonstrating OpenTelemetry distributed tracing with the Durable Task SDK, including local infra via Docker Compose.

Changes:

Bumped multiple central package versions (Microsoft.Extensions.*, gRPC/Protobuf, System.Text.Json, AsyncInterfaces) and removed a duplicate package entry.
Added a new DistributedTracingSample demonstrating fan-out/fan-in orchestration tracing with OpenTelemetry + Jaeger.
Updated solution/projects to include the new sample and align Protobuf versions.

Reviewed changes

Copilot reviewed 10 out of 10 changed files in this pull request and generated 7 comments.

Show a summary per file

File	Description
`Directory.Packages.props`	Updates central dependency versions (Extensions, gRPC/Protobuf, System.*).
`src/Worker/Grpc/Worker.Grpc.csproj`	Updates Google.Protobuf version override.
`src/InProcessTestHost/InProcessTestHost.csproj`	Adds conditional Grpc.AspNetCore.Server version pin for net6.0.
`samples/Directory.Packages.props`	Adds OpenTelemetry package versions; removes old Functions sample package entries (leaving an empty group).
`samples/AzureFunctionsApp/AzureFunctionsApp.csproj`	Updates Google.Protobuf version override.
`samples/DistributedTracingSample/DistributedTracingSample.csproj`	Introduces new sample project (net10.0) with OTel package references and p2p refs.
`samples/DistributedTracingSample/Program.cs`	Implements OTel wiring and a fan-out/fan-in orchestration + activities.
`samples/DistributedTracingSample/README.md`	Documents how to run the tracing sample with Jaeger + DTS emulator.
`samples/DistributedTracingSample/docker-compose.yml`	Adds Jaeger + DTS emulator compose services for local testing.
`Microsoft.DurableTask.sln`	Adds the new sample project and introduces additional solution configurations (x64/x86).

samples/DistributedTracingSample/Program.cs

samples/DistributedTracingSample/docker-compose.yml

samples/Directory.Packages.props

Microsoft.DurableTask.sln

src/InProcessTestHost/InProcessTestHost.csproj

samples/DistributedTracingSample/README.md

Copilot

Pull request overview

Copilot reviewed 10 out of 10 changed files in this pull request and generated 2 comments.

samples/DistributedTracingSample/Program.cs

…ing sample - Updated Google.Protobuf package version from 3.33.2 to 3.33.5 in AzureFunctionsApp and Worker.Grpc projects. - Removed outdated package references in Directory.Packages.props for Functions sample. - Added OpenTelemetry packages for distributed tracing in Directory.Packages.props. - Introduced a new DistributedTracingSample project demonstrating OpenTelemetry integration with Durable Task SDK. - Implemented a fan-out/fan-in orchestration pattern in the DistributedTracingSample. - Created a README.md for the DistributedTracingSample with setup instructions and usage details. - Added a docker-compose.yml for running Jaeger and the DTS emulator for testing the sample.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

torosent · 2026-02-27T19:52:35Z

@copilot open a new pull request to apply changes based on the comments in this thread

Copilot · 2026-02-27T19:52:42Z

@torosent I've opened a new pull request, #649, to work on those changes. Once the pull request is ready, I'll request review from you.

#649)

Copilot · 2026-02-27T19:59:18Z

@torosent I've opened a new pull request, #650, to work on those changes. Once the pull request is ready, I'll request review from you.

Copilot

Pull request overview

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

samples/DistributedTracingSample/DistributedTracingSample.csproj

samples/DistributedTracingSample/Program.cs

samples/DistributedTracingSample/docker-compose.yml

Microsoft.DurableTask.sln

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Copilot

Pull request overview

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

Copilot · 2026-02-27T20:10:54Z

samples/AzureFunctionsApp/AzureFunctionsApp.csproj


  <ItemGroup>
-    <PackageReference Include="Google.Protobuf" VersionOverride="3.33.2" />
+    <PackageReference Include="Google.Protobuf" VersionOverride="3.33.5" />


Google.Protobuf is centrally versioned (and already updated to 3.33.5). This VersionOverride matches the central version, so it's redundant—consider removing it and letting central package management control the version.

Suggested change

<PackageReference Include="Google.Protobuf" VersionOverride="3.33.5" />

<PackageReference Include="Google.Protobuf" />

Copilot · 2026-02-27T20:10:54Z

samples/DistributedTracingSample/README.md

+The sample connects to the DTS emulator by default using the connection string:
+```
+Endpoint=http://localhost:8080;Authentication=None;TaskHub=default
+```
+
+To override, set the `DURABLE_TASK_SCHEDULER_CONNECTION_STRING` environment variable:


This section states the sample connects to the DTS emulator by default using a localhost connection string, but the current Program.cs requires DURABLE_TASK_SCHEDULER_CONNECTION_STRING to be set and will fail otherwise. Please align the instructions with the actual startup behavior (either document the requirement or change the code to have a default).

Suggested change

The sample connects to the DTS emulator by default using the connection string:

```

Endpoint=http://localhost:8080;Authentication=None;TaskHub=default

```

To override, set the `DURABLE_TASK_SCHEDULER_CONNECTION_STRING` environment variable:

Before running, configure the DTS emulator connection string using the `DURABLE_TASK_SCHEDULER_CONNECTION_STRING` environment variable. For the local emulator started via Docker Compose, use:

Copilot · 2026-02-27T20:10:54Z

src/Worker/Grpc/Worker.Grpc.csproj


  <ItemGroup>
-    <PackageReference Include="Google.Protobuf" VersionOverride="3.33.2" />
+    <PackageReference Include="Google.Protobuf" VersionOverride="3.33.5" />


Google.Protobuf is already centrally versioned in Directory.Packages.props (currently 3.33.5). Since this VersionOverride matches the central version, it's redundant and adds maintenance overhead—consider removing the VersionOverride and relying on central package management.

Suggested change

<PackageReference Include="Google.Protobuf" VersionOverride="3.33.5" />

<PackageReference Include="Google.Protobuf" />

Copilot AI review requested due to automatic review settings February 7, 2026 03:20

Copilot started reviewing on behalf of torosent February 7, 2026 03:20 View session

Copilot AI reviewed Feb 7, 2026

View reviewed changes

Copilot AI review requested due to automatic review settings February 7, 2026 03:40

Copilot started reviewing on behalf of torosent February 7, 2026 03:40 View session

Copilot AI reviewed Feb 7, 2026

View reviewed changes

samples/DistributedTracingSample/Program.cs Outdated Show resolved Hide resolved

samples/DistributedTracingSample/Program.cs Show resolved Hide resolved

torosent and others added 4 commits February 27, 2026 11:52

Update samples/DistributedTracingSample/README.md

ff03198

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Update Microsoft.DurableTask.sln

d61afbf

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Update samples/DistributedTracingSample/Program.cs

55f978f

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

torosent force-pushed the torosent/add-otel-sample branch from 4c8c470 to 55f978f Compare February 27, 2026 19:52

Copilot AI mentioned this pull request Feb 27, 2026

Fix int.MinValue overflow and CTS disposal in DistributedTracingSample #649

Merged

Fix int.MinValue overflow and CTS disposal in DistributedTracingSample (

0644db7

#649)

Copilot AI review requested due to automatic review settings February 27, 2026 19:58

Copilot started reviewing on behalf of torosent February 27, 2026 19:58 View session

Copilot AI mentioned this pull request Feb 27, 2026

Dispose CancellationTokenSource in DistributedTracingSample #650

Closed

Copilot AI reviewed Feb 27, 2026

View reviewed changes

Update samples/DistributedTracingSample/Program.cs

1c0fb34

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Copilot AI review requested due to automatic review settings February 27, 2026 20:04

Update Microsoft.DurableTask.sln

268f4f6

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Copilot started reviewing on behalf of torosent February 27, 2026 20:05 View session

Merge branch 'main' into torosent/add-otel-sample

70c80e1

Copilot AI reviewed Feb 27, 2026

View reviewed changes

YunchuWang approved these changes Feb 27, 2026

View reviewed changes

torosent merged commit 838ca21 into main Feb 27, 2026
7 checks passed

torosent deleted the torosent/add-otel-sample branch February 27, 2026 20:13

	<PackageReference Include="Google.Protobuf" VersionOverride="3.33.5" />
	<PackageReference Include="Google.Protobuf" />

Conversation

torosent commented Feb 7, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

What changed?

Why is this change needed?

Issues / work items

Project checklist

AI-assisted code disclosure (required)

Was an AI tool used? (select one)

Testing

Automated tests

Manual validation (only if runtime/behavior changed)

Notes for reviewers

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

torosent commented Feb 27, 2026

Uh oh!

Copilot AI commented Feb 27, 2026

Uh oh!

Copilot AI commented Feb 27, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Copilot AI Feb 27, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 27, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 27, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

torosent commented Feb 7, 2026 •

edited

Loading