Skip to content

OTel experimentation #49409

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
wants to merge 17 commits into
base: main
Choose a base branch
from
Draft

OTel experimentation #49409

wants to merge 17 commits into from

Conversation

baronfel
Copy link
Member

@baronfel baronfel commented Jun 15, 2025

This is entirely exploratory so far - we can't use AppInsights libraries if we do AOT in the SDK, so I'm exploring what an OTel-based way of doing telemetry might look like.

  • All CLI calls make at least one Activity
  • TrackEvent now creates Events on the ambient Activity with the same content as before

Q: telemetry filtering/hashing
A: for now we should keep everything as it is. As we make new activities, we should use Activity.IsAllDataRequested as a check before adding/computing expensive data.

Q: do we keep the store-out-of-band-then-rehydrate-and-send approach, or just do a best-case flush?
A: We can be very optimistic! We can have multiple collectors, each for different purposes. The default OTel collector can be for local use only (by default) and can be verbose, while the Azure Monitor collector can be for production only and be more restrained. Each Collector has its own implementation, and the Azure Monitor collector even handles durable message persistence, so we can delete all of our current implementation. We should still have it store its durable store to our owned directory, though.

Example Aspire Dashboard outputs for dotnet new classlib:
image

Telemetry events are visible in this view if the CLI is opted in to telemetry.

MiYanni and others added 4 commits June 26, 2025 17:54
…nts and several nullable disable removals. Changed s_source to Source as a static property.
…ader and ToolPackageDownloaderBase since those will be updated in main. Not interested in cleaning up test files.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants