Dan Kenigsberg has submitted this change and it was merged. Change subject: protocoldetector: Fix next_check_interval ......................................................................
protocoldetector: Fix next_check_interval Commit b25dbdcce4 (Have the protocol detector use the Reactor) added the check_next_interval() interface, returning the timeout until the next check. However the implementation in some of the dispatchers was wrong, always returning 0. This cause the event loop to run a busy polling loop instead of waiting until the next dispatcher check time. Change-Id: If7f71657df4baea22ade66dbda0a679aa4cb6832 Signed-off-by: Nir Soffer <[email protected]> Reviewed-on: https://gerrit.ovirt.org/39994 Reviewed-by: Yaniv Bronhaim <[email protected]> Reviewed-by: Dima Kuznetsov <[email protected]> Reviewed-by: Piotr Kliczewski <[email protected]> Reviewed-by: Dan Kenigsberg <[email protected]> --- M lib/vdsm/sslutils.py M vdsm/protocoldetector.py 2 files changed, 2 insertions(+), 2 deletions(-) Approvals: Piotr Kliczewski: Looks good to me, but someone else must approve Nir Soffer: Verified Yaniv Bronhaim: Looks good to me, but someone else must approve Dima Kuznetsov: Looks good to me, but someone else must approve Dan Kenigsberg: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/39994 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If7f71657df4baea22ade66dbda0a679aa4cb6832 Gerrit-PatchSet: 2 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Nir Soffer <[email protected]> Gerrit-Reviewer: Allon Mureinik <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Dima Kuznetsov <[email protected]> Gerrit-Reviewer: Nir Soffer <[email protected]> Gerrit-Reviewer: Piotr Kliczewski <[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
