removable_storage_test crashed with OSError in write_file(): [Errno 30] Read-only file system:

Bug #1153894 reported by Jonathan Brier
10
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Checkbox Provider - Base
Fix Released
Undecided
Unassigned
checkbox (Ubuntu)
Fix Released
Medium
Jeff Lane 

Bug Description

After conducting the system test for the SD card Insert and removal this report appeared stating an error. (I skipped the MMC card test for my system).

Expected no errors in testing on Raring 13.04.

checkbox version 0.15.4 installed

ProblemType: Crash
DistroRelease: Ubuntu 13.04
Package: checkbox 0.15.4
ProcVersionSignature: Ubuntu 3.8.0-12.21-generic 3.8.2
Uname: Linux 3.8.0-12-generic x86_64
ApportVersion: 2.9.1-0ubuntu1
Architecture: amd64
CrashCounter: 1
Date: Tue Mar 12 00:12:50 2013
ExecutablePath: /usr/lib/checkbox/bin/removable_storage_test
InstallationDate: Installed on 2012-08-17 (206 days ago)
InstallationMedia: Ubuntu 12.04 LTS "Precise Pangolin" - Release amd64 (20120425)
InterpreterPath: /usr/bin/python3.3
MarkForUpload: True
ProcCmdline: /usr/bin/python3 /usr/share/checkbox/scripts/removable_storage_test --memorycard sdio usb scsi
ProcEnviron:
 TERM=unknown
 SHELL=/bin/bash
 PATH=(custom, no user)
 LANG=en_US.UTF-8
 PYTHONPATH=/usr/share/checkbox
PythonArgs: ['/usr/share/checkbox/scripts/removable_storage_test', '--memorycard', 'sdio', 'usb', 'scsi']
SourcePackage: checkbox
Title: removable_storage_test crashed with OSError in write_file(): [Errno 30] Read-only file system: '/media/jonathan/CANON_DC/tmp3f18uo.0'
UpgradeStatus: Upgraded to raring on 2013-03-12 (0 days ago)
UserGroups:

Related branches

Revision history for this message
Jonathan Brier (brierjon) wrote :
tags: removed: need-duplicate-check
Changed in checkbox (Ubuntu):
importance: Undecided → Medium
Revision history for this message
Jonathan Brier (brierjon) wrote :

Only occuring when using an SD card set to the locked position.

Revision history for this message
Jeff Lane  (bladernr) wrote :

Can you verify that your SD card is not locked (there should be a little slider to lock or unlock the card.

Once you've done that, can you send the output of the following after you insert the card?

/usr/share/checkbox/scripts/removable_storage_test -l --memorycard sdio usb scsi
/usr/share/checkbox/scripts/removable_storage_test -s 67120000 --memorycard sdio usb scsi

and also, just for kicks, try this:

touch /path/to/mediacard/foo.txt

obviously, use the actual path which should look something like this:

/media/USERNAME/MEDIACARDNAME/

Changed in checkbox (Ubuntu):
status: New → Incomplete
Revision history for this message
Jeff Lane  (bladernr) wrote :

OH... rubbish... I completely missed your second comment.

Yes this will happen when you use a locked media card. When the card is locked, the OS mounts it as read only, which makes running a test that actually writes data to the card rather impossible.

So, your bug is not really a bug. BUT you've pointed out a weakness in the test description. I'll amend the media card tests to be sure to specify an unlocked device explicitly.

You can ignore my requests in comment #3

Changed in checkbox (Ubuntu):
status: Incomplete → Confirmed
assignee: nobody → Jeff Lane (bladernr)
status: Confirmed → Triaged
Jeff Lane  (bladernr)
Changed in checkbox (Ubuntu):
status: Triaged → In Progress
information type: Private → Public
Jeff Lane  (bladernr)
Changed in checkbox (Ubuntu):
status: In Progress → Fix Committed
Jeff Lane  (bladernr)
Changed in checkbox:
status: New → Fix Committed
Revision history for this message
Brendan Donegan (brendan-donegan) wrote :

Fix Commited means 'in trunk' (I know it has a lot of meanings, but now we have a release process it should mean this). This hasn't been merged yet so it's not Fix Commited.

Changed in checkbox:
status: Fix Committed → In Progress
Changed in checkbox (Ubuntu):
status: Fix Committed → In Progress
Revision history for this message
Launchpad Janitor (janitor) wrote :
Download full text (3.1 KiB)

This bug was fixed in the package checkbox - 0.16.1

---------------
checkbox (0.16.1) saucy; urgency=low

  * New upstream release (LP: #1180545):

  * Launchpad automated translation updates

  [ Alberto Milone ]
  * scripts/graphics_stress_test, scripts/rotation_test: make sure to
    always reset the "screen" variable. Somehow the NVIDIA driver manages
    to make it unusable after the first time. (LP: #1172667)

  [ Brendan Donegan ]
  * checkbox/parsers/submission.py - publish kernel-release information to
    interested parties.
  * scripts/rendercheck_test - change nargs='+' to action='append' for blacklist
    option so it works as expected.
    jobs/rendercheck.txt.in - blacklist gradients test as it is known to produce
    false positives. (LP: #1093718)
  * plugins/hexr_transport.py - added plugin for submitting to HEXR and
    certification based on certify_new_transport from checkbox-certification.
    examples/checkbox-qt.ini - blacklisted hexr_transport as we won't use it
    examples/checkbox-cli.ini - blacklisted hexr_transport as we won't use it
    examples/checkbox-urwid.ini - blacklisted hexr_transport as we won't use it

  [ Daniel Manrique ]
  * Ensured that button strings from the "continue" dialog are translatable
    (LP: #1176695)

  [ Jeff Lane ]
  * checkbox/parsers/cpuinfo.py - split on first instance of ':' in cpuinfo
    output lines to avoid splitting into more than 2 items. Also fixed a pep8
    issue discovered while working on this. (LP: #1180496)
  * scripts/cpu_offlining: Modified script to no longer offline cpu0 to resolve
    a bug on ARM. Modified output so most of it is redirected to stderr for
    fail cases, we don't need that much for success cases. (LP: #1078897)
  * jobs/mediacard.txt.in: Modified test instructions to be less confusing
    (LP: #970857)
  * scripts/cpu_topology: define the cpuinfo nested dicts on creation rather
    than define elements during parsing of /proc/cpuinfo (LP: #1111878)
  * scripts/lsmod_info: Corrected error handling for the check_output() call to
    trap the correct error. (LP: #1103647)
  * jobs/camera.txt.in: removed an extraneous requres line for gir1.2
    scripts/camera_test: added code to determine what version of gst we're
    using and set video type and plugin accordingly. (LP: #1100594)
  * scripts/network_check: added ability to specify custom target URL for
    debugging failures (LP: #1128017)
  * scripts/removable_storage_test: Added error handling to trap OSError on
    non-writable media and modified output to handle subsequent
    ZeroDivisionError issues when summarizing test results.
    jobs/media.txt.in: Modified instructions for SD/SDHC to specify using
    UNLOCKED cards to avoid issues when testing read-only media (LP: #1153894)

  [ Sylvain Pineau ]
  * jobs/suspend.txt.in, scripts/gpu_test: Remove the need of running the script
    with the root user, restore the workspaces switch and the HTML5 video
    playback ; remove the extra suspend/resume (LP: #1172851)
  * checkbox/parsers/udevadm.py: Only filter devices without product AND vendor
    information (LP: #1167733)
 -- Daniel Manrique <email address hidden> Wed, 15 May 2013 16:39:12 ...

Read more...

Changed in checkbox (Ubuntu):
status: In Progress → Fix Released
Zygmunt Krynicki (zyga)
affects: checkbox → plainbox-provider-checkbox
Changed in plainbox-provider-checkbox:
status: In Progress → Fix Released
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.