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.

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

chore: setup open-telemetry #374

Closed
wants to merge 11 commits into from
Closed

chore: setup open-telemetry #374

wants to merge 11 commits into from

Conversation

hi-ogawa
Copy link
Owner

@hi-ogawa hi-ogawa commented May 6, 2023 •
edited
Loading

todo

  • patch node-sdk to strip unused adapters
  • trace more
    • trpc call
    • remix loader
    • remix page request
      • resolve route id
  • configure via environment variables
grep OTEL_ .env.staging | bash scripts/env-vercel.sh preview
grep OTEL_ .env.production | bash scripts/env-vercel.sh production
  • verify trace on newrelic

image

image

examples

on local jaeger

image

image

@hi-ogawa hi-ogawa marked this pull request as ready for review May 6, 2023 10:25
@hi-ogawa hi-ogawa mentioned this pull request May 9, 2023
Open
10 tasks
@hi-ogawa
Copy link
Owner Author

This should wait for

@hi-ogawa hi-ogawa mentioned this pull request Jun 3, 2023
Merged
4 tasks
@hi-ogawa
Copy link
Owner Author

hi-ogawa commented Jun 3, 2023

@hi-ogawa hi-ogawa closed this Jun 3, 2023
@hi-ogawa hi-ogawa deleted the chore-opentelemetry branch June 10, 2023 04:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

chore: setup opentelemetry
1 participant
@hi-ogawa