NEW Monitoring graph

jdillard

@virgiliomi:

Quality graph: Loss is identified in "ms"… oops! :)

Fixed. I'll rev the package first thing in the morning so all the fixes I made today show up.

https://github.com/pfsense/FreeBSD-ports/commits/devel?author=jdillard

@virgiliomi:

I set both left and right axes, then when I set the right axis to none and updated, the colors and lines remained on the graph (though they moved vertically a bit when I pressed update). The right axis lines remained at whatever time scale it was set to when it was on (evident by the fact that when I was at the 8 hour mark for the left axis, the line dots were about 1/3 across on the right axis).

I think I need to feed it an empty set of data when "None" is selected. I'm saving this one for the end since a refresh fixes it for now.

@virgiliomi:

RRD graph for NTP is enabled, but no option for NTP as an axis.

I need to add some code for that as well as captive portal and more. Here is a ticket with more feedback (I put the NTP axis in there): https://redmine.pfsense.org/issues/5498

Thanks to everyone for the feedback so far!

heper

At devs:please dont forget to port the "hidden" dhcpd rrd

bblasius

Thanks to everyone for the feedback so far!

With the old graphs, we had the ability to set which category we wanted to see first… i.e traffic instead of system. Will this option be available with the new graphs?

M_Devil

Traffic displays bits / sec, but seems to represent bytes / sec
Same for data summary below graph

Also not consistent with https://forum.pfsense.org/index.php?topic=106547.0

MikeV7896

How about an option to refresh the graph in its current state (meaning if I've hidden one or more items, they stay hidden post-refresh), or an auto-refresh option? If I just hit the Update button to refresh, any lines that I've disabled are shown again.

jdillard

@heper:

At devs:please dont forget to port the "hidden" dhcpd rrd

I expect to have the missing graphs in place today (some are already fixed, just not committed)

jdillard

@virgiliomi:

How about an option to refresh the graph in its current state (meaning if I've hidden one or more items, they stay hidden post-refresh), or an auto-refresh option? If I just hit the Update button to refresh, any lines that I've disabled are shown again.

That is a little technically challenging, so I'll hold off attempting. Although, an easier (partial) step I could work on would be to have it remember the selected drop downs.

jdillard

@bblasius:

Thanks to everyone for the feedback so far!

With the old graphs, we had the ability to set which category we wanted to see first… i.e traffic instead of system. Will this option be available with the new graphs?

I should be able to reuse that current config value for the left Axis. I'll need to build in more config options at some point. Short answer is yes, just not sure how soon.

Raul Ramos

Hi

Cosmetic thing for now. In the "white" theme the graph grid color seems ok not to much on focus on the dark theme the first thing i see is grids everywhere. A barely visible grid should be ok

For the rest i know is in a very early development but i see a ton of drop down menus for a monitor. To much clicks, my mouse wealthy go south very fast with this design i use this a lot and plenty of categorys ISPs in this this town are very bad so i need to use this thing a lot.
Maybe a button style:
Left axis: Right axis:
System Traffic Packets Quality <checkbox to="" enable="" or="" disable=""> System Traffic Packets Quality <checkbox to="" enable="" or="" disable="">- States Processes Memory Mbuf - Wan Lan IPsec
or vertical
Right axis:
System Traffic Packets Quality <checkbox to="" enable="" or="" disable="">- Wan Lan IPsec

On top the category and depending of the category in the second line is the Graph.
Options cold be on the right side of the Update button like Time: 1h 8 hrs 1day etc Inverse: <checkbox>(or drop down but i don't like it on info graphs i personally try to avoid )

I'm a simple guy so this is my personally opinion. But this graphs looks cool so i spect this on the traffic graph status and widgets.

Thanks
</checkbox></checkbox></checkbox></checkbox>

grandrivers

black font is hard to read in dark theme

Raul Ramos

@grandrivers:

black font is hard to read in dark theme

Black font? for the grid a less dar grey than the graph background not the light grey, for the name of the category and graph text should be the base text color or second base or whatever if for some reason they decide to go with a more buttonised approach. But some black text are in the switches and labels on the graph, to much dark yes, i agree.

jdillard

@grandrivers:

black font is hard to read in dark theme

I had it working for a while, but something happened with the CSS, I plan on looking into it today.

jdillard

I added the NTP, DHCP, Captive Portal, and Traffic Shaping graphs. I also made some of the interfaces friendly, still working on the VPN interfaces though.

I also forgot to take some troubleshooting code out, so depending on how soon you upgrade you might see it at the top :-\

edit: Next version will also be showing VPN Users and friendly names for the OpenVPN servers.

Clown

Have an issue with that, until I established an OpenVPN-Interface the Monitoring looked good.

Afterwards please look at the attached Picture.

I could reproduse it on two Systems.

Monitoring.PNG_thumb

jdillard

See one post above…I was looking at it for so long I forgot it was there.

@jdillard:

I also forgot to take some troubleshooting code out, so depending on how soon you upgrade you might see it at the top :-\

It won't be in the next snapshot though.

And speaking of OpenVPN interfaces, I am cleaning up their names in the dropdown now.

edit: I just bumped the version so the next one will also be showing VPN Users and friendly names for the OpenVPN servers. (as well as remove the stuff at the top)

jdillard

@jdillard:

@grandrivers:

black font is hard to read in dark theme

I had it working for a while, but something happened with the CSS, I plan on looking into it today.

There was a missing bracket in the CSS. I just pushed the fix.

heper

Version 	2.3-BETA (i386)
built on Fri Mar 04 10:20:14 CST 2016
FreeBSD 10.3-RC1

Warning: Invalid argument supplied for foreach() in /usr/local/www/status_monitoring.php on line 150 Call Stack: 0.0004 118996 1\. {main}() /usr/local/www/status_monitoring.php:0 Warning: Invalid argument supplied for foreach() in /usr/local/www/status_monitoring.php on line 124 Call Stack: 0.0004 118996 1\. {main}() /usr/local/www/status_monitoring.php:0 Warning: Invalid argument supplied for foreach() in /usr/local/www/status_monitoring.php on line 200 Call Stack: 0.0004 118996 1\. {main}() /usr/local/www/status_monitoring.php:0

jdillard

Arg, …so many test cases...so many VMs. Thanks heper! I think I just need to throw in some if statements for the openvpn stuff.

NOYB

Min: 0%
Avg: 1%
Max: 0%
See attached image.

I must have been daydreaming when that was being covered. Could someone please tutor me before the test.

MinAvgMax.jpg_thumb

jdillard

@NOYB:

Min: 0%
Avg: 1%
Max: 0%
See attached image.

I must have been daydreaming when that was being covered. Could someone please tutor me before the test.

I forget why we decided to put that code in, but I just took it back out.