search

paritytech / substrate-telemetry

Polkadot Telemetry service
GNU General Public License v3.0
309 stars 205 forks source link

Show more CPU info of the nodes #218

Open yanganto opened 4 years ago

yanganto commented 4 years ago

The CPU percentage usage is shown in the panel, and some nodes ran into heavy CPU usage problems. Providing more information about CPU may be a better way to know more about this issue.

Besides, there is a crate for CPU info. https://crates.io/crates/raw-cpuid

Maybe we can provide the CPU info in the telemetry to help others know more about the node.

maciejhirsz commented 4 years ago

That would be useful, it would have to be done in Substrate first.

One concern is that we don't want to reveal too much about nodes, at least not in the default configuration, mostly due to GDPR.

yanganto commented 4 years ago

Thanks for reply. I am not sure how the CPU info will be related to PII, and break liability of GDPR. If it is acceptable to make this info with an option configuration, I am willing to try to make this happen. Once you know other issues that we can not do this on Substrate, please kindly tell me. Thank you.

yanganto commented 4 years ago

The complete environment of substrate and telemetry is complicated. I try to work on this issue in this commit, if you @maciejhirsz have time to give me more suggestion. It will be appreciated. Thanks. https://github.com/yanganto/substrate/commit/5662d2396ee707c9393aa8c5c14d0b056bf6b256