Change in ovirt-log-collector[master]: tools: Gracefully handle cases where API is down.

oschreib at redhat.com oschreib at redhat.com
Thu Jul 5 10:41:03 UTC 2012


Ofer Schreiber has submitted this change and it was merged.

Change subject: tools: Gracefully handle cases where API is down.
......................................................................


tools: Gracefully handle cases where API is down.

Didn't have gettext imported which caused many
exception catches to be bypassed and resulted
in ungraceful exits.  Manifested obviously when
REST API was down or when incorrect PW supplied.

Change-Id: Icd77cb075e6984e4feb885fd407e21ed34f09cc1
Signed-off-by: Keith Robertson <kroberts at redhat.com>
---
M src/rhev/helper/hypervisors.py
M src/rhev/logcollector.py
2 files changed, 19 insertions(+), 4 deletions(-)

Approvals:
  Ofer Schreiber: Looks good to me, approved
  Keith Robertson: Verified


--
To view, visit http://gerrit.ovirt.org/5785
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Icd77cb075e6984e4feb885fd407e21ed34f09cc1
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-log-collector
Gerrit-Branch: master
Gerrit-Owner: Keith Robertson <kroberts at redhat.com>
Gerrit-Reviewer: Keith Robertson <kroberts at redhat.com>
Gerrit-Reviewer: Kiril Nesenko <knesenko at redhat.com>
Gerrit-Reviewer: Lee Yarwood <lyarwood at redhat.com>
Gerrit-Reviewer: Moran Goldboim <mgoldboi at redhat.com>
Gerrit-Reviewer: Ofer Schreiber <oschreib at redhat.com>



More information about the Engine-commits mailing list