Project

General

Profile

Bug #9807

Packets Monitoring graphs are being incorrectly scaled

Added by Steve Wheeler 4 months ago. Updated 30 days ago.

Status:
Resolved
Priority:
Normal
Category:
Status_Monitoring
Target version:
Start date:
10/03/2019
Due date:
% Done:

100%

Estimated time:
Affected Version:
All
Affected Architecture:

Description

It looks like the data used for the packets logging is being incorrectly multiplied by 8 as though it is assuming a Bytes value that shouldn't be be applied to PPS.

Whilst running an iperf3 test the monitoring graphs show a pps value of 644.22 kpps but the switch port logs a 5s average of 80600 pps for the same connection.

Tested against:

2.5.0-DEVELOPMENT (amd64)
built on Sat Sep 28 22:21:52 EDT 2019
FreeBSD 12.0-RELEASE-p10

Also seen here in 2.4.4p3: https://forum.netgate.com/topic/147011/pfsense-monitoring-graphs

rrd_fetch_json.php (12.5 KB) rrd_fetch_json.php fixed version Viktor Gurov, 12/12/2019 06:39 AM

History

#1 Updated by Viktor Gurov about 1 month ago

for some reason there is no rrd_fetch_json.php and status_monitoring.php files on github

fixed version:

...
                                case "inpass":
                                if ($settings['category'] === "traffic") {
                                        $multiplier = 8;
                                }
                                $ninetyfifth = true;
                                $format = "s";
                                break;
                        case "max":
                                $format = "s";
                                break;
                        case "inpass6":
                                if ($settings['category'] === "traffic") {
                                        $multiplier = 8;
                                }
                                $ninetyfifth = true;
                                $format = "s";
                                break;
                        case "outpass":
                                if ($settings['category'] === "traffic") {
                                        $multiplier = 8;
                                }
                                $invert = $invert_graph;
                                $ninetyfifth = true;
                                $format = "s";
                                break;
                        case "outpass6":
                                if ($settings['category'] === "traffic") {
                                        $multiplier = 8;
                                }
                                $invert = $invert_graph;
                                $ninetyfifth = true;
                                $format = "s";
...

#2 Updated by Jim Pingle about 1 month ago

  • Project changed from pfSense to pfSense Packages
  • Category changed from RRD Graphs to Status_Monitoring

They are a part of the Status_Monitoring package (which is included in the base install), so the files are in the freebsd-ports repo under sysutils/pfSense-Status_Monitoring

#4 Updated by Jim Pingle about 1 month ago

  • Status changed from New to Pull Request Review

#5 Updated by Renato Botelho about 1 month ago

  • Status changed from Pull Request Review to Feedback
  • Assignee set to Renato Botelho
  • % Done changed from 0 to 100

PR has been merged. Thanks!

#6 Updated by Jim Pingle about 1 month ago

  • Target version changed from 2.5.0 to 2.4.5

This was picked back to 2.4.5 as well, so needs testing there.

#7 Updated by Viktor Gurov 30 days ago

  • Status changed from Feedback to Resolved

tested on pfSense 2.4.5.a.20191220.1407

works,
Resolved

Also available in: Atom PDF