LAVA Dispatcher

Merge lp://staging/~qzhang/lava-dispatcher/fix-844299-844301-844462 into lp://staging/lava-dispatcher

fix-844299-844301-844462
Merge into trunk

Proposed by Spring Zhang on 2011-09-13

Status:

Merged

Merged at revision:

112

Proposed branch:

lp://staging/~qzhang/lava-dispatcher/fix-844299-844301-844462

Merge into:

lp://staging/lava-dispatcher

Diff against target:

108 lines (+20/-12)

3 files modified

lava_dispatcher/actions/boot_control.py (+8/-4)
lava_dispatcher/actions/deploy.py (+9/-2)
lava_dispatcher/actions/lava-test.py (+3/-6)

To merge this branch:

bzr merge lp://staging/~qzhang/lava-dispatcher/fix-844299-844301-844462

Related bugs:

Bug #844299: dispatcher detects failure to install lava-test, still tries to run tests	Medium	Fix Released
Bug #844301: dispatcher didn't detect deployment failure	Medium	Fix Released
Bug #844462: Failing to boot the image should be a critical error	High	Fix Released

Link a bug report

Reviewer	Review Type	Date Requested	Status
Paul Larson (community)		2011-09-13	Approve on 2011-09-16
Review via email: mp+75124@code.staging.launchpad.net

Description of the change

Fix three bugs: bug 844299, bug 844301, bug 844462

Revision history for this message

Paul Larson (pwlars) wrote on 2011-09-16:

35 - self.context.test_data.add_result("boot_image", status)
36 + self.context.test_data.add_result("boot_linaro_image", status)
Would like to keep this as-is actually. iirc we use this somewhere.

I fixed this and a few tweaks to the error messages. Looks good, merging now. Thanks!

review: Approve