Change in ovirt-hosted-engine-ha[master]: refactor the brokerlink connect() code

msivak at redhat.com msivak at redhat.com
Wed Jul 23 13:37:22 UTC 2014


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 at 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 at redhat.com>
Gerrit-Reviewer: Jiří Moskovčák <jmoskovc at redhat.com>
Gerrit-Reviewer: Martin Sivák <msivak at redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list