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@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@redhat.com> Gerrit-Reviewer: Keith Robertson <kroberts@redhat.com> Gerrit-Reviewer: Kiril Nesenko <knesenko@redhat.com> Gerrit-Reviewer: Lee Yarwood <lyarwood@redhat.com> Gerrit-Reviewer: Moran Goldboim <mgoldboi@redhat.com> Gerrit-Reviewer: Ofer Schreiber <oschreib@redhat.com>