diff mbox series

[V1] tests/vf_pmd_stacked_bonded: optimize script

Message ID 20230410162227.833948-1-songx.jiale@intel.com (mailing list archive)
State Accepted
Headers
Series [V1] tests/vf_pmd_stacked_bonded: optimize script |

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 April 10, 2023, 4:22 p.m. UTC 
some nic do not have "link-down-on-close" flags, but they also support
synchronizing the status of the vf port and the port of the peer port.

Signed-off-by: Jiale Song <songx.jiale@intel.com>
---
 tests/TestSuite_vf_pmd_stacked_bonded.py | 10 ++++++++--
 1 file changed, 8 insertions(+), 2 deletions(-)

Comments

Tu, Lijuan April 11, 2023, 7:55 a.m. UTC | #1 
On Mon, 10 Apr 2023 16:22:27 +0000, Jiale Song <songx.jiale@intel.com> wrote:
> some nic do not have "link-down-on-close" flags, but they also support
> synchronizing the status of the vf port and the port of the peer port.
> 
> Signed-off-by: Jiale Song <songx.jiale@intel.com>


Applied, thanks
diff mbox series

Patch

diff --git a/tests/TestSuite_vf_pmd_stacked_bonded.py b/tests/TestSuite_vf_pmd_stacked_bonded.py
index 5765481e..615e9517 100644
--- a/tests/TestSuite_vf_pmd_stacked_bonded.py
+++ b/tests/TestSuite_vf_pmd_stacked_bonded.py
@@ -566,7 +566,10 @@  class TestVFPmdStackedBonded(TestCase):
         """
         Test Case: active-backup stacked bonded rx traffic with slave down
         """
-        self.verify(self.default_stats, "tester port not support '%s'" % self.flag)
+        if not self.default_stats:
+            self.logger.warning(
+                "Please ensure that the link status of the vf port can be synchronized with the peer port!!!"
+            )
         self.backup_check_traffic_with_slave_down()
 
     def test_mode_xor_rx(self):
@@ -579,7 +582,10 @@  class TestVFPmdStackedBonded(TestCase):
         """
         Test Case: balance-xor stacked bonded rx traffic with slave down
         """
-        self.verify(self.default_stats, "tester port not support '%s'" % self.flag)
+        if not self.default_stats:
+            self.logger.warning(
+                "Please ensure that the link status of the vf port can be synchronized with the peer port!!!"
+            )
         self.xor_check_stacked_rx_one_slave_down()
 
     def tear_down(self):