Martin Sivák has submitted this change and it was merged.
Change subject: refactor the brokerlink connect() code
......................................................................
refactor the brokerlink connect() code
- so it actually ends after a few attempts to connect
- throw a proper exception when the attempts are exhausted
Bug-Url:
https://bugzilla.redhat.com/show_bug.cgi?id=1093646
Change-Id: I8875c0a19d61c5331a27c11bfa762177dd6a335b
Signed-off-by: Jiri Moskovcak <jmoskovc(a)redhat.com>
---
M ovirt_hosted_engine_ha/agent/constants.py.in
M ovirt_hosted_engine_ha/lib/brokerlink.py
M ovirt_hosted_engine_ha/lib/exceptions.py
3 files changed, 23 insertions(+), 15 deletions(-)
Approvals:
Sandro Bonazzola: Looks good to me, but someone else must approve
Martin Sivák: Looks good to me, approved
Jiří Moskovčák: Verified
--
To view, visit
http://gerrit.ovirt.org/30113
To unsubscribe, visit
http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I8875c0a19d61c5331a27c11bfa762177dd6a335b
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-hosted-engine-ha
Gerrit-Branch: master
Gerrit-Owner: Jiří Moskovčák <jmoskovc(a)redhat.com>
Gerrit-Reviewer: Jiří Moskovčák <jmoskovc(a)redhat.com>
Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server