Merge lp:~newell-jensen/maas/fix-1690781 into lp:~maas-committers/maas/trunk
Proposed by
Newell Jensen
Status: | Merged |
---|---|
Approved by: | Newell Jensen |
Approved revision: | no longer in the source branch. |
Merged at revision: | 6056 |
Proposed branch: | lp:~newell-jensen/maas/fix-1690781 |
Merge into: | lp:~maas-committers/maas/trunk |
Diff against target: |
35 lines (+14/-0) 2 files modified
src/provisioningserver/drivers/pod/tests/test_virsh.py (+4/-0) src/provisioningserver/drivers/pod/virsh.py (+10/-0) |
To merge this branch: | bzr merge lp:~newell-jensen/maas/fix-1690781 |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Данило Шеган (community) | Approve | ||
MAAS Maintainers | Pending | ||
Review via email: mp+324220@code.launchpad.net |
Commit message
Raises VirshError if the `virsh domcapabilities --virttype kvm|qemu` command returns empty XML with a suggestion on how to potentially fix the situation.
To post a comment you must log in.
I was unable to reproduce this bug following the steps outlined in the bug report. Regardless of this fact, it is a good idea to test to see whether or not the XML is empty from running `virsh domcapabilities --virttype kvm|qemu` as a precaution since the bug reporters has run into this.