Change in ovirt-hosted-engine-ha[v2.1.z]: Fix ping submonitor - casting None to float

Code Review gerrit at ovirt.org
Fri Jun 16 13:25:09 UTC 2017


>From Martin Sivák <msivak at redhat.com>:

Martin Sivák has submitted this change and it was merged.

Change subject: Fix ping submonitor - casting None to float
......................................................................


Fix ping submonitor - casting None to float

Ping monitor returns None, before the first monitoring cycle.
Agent tries to convert it to float which causes TypeError exception.

Change-Id: Iab51dbdb4f394a11c9c3d103c1287904b350512f
Bug-Url:https://bugzilla.redhat.com/1449333
Signed-off-by: Andrej Krejcir <akrejcir at redhat.com>
(cherry picked from commit 18dac9598827b440025adf3305b7bd3ae4dd23e6)
---
M ovirt_hosted_engine_ha/broker/submonitors/ping.py
1 file changed, 3 insertions(+), 0 deletions(-)

Approvals:
  Martin Sivák: Looks good to me, approved
  Jenkins CI: Passed CI tests
  Andrej Krejcir: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Iab51dbdb4f394a11c9c3d103c1287904b350512f
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-hosted-engine-ha
Gerrit-Branch: v2.1.z
Gerrit-Owner: Andrej Krejcir <akrejcir at redhat.com>
Gerrit-Reviewer: Andrej Krejcir <akrejcir at redhat.com>
Gerrit-Reviewer: Denis Chaplygin <dchaplyg at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Jenny Tokar <jtokar at redhat.com>
Gerrit-Reviewer: Martin Sivák <msivak at redhat.com>
Gerrit-Reviewer: Phillip Bailey <phbailey at redhat.com>
Gerrit-Reviewer: Roman Mohr <rmohr at redhat.com>
Gerrit-Reviewer: Yanir Quinn <yquinn at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>


More information about the Engine-commits mailing list