Network speed incorrectly detected as "-1"

Bug #1723460 reported by Rod Smith
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Certification Docs
Fix Released
Medium
Jeff Lane 
Checkbox Provider - Base
Fix Released
Medium
Jeff Lane 
Provider for Plainbox - Canonical Certification Server
Fix Released
Medium
Jeff Lane 

Bug Description

In testing of the impending next Checkbox release, I've discovered a bug, and verified it exists in the current version (plainbox-provider-checkbox 0.39.0~ppa~ubuntu16.04.1): On jellyplum (an HP ProLiant DL385 G7), three of the four network devices fails the ethernet/multi_iperf3_nic_device#_* test with the following output:

ERROR:root:Detected link speed (-1) is lower than detected max speed (1000)
ERROR:root:Check your device configuration and try again.
ERROR:root:If you want to override and test despite this under-speed link, use
ERROR:root:the --underspeed-ok option.

For more context and details, see:

https://certification.canonical.com/hardware/201103-7388/submission/123469/

All four NICs are connected to the network and have IP addresses. The fourth NIC (the first one listed, actually) passes its test fine.

Tags: servcert-285
Rod Smith (rodsmith)
Changed in plainbox-provider-checkbox:
assignee: nobody → Rod Smith (rodsmith)
Changed in plainbox-provider-checkbox:
importance: Undecided → Medium
Revision history for this message
Jeff Lane  (bladernr) wrote :

This is a problem with how some drivers are reporting link speed via sysfs.

We check the speed entry in /sys/devices/net/DEVNAME/speed in the script. On some drivers, (10Gb and newer) some drivers are reporting -1 as the link speed.

The network script is configured to cease testing on any link speed less than advertised max.

On OUR end, we need to change this to cease on any speed less max that is < 0.

For anything 0 or less, we should assume underspeed-ok and continue testing. but still fail the test run. This will trigger a review but give us usable test data to determine if we should accept the results anyway.

Changed in plainbox-provider-checkbox:
assignee: Rod Smith (rodsmith) → Jeff Lane (bladernr)
status: New → In Progress
Jeff Lane  (bladernr)
Changed in certification-docs:
status: New → In Progress
assignee: nobody → Jeff Lane (bladernr)
importance: Undecided → Medium
Changed in plainbox-provider-certification-server:
status: New → In Progress
importance: Undecided → Medium
assignee: nobody → Jeff Lane (bladernr)
Jeff Lane  (bladernr)
Changed in certification-docs:
status: In Progress → Fix Committed
Changed in plainbox-provider-certification-server:
status: In Progress → Fix Committed
Changed in plainbox-provider-checkbox:
status: In Progress → Fix Committed
Changed in certification-docs:
status: Fix Committed → Fix Released
Changed in plainbox-provider-certification-server:
milestone: none → 0.42.0
Changed in plainbox-provider-checkbox:
milestone: none → 0.48.0
Changed in plainbox-provider-checkbox:
status: Fix Committed → Fix Released
Changed in plainbox-provider-certification-server:
status: Fix Committed → Fix Released
Jeff Lane  (bladernr)
tags: added: servcert-285
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.