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

refactor: tracing #8195

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

refactor: tracing #8195

wants to merge 1 commit into from

Conversation

h-a-n-a
Copy link
Collaborator

@h-a-n-a h-a-n-a commented Oct 22, 2024

Summary

Supported open-telemetry as tracing layer.

Enable open-telemetry tracing with env:

RSPACK_PROFILE="TRACE=filter=trace&layer=otel" rspack build

You may set OTEL_BSP_MAX_QUEUE_SIZE if open-telemetry reports the following error:

OpenTelemetry trace error occurred. cannot send message to batch processor as the channel is full

To visualize the result, jaeger is recommended as a backend and UI. Rspack by default uses the default gRPC endpoint which is the same as jaeger all-in-one. For more detail, please refer to https://www.jaegertracing.io/docs/1.62/getting-started/#all-in-one

image

Checklist

  • Tests updated (or not required).
  • Documentation updated (or not required).

Copy link

netlify bot commented Oct 22, 2024

Deploy Preview for rspack canceled.

Name Link
🔨 Latest commit 6fedece
🔍 Latest deploy log https://app.netlify.com/sites/rspack/deploys/673481d281c9720008719bbf

@h-a-n-a h-a-n-a force-pushed the refactor-tracing branch 10 times, most recently from e85c832 to e1dbf4d Compare November 13, 2024 09:38
@h-a-n-a h-a-n-a marked this pull request as ready for review November 13, 2024 09:44
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.

1 participant