[V1,2/2] tests/queue_start_stop: optimize test script to remove re-compile dpdk test

Message ID 20230406062844.3539-2-linglix.chen@intel.com (mailing list archive)
State Superseded
Headers
Series [V1,1/2] test_plans/queue_start_stop: optimize test_plans to remove re-compile dpdk test step |

Checks

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

Commit Message

Lingli Chen April 6, 2023, 6:28 a.m. UTC
  remove re-compile dpdk test step to sync with test_plans

Signed-off-by: Lingli Chen <linglix.chen@intel.com>
---
 tests/TestSuite_queue_start_stop.py | 43 +++--------------------------
 1 file changed, 4 insertions(+), 39 deletions(-)
  

Comments

David Marchand April 6, 2023, 7:41 a.m. UTC | #1
On Thu, Apr 6, 2023 at 9:38 AM Lingli Chen <linglix.chen@intel.com> wrote:
>
> remove re-compile dpdk test step to sync with test_plans
>
> Signed-off-by: Lingli Chen <linglix.chen@intel.com>
> ---
>  tests/TestSuite_queue_start_stop.py | 43 +++--------------------------

The dep/macfwd_log.patch file can be removed.
  
Lingli Chen April 6, 2023, 7:53 a.m. UTC | #2
> -----Original Message-----
> From: David Marchand <david.marchand@redhat.com>
> Sent: Thursday, April 6, 2023 3:42 PM
> To: Chen, LingliX <linglix.chen@intel.com>
> Cc: dts@dpdk.org; Huang, ZhiminX <zhiminx.huang@intel.com>
> Subject: Re: [dts][PATCH V1 2/2] tests/queue_start_stop: optimize test script
> to remove re-compile dpdk test
> 
> On Thu, Apr 6, 2023 at 9:38 AM Lingli Chen <linglix.chen@intel.com> wrote:
> >
> > remove re-compile dpdk test step to sync with test_plans
> >
> > Signed-off-by: Lingli Chen <linglix.chen@intel.com>
> > ---
> >  tests/TestSuite_queue_start_stop.py | 43 +++--------------------------
> 
> The dep/macfwd_log.patch file can be removed.
> 
> 
> --
> David Marchand

Yes. Thanks for the reminde.

Regards,
Chen, Lingli
  

Patch

diff --git a/tests/TestSuite_queue_start_stop.py b/tests/TestSuite_queue_start_stop.py
index 80baf8d6..cfae1ada 100644
--- a/tests/TestSuite_queue_start_stop.py
+++ b/tests/TestSuite_queue_start_stop.py
@@ -43,21 +43,7 @@  class TestQueueStartStop(TestCase):
         """
         Run before each test case.
         """
-        try:
-            patch_file = FOLDERS["Depends"] + r"/macfwd_log.patch"
-        except:
-            self.logger.warning(str(FOLDERS))
-            patch_file = r"dep/macfwd_log.patch"
-            FOLDERS["Depends"] = "dep"
-        patch_dst = "/tmp/"
-
-        # dpdk patch and build
-        try:
-            self.dut.session.copy_file_to(patch_file, patch_dst)
-            self.patch_hotfix_dpdk(patch_dst + "macfwd_log.patch", True)
-            self.dut.build_install_dpdk(self.target)
-        except Exception as e:
-            raise IOError("dpdk setup failure: %s" % e)
+        pass
 
     def check_forwarding(self, ports, nic, pktSize=64, received=True):
         self.send_packet(ports[0], ports[1], self.nic, pktSize, received)
@@ -85,20 +71,6 @@  class TestQueueStartStop(TestCase):
         else:
             self.verify(len(sniff_pkts) == 0, "stop queue not work as expected")
 
-    def patch_hotfix_dpdk(self, patch_dir, on=True):
-        """
-        This function is to apply or remove patch for dpdk.
-        patch_dir: the abs path of the patch
-        on: True for apply, False for remove
-        """
-        try:
-            if on:
-                self.dut.send_expect("patch -p0 < %s" % patch_dir, "[~|~\]]# ")
-            else:
-                self.dut.send_expect("patch -p0 -R < %s" % patch_dir, "[~|~\]]# ")
-        except Exception as e:
-            raise ValueError("patch_hotfix_dpdk failure: %s" % e)
-
     def test_queue_start_stop(self):
         """
         queue start/stop test
@@ -114,6 +86,7 @@  class TestQueueStartStop(TestCase):
             )
             time.sleep(5)
             self.dut.send_expect("set fwd mac", "testpmd>")
+            self.dut.send_expect("set verbose 1", "testpmd>")
             self.dut.send_expect("start", "testpmd>")
             self.check_forwarding([0, 0], self.nic)
         except Exception as e:
@@ -138,7 +111,7 @@  class TestQueueStartStop(TestCase):
         except Exception as e:
             raise IOError("queue start/stop forward failure: %s" % e)
         self.verify(
-            "ports 0 queue 0 receive " not in out,
+            "port 0/queue 0: received 1 packets" not in out,
             "start queue revice package failed, out = %s" % out,
         )
 
@@ -153,7 +126,7 @@  class TestQueueStartStop(TestCase):
         except Exception as e:
             raise IOError("queue start/stop forward failure: %s" % e)
         self.verify(
-            "ports 0 queue 0 receive " in out,
+            "port 0/queue 0: received 1 packets" in out,
             "start queue revice package failed, out = %s" % out,
         )
 
@@ -161,7 +134,6 @@  class TestQueueStartStop(TestCase):
         """
         Run after each test case.
         """
-        patch_dst = "/tmp/"
 
         try:
             self.dut.send_expect("stop", "testpmd>")
@@ -169,13 +141,6 @@  class TestQueueStartStop(TestCase):
         except:
             print("Failed to quit testpmd")
 
-        self.dut.kill_all()
-
-        try:
-            self.patch_hotfix_dpdk(patch_dst + "macfwd_log.patch", False)
-        except Exception as e:
-            print(("patch_hotfix_dpdk remove failure :%s" % e))
-
     def tear_down_all(self):
         """
         Run after each test suite.