Sandro Bonazzola has submitted this change and it was merged.
Change subject: ovirt-iso-uploader: Provide more informative error messages
......................................................................
ovirt-iso-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: Ib0c846a556d9658821e855567ca9688e7eb390c1
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(+), 7 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/42519
To unsubscribe, visit
https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ib0c846a556d9658821e855567ca9688e7eb390c1
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-iso-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: automation(a)ovirt.org