diff options
| author |   Vratko Polak <vrpolak@cisco.com> | 2021-06-01 11:09:58 +0200 |
|---|---|---|
| committer | Vratko Polak <vrpolak@cisco.com> | 2021-06-01 11:09:58 +0200 |
| commit | 3793e1525f12ca1bdbd2140254eae31535818c27 (patch) | |
| tree | 2fa1b9f0fd890f665da97cc791f7576bc3cbd1f8 /GPL | |
| parent | 1d8887789f98551aac51571b8c4c0c3a18612203 (diff) | |
ASTF: Fix syntax error in stop
When ASTF suites were introduced,
they already edited [0] actions to avoid explicit stop.
Then a refactor [1] introduced the syntax error,
but as stop never run, we didnot see it as a failure.
Finally another refactor [2] caused the stop to be called again,
exposing the syntax error.
As a related issue, pylint script was not updated to check files
in GPL directory until [3].
[0] https://gerrit.fd.io/r/c/csit/+/28205/8/tests/vpp/perf/ip4/2n1l-10ge2p1x710-ethip4udp-ip4base-h1024-p63-s64512-ndrpdr.robot#84
[1] https://gerrit.fd.io/r/c/csit/+/28208/179/GPL/tools/trex/trex_astf_stop.py#102
[2] https://gerrit.fd.io/r/c/csit/+/32030/37/resources/libraries/robot/performance/performance_utils.robot#518
[3] https://gerrit.fd.io/r/c/csit/+/32504
Change-Id: Ibd20b43cb263170d9f17255ff1bcd20f09c42cb5
Signed-off-by: Vratko Polak <vrpolak@cisco.com>
Diffstat (limited to 'GPL')
| -rw-r--r-- | GPL/tools/trex/trex_astf_stop.py | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/GPL/tools/trex/trex_astf_stop.py b/GPL/tools/trex/trex_astf_stop.py index b214d58a2f..1db122ba0f 100644 --- a/GPL/tools/trex/trex_astf_stop.py +++ b/GPL/tools/trex/trex_astf_stop.py @@ -109,8 +109,8 @@ def main(): f"frame_loss={total_lost}; " f"latency_stream_0(usec)=-1/-1/-1; " f"latency_stream_1(usec)=-1/-1/-1; " - f"latency_hist_stream_0={}; " - f"latency_hist_stream_1={}; " + f"latency_hist_stream_0=; " + f"latency_hist_stream_1=; " ) |