Change in vdsm[master]: libvirtconnection: ping libvirt upon disconnection

Michal Skrivanek has posted comments on this change.

Change subject: libvirtconnection: ping libvirt upon disconnection
......................................................................

Patch Set 12:

(1 comment)

....................................................
File lib/vdsm/libvirtconnection.py
Line 131:             log.debug('trying to connect libvirt')
Line 132:             conn = utils.retry(libvirtOpenAuth, timeout=10, sleep=0.2)
Line 133:             __connections[id(target)] = conn
Line 134: 
Line 135:             setattr(conn, 'pingLibvirt', getattr(conn, 'getLibVersion'))
since libvirtconnection is trying to be just a wrapper, IMHO it's better to avoid creating of new verbs...why not just use getLibVersion at line 95
Line 136:             for name in dir(libvirt.virConnect):
Line 137:                 method = getattr(conn, name)
Line 138:                 if callable(method) and name[0] != '_':
Line 139:                     setattr(conn, name, wrapMethod(method))