8000 Enable psutil CPU Metrics via Configuration or by default · Issue #497 · tarantool/metrics · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content
/ metrics Public
  • Notifications You must be signed in to change notification settings
  • < 8000 a icon="repo-forked" id="fork-button" href="/login?return_to=%2Ftarantool%2Fmetrics" rel="nofollow" data-hydro-click="{"event_type":"authentication.click","payload":{"location_in_page":"repo details fork button","repository_id":125366028,"auth_type":"LOG_IN","originating_url":"https://github.com/tarantool/metrics/issues/497","user_id":null}}" data-hydro-click-hmac="93ed2e0696e5e35da78729ffd99c573967ee0cb3d492785fa18b80a59427be72" data-view-component="true" class="btn-sm btn"> Fork 24
  • Star 38

Enable psutil CPU Metrics via Configuration or by default #497

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.

Sign up for GitHub

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

Jump to bottom
Closed
Satbek opened this issue Feb 3, 2025 · 1 comment · Fixed by #498
Closed

Enable psutil CPU Metrics via Configuration or by default #497

Satbek opened this issue Feb 3, 2025 · 1 comment · Fixed by #498
Assignees
@yngvar-antonsson

Comments

@Satbek
Copy link
Satbek commented Feb 3, 2025

Description

Add the ability to enable psutil CPU metrics via configuration instead of requiring manual registration in the code.

Context

Currently, CPU metrics that rely on psutil are not available by default. Results can only be obtained for Linux. Reference. These metrics are useful as they provide insights into per-thread CPU load.

However, activating these metrics requires explicit registration of a callback in the code, which is inconvenient.

Expected Behavior

  • The ability to enable CPU metrics via configuration instead of requiring manual callback registration.
  • Alternatively, have these metrics enabled by default, as they already work correctly on all plafrorms (returns nothing for not linux platform).

Motivation

  • Improves usability by reducing the need for manual code changes.
  • Ensures CPU metrics are available out of the box for Linux users.
@Satbek
Copy link
Author
Satbek commented Feb 3, 2025

As an option, psutil-based CPU metrics could be moved to a cpu_extended group and configured like this:

metrics:
  include: [
      cpu_extended
 ]

@yngvar-antonsson yngvar-antonsson mentioned this issue Feb 3, 2025
Merged
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@yngvar-antonsson yngvar-antonsson

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add new metrics category: cpu extended tarantool/metrics
2 participants
@Satbek @yngvar-antonsson
0