removable_storage_test fails on server for USB devices
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Checkbox |
Fix Released
|
High
|
Daniel Manrique | ||
checkbox (Ubuntu) |
Fix Released
|
Undecided
|
Unassigned | ||
Precise |
Fix Released
|
Undecided
|
Unassigned |
Bug Description
[IMPACT]
* checkbox job "usb/storage-
due to the requirement of needing root privileges to mount removable usb devices.
* backport will ensure that automated storage-preinserted test within checkbox will
run and not fail immediately due to the necessity of mounting a device as user (root).
[TESTCASE]
* Do a server installation, then reboot with the USB stick plugged in.
* Install checkbox-
* Emulate the usb/storage-
checkbox-cli -W <(cat <<EOF
`head -n 14 /usr/share/
__usb__
usb/insert
usb/
EOF
)
supply root password when prompted.
* Expected result:
- Success exit code and something like this in the log (this was obtained
on a DESKTOP system):
Found the following mounted ['usb'] partitions:
/dev/sdb1 : /media/MUSHKIN
--
Running ['usb'] file transfer test for 1 iterations
Creating Temp Data file
File name is :/tmp/tmpkCn07B
File size is 1048576 bytes
Parent hash is: 2ebe1e55057a5b0
Copying /tmp/tmpkCn07B to /media/MUSHKIN
Hashing copy on /media/MUSHKIN
Hash of /media/
is 2ebe1e55057a5b0
Successfully completed 1 ['usb'] file transfer test iterations
* Actual result:
- Error exit code
mount: only root can do that
Error: can't mount /dev/sdb1
There're total 1 device(s) failed at mounting.
No device being mounted successfully for testing, aborting
[Regression Potential]
* This is a simple change to the usb/storage-
power-
the proper permissions.
* We are changing a test that used to run without
root privileges to now having them.
Related branches
- Marc Tardif (community): Approve
-
Diff: 48 lines (+6/-2)3 files modifieddebian/changelog (+4/-2)
jobs/power-management.txt.in (+1/-0)
jobs/usb.txt.in (+1/-0)
Changed in checkbox: | |
assignee: | nobody → Daniel Manrique (roadmr) |
status: | In Progress → Fix Committed |
Changed in checkbox: | |
status: | Fix Committed → Fix Released |
description: | updated |
description: | updated |
description: | updated |
Changed in checkbox (Ubuntu): | |
status: | New → Fix Released |
As an update, this mainly affects usb/storage- preinserted, which is perhaps the only USB test that runs on servers, as per this failure:
https:/ /certification. canonical. com/hardware/ 201204- 10880/submissio n/N7AU24y3iSAQf 2G/result/ test:usb% 252Fstorage- certification
preinserted:
So making that particular job definition user: root has less of an impact than initially thought. I'll propose that as a solution.