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

Saggi Mizrahi has posted comments on this change.

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

Patch Set 12: Code-Review+1

I approve this as the major issue preventing people from grasping the change is that the original design of the module is backwards.

The whole idea of passing "target" is odd and makes everything look weird.

get() should just give an instance proxy object and than we use standard ways to share the proxy.

This proxy, if existed, would have done these checks and invalidated the connection it proxies.

The immediate next phase would be to change the design to that model so that future logic has a place to reside in.