-
Notifications
You must be signed in to change notification settings - Fork 4.5k
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: opentelemetry and prometheus #2543
Conversation
Pull Request Validation ReportThis comment is automatically generated by Conventional PR Whitelist Report
Result Pull request does not satisfy any enabled whitelist criteria. Pull request will be validated. Validation Report
Result Pull request satisfies all enabled pull request rules. Last Modified at 04 Jul 24 21:19 UTC |
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.
This is awesome. Thanks, @zzzming
LGTM
* opentelemetry and prometheus * set env override in create_app * add prometheus_client to pyproject * update top level poetry.lock
* opentelemetry and prometheus * set env override in create_app * add prometheus_client to pyproject * update top level poetry.lock
* opentelemetry and prometheus * set env override in create_app * add prometheus_client to pyproject * update top level poetry.lock (cherry picked from commit c444a69)
This PR introduces as the first phase of OpenTelemetry implementation.
What's not implemented in this PR
An example of prometheus scraping