Dan Kenigsberg has submitted this change and it was merged. Change subject: getVMFullList fails when processing externally managed vms ......................................................................
getVMFullList fails when processing externally managed vms When we send 22k response we get -1 during send. After the investigation we found that m2crypto ignores error codes and returns -1. We are not sure which error is returned. It can be one of SSL_ERROR_WANT_WRITE, SSL_ERROR_WANT_READ, SSL_ERROR_NONE, SSL_ERROR_ZERO_RETURN. When we get -1 we need to get back to poll loop so we override send method and return 0 when we see -1 which fixes the issue. Bug-Url: https://bugzilla.redhat.com/1135959 Change-Id: Ia728691880771388b6179e85a97df3f303aeddf8 Signed-off-by: pkliczewski <[email protected]> Reviewed-on: http://gerrit.ovirt.org/32263 Reviewed-by: Saggi Mizrahi <[email protected]> --- M lib/yajsonrpc/betterAsyncore.py 1 file changed, 16 insertions(+), 0 deletions(-) Approvals: Piotr Kliczewski: Verified Saggi Mizrahi: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/32263 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia728691880771388b6179e85a97df3f303aeddf8 Gerrit-PatchSet: 3 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Piotr Kliczewski <[email protected]> Gerrit-Reviewer: Barak Azulay <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Piotr Kliczewski <[email protected]> Gerrit-Reviewer: Saggi Mizrahi <[email protected]> Gerrit-Reviewer: Yaniv Bronhaim <[email protected]> Gerrit-Reviewer: [email protected] Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
