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