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

sbonazzo at redhat.com sbonazzo at redhat.com
Thu Jun 18 07:34:22 UTC 2015


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 at 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 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: automation at ovirt.org



More information about the Engine-commits mailing list