Yaniv Bronhaim has submitted this change and it was merged. Change subject: protocoldetecor: SSLError handled not correctly ......................................................................
protocoldetecor: SSLError handled not correctly Whenever there was ssl issue during protocol detection the code entered infinite loop because we haven't remove fd from pending_connections. It is fixed by caching SSLError reading data and clean up fd correctly. Label: ovirt-3.5-only Change-Id: I8ea5c305b19c0a7421ea74e069c3ad02d9ffd141 Signed-off-by: pkliczewski <[email protected]> Bug-Url: https://bugzilla.redhat.com/1229859 Bug-Url: https://bugzilla.redhat.com/1215236 Reviewed-on: https://gerrit.ovirt.org/42206 Continuous-Integration: Jenkins CI Reviewed-by: Nir Soffer <[email protected]> Reviewed-by: Yaniv Bronhaim <[email protected]> --- M vdsm/protocoldetector.py 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Piotr Kliczewski: Verified Nir Soffer: Looks good to me, but someone else must approve Yaniv Bronhaim: Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/42206 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8ea5c305b19c0a7421ea74e069c3ad02d9ffd141 Gerrit-PatchSet: 7 Gerrit-Project: vdsm Gerrit-Branch: ovirt-3.5 Gerrit-Owner: Piotr Kliczewski <[email protected]> Gerrit-Reviewer: Dima Kuznetsov <[email protected]> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Nir Soffer <[email protected]> Gerrit-Reviewer: Piotr Kliczewski <[email protected]> Gerrit-Reviewer: Yaniv Bronhaim <[email protected]> Gerrit-Reviewer: Yeela Kaplan <[email protected]> Gerrit-Reviewer: [email protected] _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
