Merge lp://qastaging/~pawciobiel/testdrive/1297377-fix-kvm-img-not-found-report-error into lp://qastaging/testdrive

Proposed by pawciobiel
Status: Merged
Merged at revision: 451
Proposed branch: lp://qastaging/~pawciobiel/testdrive/1297377-fix-kvm-img-not-found-report-error
Merge into: lp://qastaging/testdrive
Diff against target: 345 lines (+108/-56)
7 files modified
bin/testdrive (+14/-9)
bin/testdrive-gtk (+18/-13)
testdrive/virt/base.py (+16/-0)
testdrive/virt/kvm.py (+24/-10)
testdrive/virt/parallels.py (+12/-10)
testdrive/virt/virtualbox.py (+23/-13)
testdrivegtk/PreferencesTestdrivegtkDialog.py (+1/-1)
To merge this branch: bzr merge lp://qastaging/~pawciobiel/testdrive/1297377-fix-kvm-img-not-found-report-error
Reviewer Review Type Date Requested Status
Andres Rodriguez Approve
Review via email: mp+234571@code.qastaging.launchpad.net

Description of the change

merge fix for lp:1297377

* fix virt.kvm so it checks if qemu-img or kvm-img is installed
* check if qemu-utils is installed when user selected kvm
* tide up virt a bit so testdrive-gtk can notify user about errors

To post a comment you must log in.
Revision history for this message
Andres Rodriguez (andreserl) wrote :

lgtm!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches