mooli tayer has posted comments on this change.
Change subject: libvirtconnection: ping libvirt upon disconnection
......................................................................
Patch Set 12: Verified-1
What I did to test this patch:
a.) Wrote unit tests
b.) Ran cross migration between two hosts running a total of 4 vms. I installed them from
ovirt-stable repo and replaced libvirtconnection.py
I discovered that in patch set 11 and before the whole thing was one big bug:
I wrote wrapper like so:
def wrapper(pingLibvirt=True, *args, **kwargs):
...
ret = f(*args, **kwargs)
and the user of this class called this wrapper for example:
conn.nwfilterLookupByName('someArg')
That would lead to:
pingLibvirt='someArg'
And worse args = []
Shame on me!
After the fix of this patch set the migration still does not work. not sure if it is
related to this patch set or not.
one host becomes non operational (the engine runs getCaps on it and gives the error:
'Host host_01 running without virtualization hardware acceleration' ) still
checking this
--
To view, visit
http://gerrit.ovirt.org/19444
To unsubscribe, visit
http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ia489e46dd8ce4c70c888988d17b86311d3c4b935
Gerrit-PatchSet: 12
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: mooli tayer <mtayer(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Saggi Mizrahi <smizrahi(a)redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei(a)redhat.com>
Gerrit-Reviewer: mooli tayer <mtayer(a)redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No