Change in vdsm[master]: virt: Handle disconnects only if endpoints match

vfeenstr at redhat.com vfeenstr at redhat.com
Thu Aug 6 11:47:11 UTC 2015


Vinzenz Feenstra has posted comments on this change.

Change subject: virt: Handle disconnects only if endpoints match
......................................................................


Patch Set 2:

The code does now only trigger the disconnects once and only on known IPs

libvirtEventLoop::DEBUG::2015-08-06 13:41:55,330::clientIF::584::virt.vm::(dispatchLibvirtEvents) vmId=`13ec279a-a895-404f-9d38-42136aad1517`::graphics event phase 2 localAddr {'node': '127.0.0.1', 'service': '', 'family': 0} remoteAddr {'node': '127.0.0.1', 'service': ''
, 'family': 0}authScheme  subject []
libvirtEventLoop::DEBUG::2015-08-06 13:41:55,330::vm::920::virt.vm::(onDisconnect) vmId=`13ec279a-a895-404f-9d38-42136aad1517`::Ignoring disconnect event because ip differs

On correct disconnects handling is performed as before, but only once (Which is the expected result)

-- 
To view, visit https://gerrit.ovirt.org/43141
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Idfb3441407aa6e7bc31d37304bce0076e12bb1a3
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Vinzenz Feenstra <vfeenstr at redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek at redhat.com>
Gerrit-Reviewer: Vinzenz Feenstra <vfeenstr at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-HasComments: No


More information about the vdsm-patches mailing list