xenapi: soft reboot should attempt hard reboot on failure

John, looking on the reboot() function:

def reboot(self, context, instance, network_info, reboot_type,
                      block_device_info=None, bad_volumes_callback=None)

There is a reboot_type parameter. For XenAPI this param is properly passed to underlying
VMOps class in nova/virt/xenapi/vmops.py which implements the reboot() for Xen.
So you can have, either proper SOFT or HARD reboot and this looks fine for me.

While in the nova/virt/libvirt/driver.py, if reboot_type=='SOFT' fails nova unconditionally
does the hard reboot what can be sometimes unexpected but at the end of the day we
want to reboot the VM.

Fix proposed to branch: master
Review: https://review.openstack.org/156417

Please keep in progress until patch is merged

Change abandoned by Joe Gordon (<email address hidden>) on branch: master
Review: https://review.openstack.org/156417
Reason: This review is > 4 weeks without comment, and failed Jenkins the last time it was checked. We are abandoning this for now. Feel free to reactivate the review by pressing the restore button and leaving a 'recheck' comment to get fresh test results.

Solving an inconsistency: The bug is "in progress" but without an assignee.
I set the status back to the last known before the change to "in progress".

Fix proposed to branch: master
Review: https://review.openstack.org/203513

Change abandoned by Michael Still (<email address hidden>) on branch: master
Review: https://review.openstack.org/203513
Reason: This patch has been sitting unchanged for more than 12 weeks. I am therefore going to abandon it to keep the nova review queue sane. Please feel free to restore the change if you're still working on it.

There are no currently open reviews on this bug, changing the status back to the previous state and unassigning. If there are active reviews related to this bug, please include links in comments.