Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add Jaeger distributed tracing support #3307

Merged
merged 5 commits into from
Aug 10, 2018
Merged

Add Jaeger distributed tracing support #3307

merged 5 commits into from
Aug 10, 2018

Conversation

leth
Copy link
Contributor

@leth leth commented Aug 8, 2018

Allows trace reporting to be configured via environment variables e.g. JAEGER_AGENT_HOST, JAEGER_SAMPLER_TYPE.

@leth leth requested review from 2opremio, bboreham and marccarre August 8, 2018 12:16
Copy link
Collaborator

@bboreham bboreham left a comment

Choose a reason for hiding this comment

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

LGTM, thanks

Copy link
Contributor

@satyamz satyamz left a comment

Choose a reason for hiding this comment

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

lgtm!

@bboreham
Copy link
Collaborator

List of changes pulled in from weaveworks/common:

@bboreham bboreham merged commit 5531a29 into master Aug 10, 2018
@leth leth deleted the init-jaeger branch August 16, 2018 13:36
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants