Skip to content

Add Microsoft OpenTelemetry distro and configuration#4

Merged
hectorhdzg merged 2 commits into
microsoft:mainfrom
hectorhdzg:hectorhdzg/rebrand-microsoft-opentelemetry
Apr 7, 2026
Merged

Add Microsoft OpenTelemetry distro and configuration#4
hectorhdzg merged 2 commits into
microsoft:mainfrom
hectorhdzg:hectorhdzg/rebrand-microsoft-opentelemetry

Conversation

@hectorhdzg
Copy link
Copy Markdown
Member

Its based on branch created in #3, once that is merged it would be easier to review this one

JacksonWeber added a commit that referenced this pull request Apr 7, 2026
…ation

- Add useMicrosoftOpenTelemetry() entry point with modular setup architecture
- Add shutdownMicrosoftOpenTelemetry() for clean teardown
- Integrate @azure/monitor-opentelemetry via internal setup module
- Add OTLP activation via env vars (OTEL_EXPORTER_OTLP_*)
- Add extensible setup modules for Azure Monitor, OTLP, and A365
- Add MicrosoftOpenTelemetryOptions with scoped config sections
- Add azure-monitor-opentelemetry/ temp copy directory (Phase 3 staging)
- Add PR validation CI workflow (ESLint + tests on Node 18/20/22)
- Add PLANNING.md aligned with Python distro PR #4
- Add README, CONTRIBUTING, CHANGELOG, PRIVACY, NOTICE docs
- Add 41 unit tests covering all activation paths

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@hectorhdzg hectorhdzg force-pushed the hectorhdzg/rebrand-microsoft-opentelemetry branch from 3fc0e2d to b284239 Compare April 7, 2026 20:00
@hectorhdzg hectorhdzg marked this pull request as ready for review April 7, 2026 20:00
Comment thread src/shared/envConfig.ts Outdated
Comment thread src/azureMonitorSetup.ts
Copy link
Copy Markdown
Contributor

@JacksonWeber JacksonWeber left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM - few comments.

@hectorhdzg hectorhdzg merged commit 362f17b into microsoft:main Apr 7, 2026
5 checks passed
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