-
Notifications
You must be signed in to change notification settings - Fork 83
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
feat: Add support for tracing for http addon #1021
base: main
Are you sure you want to change the base?
feat: Add support for tracing for http addon #1021
Conversation
f945b23
to
1eb19da
Compare
f64c6dc
to
480050e
Compare
c95f18e
to
eb64c18
Compare
@bhussain91 any update on this please? |
eb64c18
to
b093281
Compare
@zroubalik I've refactored the code as per comments by @JorTurFer |
8075523
to
91702b3
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.
@bhussain91 could you please fix the linter problem?
91702b3
to
444c2b5
Compare
@zroubalik |
Signed-off-by: Bilal Hussain <bilal.hussain@10xbanking.com>
444c2b5
to
4d127f0
Compare
Signed-off-by: Bilal Hussain bilal.hussain@10xbanking.com
Provide a description of what has been changed
The change is to add tracing capability into the keda http addon using opentelemetry. The changes allows the add on to generate and export spans to console (development) or otel gRPC endpoint. Further information on how to configure tracing can be found in the docs
Checklist
README.md
docs/
directoryFixes #