From 08e7b8758cf60f7f5974d66df3faf63650ca4bd6 Mon Sep 17 00:00:00 2001 From: Peter Mikus Date: Fri, 5 Jul 2019 05:08:13 +0000 Subject: Remove pkt_trace as global variable - Use explicit parameters Change-Id: I21cbc8cf13ec5e455b20c685c4427a53a655f71f Signed-off-by: Peter Mikus --- resources/libraries/robot/performance/performance_utils.robot | 5 +++-- resources/libraries/robot/shared/test_teardown.robot | 3 +-- 2 files changed, 4 insertions(+), 4 deletions(-) (limited to 'resources/libraries/robot') diff --git a/resources/libraries/robot/performance/performance_utils.robot b/resources/libraries/robot/performance/performance_utils.robot index 71e20afc72..8507ae68f5 100644 --- a/resources/libraries/robot/performance/performance_utils.robot +++ b/resources/libraries/robot/performance/performance_utils.robot @@ -363,15 +363,16 @@ | | ... | - unidirection - False if traffic is bidirectional. Type: boolean | | ... | - tx_port - TX port of TG, default 0. Type: integer | | ... | - rx_port - RX port of TG, default 1. Type: integer +| | ... | - pkt_trace - True to enable packet trace. Type: boolean | | ... | | ... | *Example:* | | ... | | ... | \| Send traffic at specified rate \| ${1.0} \| 4.0mpps \| 64 \ -| | ... | \| 3-node-IPv4 \| ${10} \| ${False} \| ${0} | ${1} \| +| | ... | \| 3-node-IPv4 \| ${10} \| ${False} \| ${0} | ${1} \| ${False} | | ... | | [Arguments] | ${trial_duration} | ${rate} | ${frame_size} | | ... | ${traffic_profile} | ${subsamples}=${1} | ${unidirection}=${False} -| | ... | ${tx_port}=${0} | ${rx_port}=${1} +| | ... | ${tx_port}=${0} | ${rx_port}=${1} | ${pkt_trace}=${False} | | ... | | Clear and show runtime counters with running traffic | ${trial_duration} | | ... | ${rate} | ${frame_size} | ${traffic_profile} diff --git a/resources/libraries/robot/shared/test_teardown.robot b/resources/libraries/robot/shared/test_teardown.robot index f95e369b65..cec0655e19 100644 --- a/resources/libraries/robot/shared/test_teardown.robot +++ b/resources/libraries/robot/shared/test_teardown.robot @@ -44,10 +44,9 @@ | | [Documentation] | | ... | Additional teardown for tests which uses performance measurement. | | ... -| | Set Test Variable | ${pkt_trace} | ${True} | | Run Keyword If Test Failed | | ... | Send traffic at specified rate | ${perf_trial_duration} | 10000pps -| | ... | ${frame_size} | ${traffic_profile} +| | ... | ${frame_size} | ${traffic_profile} | pkt_trace=${True} | Additional Test Tear Down Action For packet_trace | | [Documentation] -- cgit 1.2.3-korg