Change in ovirt-image-uploader[master]: ovirt-image-uploader: Provide more informative error messages

sbonazzo at redhat.com sbonazzo at redhat.com
Mon Jul 27 11:46:43 UTC 2015


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 at 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 at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Lev Veyde <lveyde at redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo at redhat.com>
Gerrit-Reviewer: Simone Tiraboschi <stirabos at redhat.com>
Gerrit-Reviewer: Yedidyah Bar David <didi at redhat.com>
Gerrit-Reviewer: automation at ovirt.org



More information about the Engine-commits mailing list