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

Default stats for memory and CPU usage #31

Closed
jsha opened this issue Sep 3, 2016 · 1 comment
Closed

Default stats for memory and CPU usage #31

jsha opened this issue Sep 3, 2016 · 1 comment

Comments

@jsha
Copy link

jsha commented Sep 3, 2016

Node provides built-in ways to get CPU and memory usage information from the process package:

https://nodejs.org/api/process.html#process_process_cpuusage_previousvalue
https://nodejs.org/api/process.html#process_process_memoryusage

It would be useful to provide these as exported variables by default. This would be in keeping with practice of other Prometheus clients, like the Go client. Would you be open to a patch?

@siimon
Copy link
Owner

siimon commented Sep 4, 2016

There is already a PR open for this, see #25!

I think it's pretty close a merge so should hopefully be published next week.

@siimon siimon closed this as completed Sep 4, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants