From Dan Kenigsberg <[email protected]>: Dan Kenigsberg has posted comments on this change.
Change subject: m2c: uses different protocol constant names ...................................................................... Patch Set 4: Code-Review+2 (1 comment) https://gerrit.ovirt.org/#/c/79664/4/lib/vdsm/m2cutils.py File lib/vdsm/m2cutils.py: Line 37: except ImportError as e: Line 38: raise compat.Unsupported(str(e)) Line 39: Line 40: CLIENT_PROTOCOL = "sslv23" Line 41: SSL_OP_NO_TLSv1_1 = 268435456 0x10000000 is SOOO much more readable. Line 42: Line 43: DEFAULT_ACCEPT_TIMEOUT = 5 Line 44: SOCKET_DEFAULT_TIMEOUT = socket._GLOBAL_DEFAULT_TIMEOUT Line 45: -- To view, visit https://gerrit.ovirt.org/79664 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I6c05bddfba94d85e2c48122b6fd56d8f336d01f5 Gerrit-PatchSet: 4 Gerrit-Project: vdsm Gerrit-Branch: ovirt-4.1 Gerrit-Owner: Piotr Kliczewski <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Francesco Romani <[email protected]> Gerrit-Reviewer: Irit Goihman <[email protected]> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Piotr Kliczewski <[email protected]> Gerrit-Reviewer: Yaniv Bronhaim <[email protected]> Gerrit-Reviewer: gerrit-hooks <[email protected]> Gerrit-HasComments: Yes _______________________________________________ vdsm-patches mailing list -- [email protected] To unsubscribe send an email to [email protected]
