Skip to content

Conversation

@dennwc
Copy link
Contributor

@dennwc dennwc commented Jan 6, 2026

Add helper for setting Jaeger tracing for running with tracing or testing in OSS version.

@dennwc dennwc requested review from a team January 6, 2026 11:11
@dennwc dennwc self-assigned this Jan 6, 2026
@changeset-bot
Copy link

changeset-bot bot commented Jan 6, 2026

🦋 Changeset detected

Latest commit: 07178b0

The changes in this PR will be included in the next version bump.

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

💥 An error occurred when fetching the changed packages and changesets in this PR
Some errors occurred when validating the changesets config:
The package or glob expression "github.com/livekit/protocol" specified in the `fixed` option does not match any package in the project. You may have misspelled the package name or provided an invalid glob expression. Note that glob expressions must be defined according to https://www.npmjs.com/package/micromatch.

Copy link
Contributor

@boks1971 boks1971 left a comment

Choose a reason for hiding this comment

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

not sure how to use this, but change lgtm!

@dennwc
Copy link
Contributor Author

dennwc commented Jan 6, 2026

@boks1971 here's an example of configuration (with Jaeger container running and jaeger_url: localhost). Add for distributed tracing to work, one needs to add a client and server options for PSRPC.

@dennwc dennwc merged commit 0a3efbf into main Jan 6, 2026
10 checks passed
@dennwc dennwc deleted the jaeger branch January 6, 2026 11:23
@github-actions github-actions bot mentioned this pull request Jan 5, 2026
@boks1971
Copy link
Contributor

boks1971 commented Jan 6, 2026

@boks1971 here's an example of configuration (with Jaeger container running and jaeger_url: localhost). Add for distributed tracing to work, one needs to add a client and server options for PSRPC.

Thank you @dennwc

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants