[Fuego] [PATCH 3/3] iperf3: use bps instead of Mbps

Daniel Sangorrin daniel.sangorrin at toshiba.co.jp
Fri Mar 2 08:57:12 UTC 2018


The -f/--format is not being used on the JSON output report.
For that reason, I decided to use bps for now and open
an issue upstream:
https://github.com/esnet/iperf/issues/709

Signed-off-by: Daniel Sangorrin <daniel.sangorrin at toshiba.co.jp>
---
 engine/tests/Benchmark.iperf3/fuego_test.sh  | 2 +-
 engine/tests/Benchmark.iperf3/parser.py      | 2 +-
 engine/tests/Benchmark.iperf3/reference.json | 6 +++---
 3 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/engine/tests/Benchmark.iperf3/fuego_test.sh b/engine/tests/Benchmark.iperf3/fuego_test.sh
index 09eac86..9f10743 100755
--- a/engine/tests/Benchmark.iperf3/fuego_test.sh
+++ b/engine/tests/Benchmark.iperf3/fuego_test.sh
@@ -26,7 +26,7 @@ function test_run {
     if [ "$ROUTE" = "local" ]; then
         echo "Starting iperf3 server on localhost (SERVER IP: $IPERF3_SERVER_IP)"
         killall iperf3 2> /dev/null || true
-        iperf3 -V -s -D -f M $BENCHMARK_IPERF3_SERVER_PARAMS
+        iperf3 -V -s -D $BENCHMARK_IPERF3_SERVER_PARAMS
     fi
 
     echo "Starting iperf3 client on the target (CLIENT IP: $IPADDR)"
diff --git a/engine/tests/Benchmark.iperf3/parser.py b/engine/tests/Benchmark.iperf3/parser.py
index 1097803..a91b88a 100755
--- a/engine/tests/Benchmark.iperf3/parser.py
+++ b/engine/tests/Benchmark.iperf3/parser.py
@@ -51,7 +51,7 @@ with open(plib.TEST_LOG) as f:
     plt.plot(time, bits_per_second)
     plt.title('iperf3 client results')
     plt.xlabel('time (s)')
-    plt.ylabel('Mbps')
+    plt.ylabel('Bits per second (bps)')
     fig.savefig(os.environ['LOGDIR'] + '/iperf3.png')
 
 sys.exit(plib.process(measurements))
diff --git a/engine/tests/Benchmark.iperf3/reference.json b/engine/tests/Benchmark.iperf3/reference.json
index 930a94d..f10dc3a 100644
--- a/engine/tests/Benchmark.iperf3/reference.json
+++ b/engine/tests/Benchmark.iperf3/reference.json
@@ -8,7 +8,7 @@
                     "measurements":[
                         {
                             "name":"bits_per_second",
-                            "unit":"Mbps"
+                            "unit":"bps"
                         },
                         {
                             "name":"retransmits",
@@ -21,7 +21,7 @@
                     "measurements":[
                         {
                             "name":"bits_per_second",
-                            "unit":"Mbps"
+                            "unit":"bps"
                         }
                     ]
                 },
@@ -30,7 +30,7 @@
                     "measurements":[
                         {
                             "name":"bits_per_second",
-                            "unit":"Mbps"
+                            "unit":"bps"
                         },
                         {
                             "name":"jitter_ms",
-- 
2.7.4




More information about the Fuego mailing list