-
Notifications
You must be signed in to change notification settings - Fork 10
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
OpenTelemetry output for OpenSearch demo #117
OpenTelemetry output for OpenSearch demo #117
Conversation
@TheElectronWill I think this is ready for review |
Is there a reason for the "97" in the PR title? |
Hi, yes. I did the dev and demo based on opensearch and I swa this issue #97 where I was waiting for feedback but in the end I got it to work. |
Oh, okay! In that case I will edit the title and add a github-specific description to link the PR to the issue. |
Please rebase instead of merging in order to keep the history clean :) |
8fcfeb0
to
09caef5
Compare
9438574
to
5e13000
Compare
Co-authored-by: Cyprien <[email protected]>
5e13000
to
b33d1dc
Compare
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.
almost done
Co-authored-by: Guillaume Raffin <[email protected]>
Replicated the structure of the prometheus plugin to export directly to opentelemetry.
It has been tested with a local deployment of Opensearch
Resolves #97