Change in vdsm[master]: virt: add and use vm.acpiReboot() method
fromani at redhat.com
fromani at redhat.com
Wed Aug 19 08:46:52 UTC 2015
Francesco Romani has posted comments on this change.
Change subject: virt: add and use vm.acpiReboot() method
......................................................................
Patch Set 3: Verified+1
verified with 34879, the same way: sent qith vdsClient reboot command ASAP after boot, intermixed with kills from kill command.
Sometimes the call blown up with
Thread-525::ERROR::2015-08-19 10:44:35,086::utils::1056::utils.Callback::(__call__) acpiCallback failed
Traceback (most recent call last):
File "/usr/lib/python2.7/site-packages/vdsm/utils.py", line 1054, in __call__
result = self.func(*self.args, **self.kwargs)
File "/usr/share/vdsm/virt/vmpowerdown.py", line 109, in acpiCallback
if response.is_error(self.vm.acpiReboot()):
File "/usr/share/vdsm/virt/vm.py", line 3818, in acpiReboot
self._dom.reboot(libvirt.VIR_DOMAIN_REBOOT_ACPI_POWER_BTN)
File "/usr/share/vdsm/virt/vm.py", line 231, in f
ret = attr(*args, **kwargs)
File "/usr/lib/python2.7/site-packages/vdsm/libvirtconnection.py", line 124, in wrapper
ret = f(*args, **kwargs)
File "/usr/lib64/python2.7/site-packages/libvirt.py", line 1897, in reboot
if ret == -1: raise libvirtError ('virDomainReboot() failed', dom=self)
libvirtError: Domain not found: no domain with matching uuid '42e30017-b29c-4646-aa75-e42c4d0fe659' (ftest1)
but this is related but different issue, which will addressed in a separate patch. This one seems to work as expected.
--
To view, visit https://gerrit.ovirt.org/44923
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I5d6dd3a81cd4ad981a839849a4a2914e35a53fa6
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: automation at ovirt.org
Gerrit-HasComments: No
More information about the vdsm-patches
mailing list