diff options
author | Fran�ois-R�gis Menguy <francoisregis.menguy@orange.com> | 2020-11-16 07:00:28 +0000 |
---|---|---|
committer | Gerrit Code Review <gerrit@opnfv.org> | 2020-11-16 07:00:28 +0000 |
commit | f684939fed81471c1381fa4fab6e1b6ea37093da (patch) | |
tree | 2e9764b22ec2e9e5a3bdbffb75fe8d69ef7d5bf2 /nfvbench/packet_stats.py | |
parent | 78f547514e54a9332b919734aed7c8ca1390cb48 (diff) | |
parent | bd0cf4ce9c30c8aae9b7a96cf9c8ea073cd9a548 (diff) |
Merge "NFVBENCH-192: Complete/fix hdrh related processings to consider all cases (multiple service chains, distribution n/a with intel VFs)"
Diffstat (limited to 'nfvbench/packet_stats.py')
-rw-r--r-- | nfvbench/packet_stats.py | 21 |
1 files changed, 12 insertions, 9 deletions
diff --git a/nfvbench/packet_stats.py b/nfvbench/packet_stats.py index d6b9a68..d3ec78a 100644 --- a/nfvbench/packet_stats.py +++ b/nfvbench/packet_stats.py @@ -239,18 +239,21 @@ class PacketPathStats(object): results = {'lat_min_usec': latency.min_usec, 'lat_max_usec': latency.max_usec, 'lat_avg_usec': latency.avg_usec} - if latency.hdrh: + if latency.hdrh_available(): results['hdrh'] = latency.hdrh decoded_histogram = HdrHistogram.decode(latency.hdrh) - # override min max and avg from hdrh - results['lat_min_usec'] = decoded_histogram.get_min_value() - results['lat_max_usec'] = decoded_histogram.get_max_value() - results['lat_avg_usec'] = decoded_histogram.get_mean_value() results['lat_percentile'] = {} - for percentile in self.config.lat_percentiles: - results['lat_percentile'][percentile] = decoded_histogram.\ - get_value_at_percentile(percentile) - + # override min max and avg from hdrh (only if histogram is valid) + if decoded_histogram.get_total_count() != 0: + results['lat_min_usec'] = decoded_histogram.get_min_value() + results['lat_max_usec'] = decoded_histogram.get_max_value() + results['lat_avg_usec'] = decoded_histogram.get_mean_value() + for percentile in self.config.lat_percentiles: + results['lat_percentile'][percentile] = decoded_histogram.\ + get_value_at_percentile(percentile) + else: + for percentile in self.config.lat_percentiles: + results['lat_percentile'][percentile] = 'n/a' else: results = {} results['packets'] = counters |