From c0b3d5a7abc6acd0282c063844471ed6aa02d36b Mon Sep 17 00:00:00 2001 From: Abhijit Sinha Date: Wed, 18 Oct 2017 16:44:45 +0100 Subject: BugFix: Negative dropped packets in Prox tests Removed the abs function which can potentially mask negative dropped packets. Dropped packets in Prox workload VNF = max((tx_packets - rx_packets), 0) Change-Id: I510a351e899cdf9a1f366d632b9f0528b1d9dcce Signed-off-by: Abhijit Sinha (cherry picked from commit a27278dacaa54ae60cd3bdfa6e6145643f76fa02) --- yardstick/network_services/vnf_generic/vnf/prox_vnf.py | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) (limited to 'yardstick/network_services/vnf_generic/vnf/prox_vnf.py') diff --git a/yardstick/network_services/vnf_generic/vnf/prox_vnf.py b/yardstick/network_services/vnf_generic/vnf/prox_vnf.py index 24712dd27..b7d295eee 100644 --- a/yardstick/network_services/vnf_generic/vnf/prox_vnf.py +++ b/yardstick/network_services/vnf_generic/vnf/prox_vnf.py @@ -89,9 +89,9 @@ class ProxApproxVnf(SampleVNF): return {} result = { - "packets_in": tx_total, - "packets_dropped": abs(rx_total - tx_total), - "packets_fwd": rx_total, + "packets_in": rx_total, + "packets_dropped": max((tx_total - rx_total), 0), + "packets_fwd": tx_total, # we share ProxResourceHelper with TG, but we want to collect # collectd KPIs here and not TG KPIs, so use a different method name "collect_stats": self.resource_helper.collect_collectd_kpi(), -- cgit 1.2.3-korg