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

port process collector #65

Closed
siddontang opened this issue Sep 19, 2016 · 9 comments
Closed

port process collector #65

siddontang opened this issue Sep 19, 2016 · 9 comments
Assignees

Comments

@siddontang
Copy link
Contributor

So we can monitor current running process CPU, Memory, etc.

@siddontang
Copy link
Contributor Author

We can use https://github.com/danburkert/procinfo-rs, but it only works in unix, I think it is ok.

@bbigras
Copy link

bbigras commented Sep 20, 2016

It would be nice to have support for other platforms too, eventually.

@siddontang
Copy link
Contributor Author

@BrunoQC

We can only focus on *nix (Linux + Mac OS X), we don't use Windows in our team.
This is optional metric, maybe you can help us support Windows or other platforms. :-)

@kamalmarhubi
Copy link
Contributor

procinfo is in fact Linux only. Still a good starting point though.

@siddontang
Copy link
Contributor Author

Thanks @kamalmarhubi

We have not found any lib that supports all platforms, so now we only support Linux. If you find some or want to write one, please tell us :-)

@overvenus
Copy link
Member

Unfortunately, procinfo does not compile with stable and beta rust (danburkert/procinfo-rs#10 and danburkert/procinfo-rs#12).

I would remove procinfo temporary, and will add it back once the problem is solved.

@kamalmarhubi
Copy link
Contributor

I just opened danburkert/procinfo-rs#14 which works around the overflow issue.

@overvenus
Copy link
Member

That's cool! I will use procinfo once that is merged.

@kamalmarhubi
Copy link
Contributor

danburkert/procinfo-rs#15 just got merged, should now work on stable.

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

4 participants