Skip to content

feat(samples): add Agent365 S2S observability demo - #244

Draft
Nikhil Navakiran (nikhilNava) wants to merge 5 commits into
mainfrom
feature/agent365-s2s-sample
Draft

feat(samples): add Agent365 S2S observability demo#244
Nikhil Navakiran (nikhilNava) wants to merge 5 commits into
mainfrom
feature/agent365-s2s-sample

Conversation

@nikhilNava

@nikhilNava Nikhil Navakiran (nikhilNava) commented Sep 15, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • add a self-contained Agent365 S2S sample with two-stage MSAL app-only authentication
  • add expiry-aware, single-flight token caching and secret-safe diagnostics
  • demonstrate InvokeAgent, Inference, and ExecuteTool manual spans with S2S export
  • log successful exporter HTTP status and correlation ID safely

Validation

  • root build, format, lint (warnings only)
  • 1094 root unit tests, 31 functional tests, 1 integration test
  • ESM import and TypeDoc (existing warnings only)
  • sample format, lint, build, and 35 tests
  • exporter 130 tests and SDKStats 7 tests
  • missing-config execution fails explicitly without secrets

Parity with microsoft/opentelemetry-distro-dotnet#155. ETW logger changes are out of scope.

nikhilc-microsoft and others added 5 commits September 15, 2026 15:17
@nikhilc-microsoft
feat(samples): add Agent365 S2S observability demo
e16d43d 
Add safe exporter success diagnostics, two-stage MSAL authentication, expiry-aware caching, deterministic manual telemetry, sample documentation, and CI validation.

Co-authored-by: Copilot <[email protected]>
Copilot-Session: 9ccc29b9-bf5d-4725-be3b-c4276b233c4c
@nikhilc-microsoft
fix(samples): harden S2S refresh handling
dd74352 
Co-authored-by: Copilot <[email protected]>

Copilot-Session: 9ccc29b9-bf5d-4725-be3b-c4276b233c4c
@nikhilc-microsoft
fix(a365): align correlation fallback
a2a0003 
Co-authored-by: Copilot <[email protected]>

Copilot-Session: 9ccc29b9-bf5d-4725-be3b-c4276b233c4c
@nikhilc-microsoft
docs: link Agent365 S2S changelog entries
2af226f 
Co-authored-by: Copilot <[email protected]>

Copilot-Session: 9ccc29b9-bf5d-4725-be3b-c4276b233c4c
@nikhilc-microsoft
fix(samples): keep SDKStats dimensions unchanged
802303d 
Co-authored-by: Copilot <[email protected]>

Copilot-Session: 9ccc29b9-bf5d-4725-be3b-c4276b233c4c

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@juliomenendez Julio Carlos Menendez (juliomenendez) Awaiting requested review from juliomenendez juliomenendez will be requested when the pull request is marked ready for review juliomenendez is a code owner
@ajmfehr ajmfehr Awaiting requested review from ajmfehr ajmfehr will be requested when the pull request is marked ready for review ajmfehr is a code owner
@gwharris7 Grant Harris (gwharris7) Awaiting requested review from gwharris7 gwharris7 will be requested when the pull request is marked ready for review gwharris7 is a code owner
@biswapm Pujarini Mohapatra (biswapm) Awaiting requested review from biswapm biswapm will be requested when the pull request is marked ready for review biswapm is a code owner
@hectorhdzg Hector Hernandez (hectorhdzg) Awaiting requested review from hectorhdzg hectorhdzg will be requested when the pull request is marked ready for review hectorhdzg is a code owner
@JacksonWeber Jackson Weber (JacksonWeber) Awaiting requested review from JacksonWeber JacksonWeber will be requested when the pull request is marked ready for review JacksonWeber is a code owner
@lzchen Leighton Chen (lzchen) Awaiting requested review from lzchen lzchen will be requested when the pull request is marked ready for review lzchen is a code owner
@MSNev Nev (MSNev) Awaiting requested review from MSNev MSNev will be requested when the pull request is marked ready for review MSNev is a code owner
@rads-1996 Radhika Gupta (rads-1996) Awaiting requested review from rads-1996 rads-1996 will be requested when the pull request is marked ready for review rads-1996 is a code owner
@jeremydvoss Jeremy Voss (jeremydvoss) Awaiting requested review from jeremydvoss jeremydvoss will be requested when the pull request is marked ready for review jeremydvoss is a code owner
Copilot code review Copilot
Copilot will automatically review once the pull request is marked ready for review

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@nikhilNava @nikhilc-microsoft