Skip to content

.NET: updates open telemetry deps - #8253

Merged
Vincent Biret (baywet) merged 3 commits into
mainfrom
deps/ot
Sep 10, 2026
Merged

.NET: updates open telemetry deps#8253
Vincent Biret (baywet) merged 3 commits into
mainfrom
deps/ot

Conversation

@baywet

@baywet Vincent Biret (baywet) commented Sep 10, 2026

Copy link
Copy Markdown
Member

upgrades open telemetry dependencies in the hope of unblocking dependabot

@baywet
chore: updates open telemetry dependencies
04bca46 
chore: updates trimmer dependency
chore: updates copilot sdk dependency

Signed-off-by: Vincent Biret <vibiret@microsoft.com>
@baywet
ci: adds an open telemetry group
19b8ff9 
Signed-off-by: Vincent Biret <vibiret@microsoft.com>
Copilot AI balanced review requested due to automatic review settings September 10, 2026 15:03
@agent-framework-automation agent-framework-automation Bot added the .NET Usage: [Issues, PRs], Target: .Net label Sep 10, 2026
@github-actions github-actions Bot changed the title deps/ot .NET: deps/ot Sep 10, 2026

This comment was marked as outdated.

Sign in to view

@github-code-quality

github-code-quality Bot commented Sep 10, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Code Coverage Overview

Languages: C#

C# / code-coverage/dotnet

The overall line coverage in commit 92744bb in the deps/ot branch is 84%. Line coverage data for the main branch is not yet available.

Show a line coverage summary of the most covered files.
File main deps/ot 92744bb +/-
/home/runner/wo...valConverter.cs — 100% —
/home/runner/wo...entsProvider.cs — 99% —
/home/runner/wo...nticAnalyzer.cs — 94% —
/home/runner/wo...tClientAgent.cs — 91% —
/home/runner/wo...putConverter.cs — 90% —
/home/runner/wo...kflowBuilder.cs — 90% —
/home/runner/wo...kflowSession.cs — 87% —
/home/runner/wo...SkillsSource.cs — 86% —
/home/runner/wo...onExtensions.cs — 81% —
/home/runner/wo...ctionVisitor.cs — 70% —

Updated September 10, 2026 16:50 UTC

github-actions[bot]

This comment was marked as outdated.

Sign in to view

@baywet
Update explicit OpenTelemetry sample pins
92744bb 
Co-authored-by: baywet <7905502+baywet@users.noreply.github.com>
@baywet
Vincent Biret (baywet) requested a balanced review from Copilot September 10, 2026 16:37
Copilot AI reviewed Sep 10, 2026
View reviewed changes

Copilot AI 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.

🟢 Approval recommended

The dependency/version updates are consistent across central management and the known explicitly pinned sample projects, and the Dependabot grouping change is valid and scoped.

Review details
  • Files reviewed: 4/4 changed files
  • Comments generated: 0 new
  • Review effort level: Lite

@baywet
Vincent Biret (baywet) marked this pull request as ready for review September 10, 2026 16:51
github-actions[bot]
github-actions Bot reviewed Sep 10, 2026
View reviewed changes

@github-actions github-actions Bot 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.

MAF Automated Review — Iteration 2

Result: No findings
Scope: 1 net-new commit(s): 92744bbbffb4
Model: gpt-5.6-sol-fast

Overview

This incremental update aligns the explicit OpenTelemetry pins in two self-contained sample projects with the repository's central 1.18.0 versions. Both projects deliberately bypass central package management, remain constrained to .NET 10, and are included in the scheduled sample solution build. No concrete Critical, High, or Medium defect was established in the changed behavior.

Reviewed the supplied incremental change set across correctness, security/reliability, architecture, and failure behavior.
No publishable findings remained after source verification for this scope.

@baywet Vincent Biret (baywet) changed the title .NET: deps/ot .NET: updates open telemetry deps Sep 10, 2026
@baywet
Vincent Biret (baywet) added this pull request to the merge queue Sep 10, 2026
Merged via the queue into main with commit b9dbe4f Sep 10, 2026
38 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@github-actions github-actions[bot] github-actions[bot] left review comments
Copilot code review Copilot
Copilot review effort, defaults to Balanced
Applies to this pull request for everyone.Learn more about Copilot code review.
Copilot left review comments
Copilot code review Copilot Copilot left review comments
@SergeyMenshykh SergeyMenshykh SergeyMenshykh approved these changes
@westey-m westey (westey-m) westey-m approved these changes
@chetantoshniwal chetantoshniwal Awaiting requested review from chetantoshniwal chetantoshniwal is a code owner
@peibekwe Peter Ibekwe (peibekwe) Awaiting requested review from peibekwe peibekwe is a code owner
@rogerbarreto Roger Barreto (rogerbarreto) Awaiting requested review from rogerbarreto rogerbarreto is a code owner
@eavanvalkenburg Eduard van Valkenburg (eavanvalkenburg) Awaiting requested review from eavanvalkenburg eavanvalkenburg is a code owner
@giles17 Giles Odigwe (giles17) Awaiting requested review from giles17 giles17 is a code owner
@moonbox3 Evan Mattson (moonbox3) Awaiting requested review from moonbox3 moonbox3 is a code owner
@TaoChenOSU Tao Chen (TaoChenOSU) Awaiting requested review from TaoChenOSU TaoChenOSU is a code owner
@jpalvarezl Jose Alvarez (jpalvarezl) Awaiting requested review from jpalvarezl jpalvarezl is a code owner

Assignees

@baywet Vincent Biret (baywet)

Labels

.NET Usage: [Issues, PRs], Target: .Net

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@baywet @SergeyMenshykh @westey-m