serverinfo icon indicating copy to clipboard operation
serverinfo copied to clipboard

CPU Load naming in servinfo ReST call

Open indigo423 opened this issue 9 years ago • 4 comments

I assume the values for CPU load are 1 minute, 5 minute and 15 minutes. It would be nice to know which value is 1 minute, 5 minute and 15 minute load.

<cpuload>
    <element>2.76806640625</element>
    <element>1.99267578125</element>
    <element>1.67626953125</element>
</cpuload>

should be similar to active users

<activeUsers>
    <last5minutes>1</last5minutes>
    <last1hour>1</last1hour>
    <last24hours>1</last24hours>
</activeUsers>

indigo423 avatar Aug 21 '16 08:08 indigo423

As I see it is a PHP internal call http://php.net/manual/de/function.sys-getloadavg.php

indigo423 avatar Aug 21 '16 09:08 indigo423

Just as a side node, I didn't had any problems in OpenNMS, cause we use XPath[1] and this is not complicated. The change would make it for other monitoring tools a little bit easier where people have to parse the XML file manually.

[1] https://github.com/opennms-config-modules/nextcloud/blob/master/xml-datacollection/nextcloud-serverinfo.xml#L6

indigo423 avatar Aug 25 '16 13:08 indigo423

Yeah, problem is that we can not rename the elements easily, since it will break existing endpoint consumers. But we could of course duplicate the elements for the time-being

nickvergessen avatar Dec 21 '16 14:12 nickvergessen

Feel free to make a pull request to do this.

nickvergessen avatar Dec 21 '16 14:12 nickvergessen