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

Custom Python runtime metrics dashboard #89

Open
Tracked by #30
xrmx opened this issue Jul 3, 2024 · 5 comments
Open
Tracked by #30

Custom Python runtime metrics dashboard #89

xrmx opened this issue Jul 3, 2024 · 5 comments

Comments

@xrmx
Copy link
Member

xrmx commented Jul 3, 2024

Is your feature request related to a problem? Please describe.

We should create our own runtime metrics dashboard in Kibana to have a decent UX out of the box.

One thing we must take into account are metrics vs fork(). On projects that are using processes for concurrency the same service may send different metrics values for each process.
Not sure we have a good solution for handling this in OTel upstream.

@AlexanderWert
Copy link
Member

Duplicate of #24

@AlexanderWert AlexanderWert marked this as a duplicate of #24 Jul 4, 2024
@AlexanderWert AlexanderWert closed this as not planned Won't fix, can't repro, duplicate, stale Jul 4, 2024
@xrmx
Copy link
Member Author

xrmx commented Jul 9, 2024

@AlexanderWert #24 was for system metrics but we later decided to disable them by default. This one was for showing something out of the box on the metrics page.

@xrmx xrmx changed the title Custom python runtime metrics dashboard Custom Python runtime metrics dashboard Jul 9, 2024
@AlexanderWert AlexanderWert reopened this Jul 9, 2024
@AlexanderWert
Copy link
Member

sorry, reopened

@xrmx
Copy link
Member Author

xrmx commented Jul 9, 2024

After discussion with the APM team it was suggested to show only aggregated data in the dashboard so that multiple processes sending the same metrics won't be a problem

@xrmx
Copy link
Member Author

xrmx commented Aug 1, 2024

And before doing a dashboard it would be better to wait for https://github.com/open-telemetry/opentelemetry-python-contrib/pull/2727 (or similar)

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

No branches or pull requests

2 participants