Change in ovirt-image-uploader[master]: ovirt-image-uploader: return non-zero returncodes

sbonazzo at redhat.com sbonazzo at redhat.com
Mon Nov 9 07:58:56 UTC 2015


Sandro Bonazzola has submitted this change and it was merged.

Change subject: ovirt-image-uploader: return non-zero returncodes
......................................................................


ovirt-image-uploader: return non-zero returncodes

Previously we returned returncode of 0 even if something went
wrong during the file upload phase.
This patch fixes this issue and makes the program return
ExitCodes.UPLOAD_ERR code (3) in this case.

Change-Id: I0717ecb13b9cae8331a21f35e1f090698aaac4f3
Signed-off-by: Lev Veyde <lveyde at redhat.com>
Bug-Url: https://bugzilla.redhat.com/1104661
---
M src/__main__.py
1 file changed, 25 insertions(+), 13 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
  Rafael Martins: Looks good to me, but someone else must approve



-- 
To view, visit https://gerrit.ovirt.org/47501
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I0717ecb13b9cae8331a21f35e1f090698aaac4f3
Gerrit-PatchSet: 5
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: Rafael Martins <rmartins 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