diff mbox series

[V1] tests/ice_dcf_date_path: turn off promisc mode

Message ID 20220519015545.1805024-1-songx.jiale@intel.com (mailing list archive)
State Superseded
Headers
Series [V1] tests/ice_dcf_date_path: turn off promisc mode |

Checks

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

Commit Message

Jiale, SongX May 19, 2022, 1:55 a.m. UTC 
  DCF already supports promise mode, and it is started by default.
promisc should be turned off during the test.

According to dpdk commit cdfbcfa110a ("net/ice: support DCF promiscuous configuration")

Signed-off-by: Jiale Song <songx.jiale@intel.com>
---
 tests/TestSuite_ice_dcf_date_path.py | 1 +
 1 file changed, 1 insertion(+)

Comments

Tu, Lijuan May 25, 2022, 10:19 a.m. UTC | #1 
> -----Original Message-----
> From: Jiale Song <songx.jiale@intel.com>
> Sent: 2022年5月19日 9:56
> To: dts@dpdk.org
> Cc: Jiale, SongX <songx.jiale@intel.com>
> Subject: [dts] [PATCH V1] tests/ice_dcf_date_path: turn off promisc mode
> 
> DCF already supports promise mode, and it is started by default.
> promisc should be turned off during the test.
> 
> According to dpdk commit cdfbcfa110a ("net/ice: support DCF promiscuous configuration")
> 
> Signed-off-by: Jiale Song <songx.jiale@intel.com>

Test plan should be modified accordingly.

And is only this case impact? If yes, the reason should be put in the test plan too.
diff mbox series

Patch

diff --git a/tests/TestSuite_ice_dcf_date_path.py b/tests/TestSuite_ice_dcf_date_path.py
index aad27f5b..21a0c875 100644
--- a/tests/TestSuite_ice_dcf_date_path.py
+++ b/tests/TestSuite_ice_dcf_date_path.py
@@ -283,6 +283,7 @@  class DcfDatePathTest(TestCase):
         )
 
         self.pmd_output.execute_cmd("set fwd rxonly")
+        self.pmd_output.execute_cmd("set promisc all off")
         self.pmd_output.execute_cmd("set verbose 1")
         self.pmd_output.execute_cmd("start")
         out = self.send_pkts_getouput(pkt, self.tester_intf0, status=True)