cpu_topology crashed with KeyError in main(): 'physical_package_id'

Bug #1111878 reported by Msirdian
14
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Checkbox
Fix Released
Undecided
Unassigned
checkbox (Ubuntu)
Fix Released
Medium
Jeff Lane 

Bug Description

ran system testing
got error report

ProblemType: Crash
DistroRelease: Ubuntu 13.04
Package: checkbox 0.15.1
ProcVersionSignature: Ubuntu 3.8.0-2.6-generic 3.8.0-rc4
Uname: Linux 3.8.0-2-generic i686
ApportVersion: 2.8-0ubuntu3
Architecture: i386
Date: Thu Jan 31 22:29:05 2013
ExecutablePath: /usr/lib/checkbox/bin/cpu_topology
InstallationDate: Installed on 2012-11-12 (80 days ago)
InstallationMedia: Ubuntu 13.04 "Raring Ringtail" - Alpha i386 (20121112.1)
InterpreterPath: /usr/bin/python3.3
MarkForUpload: True
ProcCmdline: /usr/bin/python3 /usr/share/checkbox/scripts/cpu_topology
ProcEnviron:
 SHELL=/bin/bash
 PATH=(custom, no user)
 LANG=en_US.UTF-8
 PYTHONPATH=:/usr/share/checkbox
 XDG_RUNTIME_DIR=<set>
PythonArgs: ['/usr/share/checkbox/scripts/cpu_topology']
SourcePackage: checkbox
Title: cpu_topology crashed with KeyError in main(): 'physical_package_id'
UpgradeStatus: No upgrade log present (probably fresh install)
UserGroups: adm cdrom dip lpadmin plugdev sambashare sudo

Related branches

Revision history for this message
Msirdian (arena7) wrote :
Revision history for this message
Msirdian (arena7) wrote :

ran system testing
got error report

tags: removed: need-duplicate-check
Changed in checkbox (Ubuntu):
importance: Undecided → Medium
Revision history for this message
Jeff Lane  (bladernr) wrote :

this seems to be a bit of a corner case. Script doesn't pre-fill the cpuinfo dicts, apparently the submitter has different output in proc/cpuinfo than I do. It would be nice to see the output of /proc/cpuinfo.

So, can you please provide the output of /proc/cpuinfo on your machine that shows this failure?

And can you tell me more about that machine? What CPU does it have? What make/model is the machine in question?

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

OK, so I pushed a branch that should resolve the key error problem. I was not able to actually recreate your failure, however, so I am really only taking a best guess based on the code and the error. So as asked above, could you please provide the contents of /proc/cpuinfo:

cat /proc/cpuinfo > cpuinfo.txt

and tell me more about the system you're running this on and encountering the error.

Jeff Lane  (bladernr)
information type: Private → Public
Jeff Lane  (bladernr)
Changed in checkbox:
status: New → Incomplete
Revision history for this message
Launchpad Janitor (janitor) wrote :
Download full text (7.8 KiB)

This bug was fixed in the package checkbox - 0.16

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

  * New upstream release (LP: #1178403):

  * Launchpad automated translation updates

  [ Jeff Lane ]
  * 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/sleep_test_log_check: added new script to parse fwts logs for
    errors
    jobs/hibernate.txt.in, jobs/stress.txt.in, jobs/suspend.txt.in: added jobs
    to use the sleep_test_log_check script after s3/s4 tests. Modified current
    s3/s4 tests to use the new None fail level for fwts_test
    scripts/fwts_test: no longer fails on fwts errors if "-f none" is chosen.
    Now it should only fail by manual indication if the actual sleep action
    fails. (LP: #1169922)
  * scripts/bluetooth_test: test no longer generates odd Broken Pipe error when
    the dpkg bit verifies obexftp is installed (LP: #1169488)
  * jobs/networking.txt.in: fixed networking/info jobs that were not being run
    because $output was being resolved too early (LP: #1065983)
  * scripts/memory_compare: cleaned up the output to be more explanatory and
    easier to visually parse (LP: #1072666)
  * jobs/miscellanea.txt.in: Added jobs for manual verification of PXE boot and
    remote IPMI to improve server test coverage.
  * jobs/led.txt.in: Modified Jobs: led/power, led/wlan, led/wlan-disabled. New
    jobs: led/power-blink-suspend, led/suspend, led/mute
    jobs/keys.txt.in: modified keys/wireless
  * plugins/lock_prompt.py: added a lock release to cleanly clear lock on stop.
    plugins/persist_prompt.py: promoted save to run before lock release.
    checkbox/lib/fifo.py: trap OSError exception at close when the input/output
    fifo fds disappear before fifo.close() can get to them. (LP: #115561)
  * jobs/info.txt.in: fixed bad driver name in audio_codecs job (LP: #1165215)
  * jobs/mediacard.txt.in: reduce test file size for MMC to 64MB (LP: #1167214)
  * jobs/led.txt.in, jobs/suspend.txt.in: removed bits about WLAN LED flashing
    from the WLAN jobs as that no longer happens on data tx/rx (LP: #1157606)
  * jobs/power-management.txt.in: added manual reboot and shutdown tests for
    server and Xen testing. Took the opportunity to correct punctuation in the
    descriptions or other manual jobs.
    jobs/virtualization.txt.in: added virtualization/x...

Read more...

Changed in checkbox (Ubuntu):
status: Incomplete → Fix Released
piotr zimoch (ebytyes)
Changed in checkbox:
status: Incomplete → New
status: New → Incomplete
status: Incomplete → Opinion
status: Opinion → Invalid
status: Invalid → Confirmed
status: Confirmed → In Progress
status: In Progress → Fix Committed
status: Fix Committed → 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.