Server USB tests fail 100% of the time
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Checkbox Provider - Base |
Fix Released
|
Critical
|
Jeff Lane | ||
Provider for Plainbox - Canonical Certification Server |
Fix Released
|
Critical
|
Jeff Lane |
Bug Description
Unfortuately, we missed this because our test machines in the labs don't have USB sticks plugged in, and when doing this testing for cert, we use Stable.
this commit from October lived in Dev and unfortunately we didn't catch it until now:
commit 41ef34811c08df2
Author: Lukas Waymann <email address hidden>
Date: Fri Oct 30 13:33:37 2020 +0800
Skip after-suspend-
this adds manifest lines to the usb preinserted tests:
bladernr@
diff --git a/units/usb/usb.pxu b/units/usb/usb.pxu
index 0b923cb..b73d36c 100644
--- a/units/usb/usb.pxu
+++ b/units/usb/usb.pxu
@@ -244,6 +244,7 @@ requires:
cpuinfo.platform != 's390x'
package.name == 'udisks2' or snap.name == 'udisks2'
package.name == 'udisks2' or (snap.name == 'core' and int(snap.revision) >= 1804)
+ manifest.
_summary:
Test USB 2.0 or 1.1 ports
_description:
@@ -261,6 +262,7 @@ requires:
usb.usb3 == 'supported'
package.name == 'udisks2' or snap.name == 'udisks2'
package.name == 'udisks2' or (snap.name == 'core' and int(snap.revision) >= 1804)
+ manifest.
estimated_
command: removable_
_summary:
and this breaks server cert across the board.
Related branches
- Sylvain Pineau: Approve
-
Diff: 149 lines (+7/-21)7 files modifiedunits/server-full-18.04.pxu (+1/-3)
units/server-full-20.04.pxu (+1/-3)
units/server-functional-18.04.pxu (+1/-3)
units/server-functional-20.04.pxu (+1/-3)
units/server-soc-cert-18.04.pxu (+1/-3)
units/server-soc-cert-20.04.pxu (+1/-3)
units/usb-only.pxu (+1/-3)
- Sylvain Pineau: Approve
-
Diff: 56 lines (+14/-2)2 files modifiedunits/usb/test-plan.pxu (+8/-0)
units/usb/usb.pxu (+6/-2)
- Devices Certification Bot: Needs Fixing
- Sylvain Pineau: Approve
-
Diff: 150 lines (+8/-21)7 files modifiedunits/server-full-18.04.pxu (+2/-3)
units/server-full-20.04.pxu (+1/-3)
units/server-functional-18.04.pxu (+1/-3)
units/server-functional-20.04.pxu (+1/-3)
units/server-soc-cert-18.04.pxu (+1/-3)
units/server-soc-cert-20.04.pxu (+1/-3)
units/usb-only.pxu (+1/-3)
- Sylvain Pineau: Disapprove
-
Diff: 63 lines (+44/-0)2 files modifiedunits/usb/test-plan.pxu (+8/-0)
units/usb/usb.pxu (+36/-0)
Changed in plainbox-provider-checkbox: | |
assignee: | nobody → Jeff Lane (bladernr) |
status: | Confirmed → In Progress |
Changed in plainbox-provider-certification-server: | |
importance: | Undecided → Critical |
assignee: | nobody → Jeff Lane (bladernr) |
status: | New → In Progress |
Changed in plainbox-provider-certification-server: | |
milestone: | none → 0.53.0 |
Changed in plainbox-provider-checkbox: | |
milestone: | none → 0.57.0 |
Changed in plainbox-provider-certification-server: | |
status: | In Progress → Fix Released |
Changed in plainbox-provider-checkbox: | |
status: | In Progress → Fix Released |
tags: | added: servcert-267 |
I can verify that I have seen this error on multiple systems.
usb/storage- preinserted not supported blocker job cannot be started: resource expression "manifest. has_usb_ storage == 'True'" evaluates to false
usb3/storage- preinserted not supported blocker job cannot be started: resource expression "manifest. has_usb_ storage == 'True'" evaluates to false