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

[OTLP] Document support for OTLP/gRPC on the standard port #4678

Closed
axw opened this issue Feb 3, 2021 · 4 comments · Fixed by #4964
Closed

[OTLP] Document support for OTLP/gRPC on the standard port #4678

axw opened this issue Feb 3, 2021 · 4 comments · Fixed by #4964
Assignees
Milestone

Comments

@axw
Copy link
Member

axw commented Feb 3, 2021

When #4503 is fully implemented, APM Server will serve OTLP gRPC over the same port as the Elastic APM agent protocol (i.e. port 8200 by default).

Like in #4621, I think we should first have a discussion about how we want the UX to look around auth tags, and again I'll mark this issue blocked until we have a decision. Let's stick with what we've got for now, we can always introduce separate tags later.

@cyrille-leclerc
Copy link
Contributor

cyrille-leclerc commented Mar 3, 2021

@bmorelli25 can we work together on this improvement of OTLP docs? I have prepared elements for this.
@bmorelli25 I would like us to learn from elastic/apm#417 to understand how we have to improve our docs.

@bmorelli25
Copy link
Member

👍 Sure! Fair warning: this issue wasn't in our 7.12 docs planning, so it hasn't been prioritized. I'd like to wrap @EamonnTP into the conversations as well in case he is able to pick it up before me.

@bmorelli25
Copy link
Member

Hmmm. Ignore that comment. I mentioned it in https://github.com/elastic/observability-dev/issues/1279#issuecomment-749772370, but it never made it into the planning doc for some reason. Oh well. Between @EamonnTP and I, I'm sure we can get it done.

@EamonnTP
Copy link
Contributor

EamonnTP commented Mar 3, 2021

@bmorelli25 I can pick this up. @cyrille-leclerc I'll put some time in your calendar to discuss more.

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

Successfully merging a pull request may close this issue.

4 participants