-
-
Notifications
You must be signed in to change notification settings - Fork 176
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
Network Graph (sometimes wrong algebraic sign for download/upload with high peaks) #19
Comments
RRD creation is defined hardcoded in "rpimonitord" as "DS:$name:$type:600:U:U". I think, we have to make this parameter (all parameters?) changeable in the "rpimontor.conf" with a new set of defnitions. Another more simple approach would be to define the rrd as "counter" instead of "derive" in rpimonitor.conf for the Bytes/s value. Best would be a combination of both, I think! Today I have no time to investigate it any more but I'll try to get it the next days ... greets, PS: Same problem is with the static value for the net up/down in static. Edit: Replace "gauge" with "strike" in third paragraph. |
When I first read the description of the issue I first think about a counter reaching its maximum and restarting from 0. I already saw that in some Cisco monitoring. |
At first approach, today I defined the rrd as "counter" instead of "derive" in rpimonitor.conf for the rx/tx Bytes/s values. |
Made some tests and only changing to "derive" doesn't work because the rrd database saves the type of data saving. So I suggest to use COUNTER type with positive values for up and down and change it in default.conf to: dynamic.10.name=net_received dynamic.11.name=net_send Then stop the rpimonitor (sudo service rpimonitor stop) Then delete the rrd-files for send and receive (net_received.rrd and net_send.rrd in /usr/share/rpimonitor/web/stat/), so that the RRDs are created new as COUNTER type. Then start the rpimonitor again (sudo service rpimonitor start) After all you get this (after some down- and uploads): I have not yet found a way to negate the rrd output to get the negative download graph as before but I don't think it is so bad this way with up and down both in same direction in the graph (maybe some people like it even more - as I do - because of the better scaling). So for now I would suggest to change it as above mentioned. For up and down on the status screen there seems to be the best way to use the help of further software like vnstat if one wants to fix this. greets, |
min and max values are now implemented in RRD and fix the issue. Closing. |
Sometimes in the RR-Database the value seems to be saved with the wrong algebraic sign.
For me, it was the 2nd time in 2 weeks now.
It then looks like this:
I've made a zoomed 14 days view to see it better, because the peak is much higher in dayview. In dayview I'm able to see, that there are two values with this wrong sign.
The peak is as high as 250 MByte/sec in dayview with the 10 second normalized values! That's much more, than the ethernet interface is able to transfer.
I have an idea, how to fix the wrong sign, but before I want to know, if anybody else has seen this, too.
The fix would be easy, I think:
It should be possible to force the alebraic sign like this:
If the error is in the RRD readout, this can be done in the readout routine (also).
But I don't think, that the problem is in the readout, because in this case a newly generted graph (reload) would have fixed this.
An implementation in the readout routine would fix the view but not the wrong values in the database.
Would not be hard to implement this and I would develop a patch for this - but only, if I'm not alone with this problem.
And does anyone have an idea, where this wrong readout values may be come from (sign and much to high value)?
greets,
Martin
Edit:
I had a look at the rrd definitions and tools and the problem with peaks is a problem with many rrd installations.
A often given tip is to define a max and min value for the values and if a value is out of range, it then will be dropped.
I have searched in the repo but haven't yet found where it is possible to define this.
I think, this would be the best way to prevent the problems with wrong signs (without any reproduceable reason) and possible range overflows if the 2nd value is smaller than the previous one (tx/rx value is only possible max of 2^32 = 4GByte) which is probably causing the sign issue!
Today I made a 6 GB upload to my ownCloud and there was a wrong peak again ...
The text was updated successfully, but these errors were encountered: