[V1] Add timeout parameter and default value for start_testpmd API

Message ID 20220316065704.1080-1-junx.dong@intel.com (mailing list archive)
State Accepted
Headers
Series [V1] Add timeout parameter and default value for start_testpmd API |

Checks

Context Check Description
ci/Intel-dts-suite-test success Testing OK

Commit Message

Jun Dong March 16, 2022, 6:57 a.m. UTC
  Signed-off-by: Jun Dong <junx.dong@intel.com>
---
 framework/pmd_output.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
  

Comments

Tu, Lijuan March 23, 2022, 7:04 a.m. UTC | #1
On Wed, 16 Mar 2022 14:57:04 +0800, Jun Dong <junx.dong@intel.com> wrote:
> Signed-off-by: Jun Dong <junx.dong@intel.com>


Applied, thanks
  

Patch

diff --git a/framework/pmd_output.py b/framework/pmd_output.py
index bb0fa04b..f27c2513 100644
--- a/framework/pmd_output.py
+++ b/framework/pmd_output.py
@@ -112,6 +112,7 @@  class PmdOutput:
         socket=0,
         fixed_prefix=False,
         expected="testpmd> ",
+        timeout=120,
         **config,
     ):
         """
@@ -170,7 +171,6 @@  class PmdOutput:
         command = command.replace("  ", " ")
         if self.session != self.dut:
             self.session.send_expect("cd %s" % self.dut.base_dir, "# ")
-        timeout = config.get("timeout", 120)
         out = self.session.send_expect(command, expected, timeout)
         self.command = command
         # wait 10s to ensure links getting up before test start.