Skip to content
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

[WIP] OpenTelemetry Web Request Tracing #388

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

EandrewJones
Copy link
Contributor

Status

  • install and build in progress
  • I've added the nodePolyfills plugin for rollup but introduced new errors.

Instructions

  • on fetch, run npm install
  • run npm run pretest to see build warnings/errors.

@Jyyjy
Copy link
Contributor

Jyyjy commented Nov 6, 2023

I'm going to dig into this, but first here's some notes on working in this repo:

  • Include the build in your branch.
  • Make PR's to test.
  • For QOL, can you separate the formatting changes and the code changes into different commits?

@EandrewJones
Copy link
Contributor Author

EandrewJones commented Nov 6, 2023 via email

@poorejc poorejc requested a review from Jyyjy December 3, 2023 04:41
@Jyyjy
Copy link
Contributor

Jyyjy commented Dec 3, 2023

After working on this with Evan, we couldn't get a successful build. The opentelemetry package requires perf hooks, but we couldn't find any polyfill in the rollup ecosystem that supports it. We concluded that adding otel would require migrating to a different bundler.

We instead updated an example webpack site that injects otel info into userale logs. Check it out here:
https://github.com/UMD-ARLIS/incubator-flagon-useralejs/tree/OTintegration

@Jyyjy Jyyjy marked this pull request as draft December 3, 2023 16:21
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