UTAH

utah crashes instead of producing nice error message

Bug #1195469 reported by Thomi Richards
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
UTAH
Fix Released
Medium
Max Brustkern
UTAH 0.14

Bug Description

I've obviously done something wrong, but UTAH raises an uncaught exception, rather than producing a nice error message. The exception stack trace is:

WARNING: xml was already set by /etc/utah/configWARNING: outputpreseed was already set by /etc/utah/configWARNING: installtype was already set by /etc/utah/configUnhandled error in UTAH:
Traceback (most recent call last):
  File "/usr/bin/run_utah_tests.py", line 128, in <module>
    exitstatus = timeout(config.jobtimeout, run_utah_tests)
  File "/usr/lib/python2.7/dist-packages/utah/timeout.py", line 65, in timeout
    return command(*args, **kw)
  File "/usr/bin/run_utah_tests.py", line 117, in run_utah_tests
    exitstatus, locallogs = run_tests(args, _get_machine(args))
  File "/usr/bin/run_utah_tests.py", line 92, in _get_machine
    return _get_unprovisioned_machine(args, image)
  File "/usr/bin/run_utah_tests.py", line 72, in _get_unprovisioned_machine
    return get_baremetal(args.arch, **kw)
  File "/usr/lib/python2.7/dist-packages/utah/provisioning/baremetal/__init__.py", line 64, in get_baremetal
    return inventory.request(**kw)
  File "/usr/lib/python2.7/dist-packages/utah/provisioning/baremetal/inventory.py", line 81, in request
    return self._take(machineinfo, machinetype, *args, **kw)
  File "/usr/lib/python2.7/dist-packages/utah/provisioning/baremetal/inventory.py", line 106, in _take
    machineinfo=machineinfo, name=name, **kw)
  File "/usr/lib/python2.7/dist-packages/utah/provisioning/baremetal/cobbler.py", line 60, in __init__
    self._cmdlinesetup()
  File "/usr/lib/python2.7/dist-packages/utah/provisioning/provisioning.py", line 940, in _cmdlinesetup
    if self.image.installtype == 'desktop':
AttributeError: 'str' object has no attribute 'installtype'

Related branches

lp:~nuclearbob/utah/bug1195469
Andy Doan (community): Approve
Diff: 25 lines (+3/-1)
2 files modified
utah/config.py (+1/-1)
utah/url.py (+2/-0)
lp:utah
Max Brustkern: Pending requested
Diff: 10666 lines (+6316/-1397)
85 files modified
COPYING (+674/-0)
Makefile (+1/-1)
client-Makefile (+4/-1)
client.py (+15/-0)
conf/utah/uqt-vm-tools.conf (+1/-1)
debian/changelog (+101/-0)
debian/control (+17/-3)
debian/rules (+21/-2)
debian/utah.install (+2/-0)
debian/utah.postinst (+10/-2)
distribute_setup.py (+3/-1)
docs/source/conf.py (+42/-23)
docs/source/reference.rst (+17/-0)
examples/run_install_test.py (+43/-18)
examples/run_test_bamboo_feeder.py (+123/-0)
examples/run_test_cobbler.py (+55/-25)
examples/run_test_vm.py (+38/-14)
examples/run_utah_tests.py (+51/-31)
examples/utah_logs.sh (+3/-1)
examples/utah_syslog.sh (+1/-1)
setup.py (+20/-6)
tests/test_parser.py (+60/-0)
tests/test_preseed.py (+476/-33)
utah/__init__.py (+15/-0)
utah/client/__init__.py (+15/-0)
utah/client/common.py (+70/-32)
utah/client/examples/test_state_partial_all_failed.yaml (+90/-0)
utah/client/examples/test_state_partial_inprogress.yaml (+90/-0)
utah/client/examples/test_state_partial_run_all.yaml (+90/-0)
utah/client/exceptions.py (+15/-0)
utah/client/phoenix.py (+40/-25)
utah/client/result.py (+48/-22)
utah/client/runner.py (+66/-58)
utah/client/state_agent.py (+15/-0)
utah/client/testcase.py (+73/-51)
utah/client/tests/__init__.py (+14/-0)
utah/client/tests/common.py (+43/-0)
utah/client/tests/manual_privileges.py (+17/-2)
utah/client/tests/test_common.py (+22/-7)
utah/client/tests/test_jsonschema.py (+21/-49)
utah/client/tests/test_misc.py (+15/-0)
utah/client/tests/test_phoenix.py (+25/-10)
utah/client/tests/test_result.py (+22/-5)
utah/client/tests/test_runner.py (+22/-7)
utah/client/tests/test_state_agent.py (+134/-68)
utah/client/tests/test_testcase.py (+46/-31)
utah/client/tests/test_testsuite.py (+45/-23)
utah/client/tests/test_vcs_bzr.py (+15/-0)
utah/client/tests/test_vcs_dev.py (+15/-0)
utah/client/tests/test_vcs_git.py (+15/-0)
utah/client/tests/test_yaml.py (+51/-36)
utah/client/testsuite.py (+31/-13)
utah/commandstr.py (+15/-0)
utah/config.py (+192/-18)
utah/exceptions.py (+15/-12)
utah/group.py (+15/-0)
utah/iso.py (+277/-118)
utah/isotest/data/file_list_di (+23/-23)
utah/isotest/data/file_list_ubiquity (+21/-21)
utah/isotest/iso_static_validation.py (+85/-60)
utah/orderedcollections.py (+90/-0)
utah/parser.py (+211/-0)
utah/preseed.py (+669/-119)
utah/process.py (+20/-5)
utah/provisioning/__init__.py (+15/-0)
utah/provisioning/baremetal/__init__.py (+15/-0)
utah/provisioning/baremetal/bamboofeeder.py (+301/-0)
utah/provisioning/baremetal/cobbler.py (+124/-130)
utah/provisioning/baremetal/exceptions.py (+15/-0)
utah/provisioning/baremetal/power.py (+90/-0)
utah/provisioning/exceptions.py (+15/-0)
utah/provisioning/inventory/__init__.py (+15/-0)
utah/provisioning/inventory/exceptions.py (+15/-0)
utah/provisioning/inventory/inventory.py (+20/-5)
utah/provisioning/inventory/sqlite.py (+87/-52)
utah/provisioning/provisioning.py (+337/-107)
utah/provisioning/vm/__init__.py (+15/-0)
utah/provisioning/vm/exceptions.py (+15/-0)
utah/provisioning/vm/libvirtvm.py (+124/-70)
utah/provisioning/vm/vm.py (+16/-0)
utah/publish.py (+146/-0)
utah/retry.py (+15/-0)
utah/run.py (+142/-34)
utah/timeout.py (+21/-4)
utah/url.py (+182/-17)
Revision history for this message
Thomi Richards (thomir-deactivatedaccount) wrote :

This happens when passing -i "" on the command line.

Andy Doan (doanac)
Changed in utah:
importance: Undecided → Medium
assignee: nobody → Max Brustkern (nuclearbob)
Revision history for this message
Max Brustkern (nuclearbob) wrote :

That's interesting. We can handle None, but not an empty string. I'll prepare a fix for that.

Max Brustkern (nuclearbob)
Changed in utah:
status: New → Fix Committed
Andy Doan (doanac)
Changed in utah:
milestone: none → 0.14
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

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