Dan Kenigsberg has submitted this change and it was merged. Change subject: vdsm: Try to reconnect on vmchannel after errors ......................................................................
vdsm: Try to reconnect on vmchannel after errors Currently the vm channel listener is not handling any errors. If an error occurred we would not try to reconnect and the connection to the guest is lost for the lifetime of the guest or until VDSM gets restarted. To handle this situation this patch introduces a mechanism to reconnect to the channel. Once an error occurred the setup callback is called, which gives the handled client the chance to recreate the socket and prepare it for a connect. After that callback was called, the errornous connection will be moved into the unconnected items dict where it will be handled by the event loop. If there have been 5 or more unsuccessful attempts made the reconnect rate will be slowed down to the same time as specified for the 'read timeout'. The items which are slowed down are moved into the 'reconnect_cooldown' dict. Change-Id: I8b9b9418e39558d45c97e9545bc9ecc4935f004e Bug-Url: https://bugzilla.redhat.com/871616 Signed-off-by: Vinzenz Feenstra <[email protected]> --- M vdsm/guestIF.py M vdsm/vmChannels.py 2 files changed, 94 insertions(+), 21 deletions(-) Approvals: Vinzenz Feenstra: Verified Dan Kenigsberg: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/11977 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8b9b9418e39558d45c97e9545bc9ecc4935f004e Gerrit-PatchSet: 19 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Vinzenz Feenstra <[email protected]> Gerrit-Reviewer: Antoni Segura Puimedon <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Federico Simoncelli <[email protected]> Gerrit-Reviewer: Gal Hammer <[email protected]> Gerrit-Reviewer: Igor Lvovsky <[email protected]> Gerrit-Reviewer: Jason Dillaman <[email protected]> Gerrit-Reviewer: Mark Wu <[email protected]> Gerrit-Reviewer: Michal Skrivanek <[email protected]> Gerrit-Reviewer: Peter V. Saveliev <[email protected]> Gerrit-Reviewer: Vinzenz Feenstra <[email protected]> Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
