commit dd20dc6a0062ef87620977b6cc3b41da2d3d46e3
author You Wang <you@opennetworking.org> Tue Feb 19 14:06:19 2019 -0800
committer You Wang <you@opennetworking.org> Thu Feb 28 23:02:36 2019 +0000
tree cc06fce5b8796302636a32a1341c774aac9bd0c7
parent 6f373975291f61c99274e7764bd5bf0312aa13dd

Exclude invalid latency numbers from intent reroute test results

Change-Id: I247f7d6e0179d978f7e4706745f07b213c7afade
(cherry picked from commit fea75169bd97475de441ec6b624df347f4f02d03)

TestON/tests/SCPF/SCPFintentRerouteLat/SCPFintentRerouteLat.params

diff --git a/TestON/tests/SCPF/SCPFintentRerouteLat/SCPFintentRerouteLat.params b/TestON/tests/SCPF/SCPFintentRerouteLat/SCPFintentRerouteLat.params
index 7501bf5..d4d1971 100644
--- a/TestON/tests/SCPF/SCPFintentRerouteLat/SCPFintentRerouteLat.params
+++ b/TestON/tests/SCPF/SCPFintentRerouteLat/SCPFintentRerouteLat.params
@@ -27,6 +27,7 @@
 
     <TEST>
         <warmUp>5</warmUp>
+        <maxLatency>1000</maxLatency>
         <sampleSize>20</sampleSize>
         <intents>1,100,1000</intents>                       #list format, will be split on ','
         <FObjintents>1,100</FObjintents>

TestON/tests/SCPF/SCPFintentRerouteLat/SCPFintentRerouteLat.py

diff --git a/TestON/tests/SCPF/SCPFintentRerouteLat/SCPFintentRerouteLat.py b/TestON/tests/SCPF/SCPFintentRerouteLat/SCPFintentRerouteLat.py
index 7a8a294..40790fb 100644
--- a/TestON/tests/SCPF/SCPFintentRerouteLat/SCPFintentRerouteLat.py
+++ b/TestON/tests/SCPF/SCPFintentRerouteLat/SCPFintentRerouteLat.py
@@ -282,8 +282,10 @@
 
                     firstLocalLatnecy, lastLocalLatnecy, firstGlobalLatency, lastGlobalLatnecy \
                         = main.intentRerouteLatFuncs.calculateLatency( main, topologyTimestamps, intentTimestamps )
-                    if firstLocalLatnecy < 0:
-                        main.log.info( "Got negative latency, skipping this iteration..." )
+                    maxLatency = float( main.params[ 'TEST' ][ 'maxLatency' ] )
+                    if firstLocalLatnecy < 0 or lastLocalLatnecy < 0 or \
+                       firstLocalLatnecy > maxLatency or lastLocalLatnecy > maxLatency:
+                        main.log.info( "Got invalid latency, skipping this iteration..." )
                         main.intentRerouteLatFuncs.bringBackTopology( main )
                         if main.validRun >= main.warmUp:
                             main.invalidRun += 1