Sandro Bonazzola has submitted this change and it was merged.
Change subject: ovirt-image-uploader: Provide more informative error messages
......................................................................
ovirt-image-uploader: Provide more informative error messages
The code was modified to provide more information on connection
failures, so now more specific information will be available to
users.
Also some error logic was added to process request errors,
that is errors that happen after the successful connection was
established, e.g. bad authentication etc. Now we can provide
specific messages based on the return code e.g. 401, 503 etc.
Change-Id: I66d8c6dc59169bdc798256bd2d150fbc8cda7665
Signed-off-by: Lev Veyde <lveyde(a)redhat.com>
Bug-Url:
https://bugzilla.redhat.com/1182344
---
M src/__main__.py
1 file changed, 24 insertions(+), 8 deletions(-)
Approvals:
Lev Veyde: Verified
Sandro Bonazzola: Looks good to me, approved
Simone Tiraboschi: Looks good to me, but someone else must approve
Jenkins CI: Passed CI tests
--
To view, visit
https://gerrit.ovirt.org/42554
To unsubscribe, visit
https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I66d8c6dc59169bdc798256bd2d150fbc8cda7665
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-image-uploader
Gerrit-Branch: master
Gerrit-Owner: Lev Veyde <lveyde(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>
Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com>
Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org