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

[docs] Add Jaeger documentation #3269

Merged
merged 7 commits into from
Feb 4, 2020

Conversation

bmorelli25
Copy link
Member

@bmorelli25 bmorelli25 commented Jan 31, 2020

First draft of Jaeger integration documentation. Closes #3123.

Open for suggestions if anyone can think of a better place for this documentation to live. Also, it'd be great to have some sort of settings reference so we can get rid of some of the noise from guides like this. It'd need to happen in Beats first, thought. I'll talk to DeDe.

An HTML preview is available here.

Copy link
Member

@axw axw left a comment

Choose a reason for hiding this comment

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

Very nice!

I'm not too sure where this should go, but to me Getting Started feels like too general of a section for it. Maybe under Configuring APM Server, with a link from Getting Started > Step 4. Next Steps?

docs/jaeger-support.asciidoc Outdated Show resolved Hide resolved
docs/jaeger-support.asciidoc Show resolved Hide resolved
docs/jaeger-support.asciidoc Outdated Show resolved Hide resolved
@bmorelli25
Copy link
Member Author

Thank you @axw!

@bmorelli25 bmorelli25 merged commit 4db6fb9 into elastic:master Feb 4, 2020
@bmorelli25 bmorelli25 deleted the jaeger-support-docs branch February 4, 2020 22:22
bmorelli25 added a commit to bmorelli25/apm-server that referenced this pull request Feb 4, 2020
bmorelli25 added a commit to bmorelli25/apm-server that referenced this pull request Feb 4, 2020
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.

[docs] Document experimental Jaeger support
2 participants