test suite reviews and discussions
 help / color / mirror / Atom feed
* [dts] [PATCH V1] tests/userspace_ethtool: cannot change CVL link status by the peer port
@ 2019-10-31 14:07 Wenjie Li
  2019-10-31  5:22 ` Zhu, WenhuiX
  2019-11-22  2:55 ` Tu, Lijuan
  0 siblings, 2 replies; 3+ messages in thread
From: Wenjie Li @ 2019-10-31 14:07 UTC (permalink / raw)
  To: dts; +Cc: Wenjie Li

In case dump_driver_info, like fortville, CVL link status cannot be changed by the peer ports.
After change the peer port status by running "ip link set dev PORT
down|up", CVL link status will not change. So skip this step.

Signed-off-by: Wenjie Li <wenjiex.a.li@intel.com>
---
 tests/TestSuite_userspace_ethtool.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tests/TestSuite_userspace_ethtool.py b/tests/TestSuite_userspace_ethtool.py
index 116fd3a..8d18b0c 100644
--- a/tests/TestSuite_userspace_ethtool.py
+++ b/tests/TestSuite_userspace_ethtool.py
@@ -259,7 +259,7 @@ class TestUserspaceEthtool(TestCase, IxiaPacketGenerator):
         # only detect physical link disconnect status
         verify_pass = True
         verify_msg = ''
-        if self.nic.startswith("fortville") == False:  
+        if self.nic.startswith("fortville") == False and self.nic.startswith("columbiaville") == False:
             # check link status dump function
             for port in self.ports:
                 tester_port = self.tester.get_local_port(port)
-- 
2.17.1


^ permalink raw reply	[flat|nested] 3+ messages in thread
end of thread, other threads:[~2019-11-22  2:56 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-10-31 14:07 [dts] [PATCH V1] tests/userspace_ethtool: cannot change CVL link status by the peer port Wenjie Li
2019-10-31  5:22 ` Zhu, WenhuiX
2019-11-22  2:55 ` Tu, Lijuan

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).