Skip to content

tracing: add the jaeger exporter#1848

Closed
nicks wants to merge 1 commit intodocker:masterfrom
nicks:nicks/tracing
Closed

tracing: add the jaeger exporter#1848
nicks wants to merge 1 commit intodocker:masterfrom
nicks:nicks/tracing

Conversation

@nicks
Copy link
Copy Markdown
Contributor

@nicks nicks commented May 26, 2023

once this env variable is set, the
the docs in https://docs.docker.com/build/building/opentelemetry/ will work

fixes #1847

@nicks nicks force-pushed the nicks/tracing branch 2 times, most recently from 01ea08b to 561d158 Compare May 26, 2023 14:06
@jedevc jedevc requested a review from tonistiigi May 26, 2023 14:08
once this env variable is set, then
the docs in https://docs.docker.com/build/building/opentelemetry/
will work

fixes docker#1847

Signed-off-by: Nick Santos <nick.santos@docker.com>
@nicks
Copy link
Copy Markdown
Contributor Author

nicks commented May 26, 2023

actually poking around a bit more at the delegated tracer, i don't think this is right, i think the tracer delegation isn't work right...still digging...

@nicks
Copy link
Copy Markdown
Contributor Author

nicks commented May 26, 2023

ok, i see the real problem, closing this PR

@nicks nicks closed this May 26, 2023
@nicks
Copy link
Copy Markdown
Contributor Author

nicks commented May 26, 2023

the real fix is at moby/buildkit#3909

@nicks nicks deleted the nicks/tracing branch May 26, 2023 15:43
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.

opentelemetry support does not send buildx spans

1 participant