Change in vdsm[master]: xmlrpc: retry when flushing socket

fromani at redhat.com fromani at redhat.com
Fri Oct 23 08:48:14 UTC 2015


Francesco Romani has posted comments on this change.

Change subject: xmlrpc: retry when flushing socket
......................................................................


Patch Set 2:

(1 comment)

https://gerrit.ovirt.org/#/c/47637/2/vdsm/rpc/bindingxmlrpc.py
File vdsm/rpc/bindingxmlrpc.py:

Line 299:                             # An final socket error may have occurred here,
Line 300:                             # such as the local error ECONNABORTED.
Line 301:                             break
Line 302:                         except SSL.SSLError as e:
Line 303:                             if e.message == 'bad write retry':
we don't have any other mean to do this check? an error code, maybe?
Line 304:                                 continue
Line 305:                             else:
Line 306:                                 break
Line 307: 


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I52dd6ad304a82ff6c8d3dff12a38269684abf055
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Piotr Kliczewski <piotr.kliczewski at gmail.com>
Gerrit-Reviewer: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski at gmail.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: Yeela Kaplan <ykaplan at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-HasComments: Yes


More information about the vdsm-patches mailing list