-
Notifications
You must be signed in to change notification settings - Fork 5
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: document supported tecnhologies #256
base: main
Are you sure you want to change the base?
Conversation
docs/supported-technologies.md
Outdated
## Elastic Stack versions | ||
|
||
The Elastic Distribution of OpenTelemetry Node.js (EDOT Python) sends data via OpenTelemetry protocol (OTLP). | ||
Elastic Observability supports OTLP natively since version 7.14, but is strongly recommended to be using a recent 8.x version. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could this lead to misunderstandings if we highlight OTLP support since 7.14? We should be clear about the versions especially of supported compatibility.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changed to It is strongly recommended to be using at least a recent 8.x version of the Elastic Observability Stack.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I propose to rephrase the sentence with supported versions to be more explicit and avoid confusion of the user.
676118f
to
215ca8f
Compare
What does this pull request do?
Add a page documenting the supported technologies