Skip to content

Set up frontend instrumentation and other OTel related refactorings #2247

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 10 commits into from
Nov 24, 2025
Merged

Set up frontend instrumentation and other OTel related refactorings #2247

merged 10 commits into from
Nov 24, 2025

Conversation

@reakaleek
Copy link
Member

@reakaleek reakaleek commented Nov 22, 2025 •
edited
Loading

Changes

Frontend

  • Set up OpenTelemetry instrumentation for browser traces and logs
  • Instrumented search and Ask AI flows with semantic conventions
  • Added automatic fetch instrumentation with trace context propagation
  • Smart handling of aborted searches (not treated as errors)

Backend refactor

  • Replaced primitive tuples with typed records (OtlpForwardResult)
  • Refactored exception handling to be type-based instead of string-parsing
  • Eliminated code duplication in endpoint mappings (DRY principle)
  • Removed unused WireMock.Net dependency (fixes high-severity vulnerability)

Related

@reakaleek reakaleek changed the base branch from main to feature/otlp-proxy November 22, 2025 01:00
@reakaleek reakaleek force-pushed the feature/frontend-tracing branch from 67946b9 to a13d39e Compare November 22, 2025 01:07
@reakaleek reakaleek force-pushed the feature/frontend-tracing branch from a13d39e to 45f6f4d Compare November 22, 2025 01:12
reakaleek
reakaleek commented Nov 22, 2025
View reviewed changes
@reakaleek reakaleek marked this pull request as ready for review November 24, 2025 10:07
@reakaleek reakaleek requested a review from a team as a code owner November 24, 2025 10:07
@reakaleek reakaleek requested a review from Mpdreamz November 24, 2025 10:07
Base automatically changed from feature/otlp-proxy to main November 24, 2025 13:26
@reakaleek reakaleek merged commit f71b13a into main Nov 24, 2025
28 checks passed
@reakaleek reakaleek deleted the feature/frontend-tracing branch November 24, 2025 14:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@cotti cotti cotti approved these changes

@Mpdreamz Mpdreamz Awaiting requested review from Mpdreamz Mpdreamz is a code owner automatically assigned from elastic/docs-engineering

Assignees

No one assigned

Labels

chore

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@reakaleek @cotti