Sandro Bonazzola has submitted this change and it was merged. Change subject: upload: nfs: handle rename errors ...................................................................... upload: nfs: handle rename errors Uploading to a NFS destination, if an error were encountered while renaming the used temporary file to the final destination, the application displayed: uploaded successfully. Now the application exits with exit status 3 (upload error) and display an error message. Added also a debug log for recording the rename action in the verbose log. Change-Id: I8fc993d0fdc36924cbf3dd7ed50569ccdcfc75b5 Bug-Url: https://bugzilla.redhat.com/1005100 Signed-off-by: Sandro Bonazzola <sbonazzo@redhat.com> --- M src/__main__.py 1 file changed, 42 insertions(+), 12 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/19095 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8fc993d0fdc36924cbf3dd7ed50569ccdcfc75b5 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-iso-uploader Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo@redhat.com> Gerrit-Reviewer: Alex Lourie <alourie@redhat.com> Gerrit-Reviewer: Keith Robertson <kroberts@redhat.com> Gerrit-Reviewer: Kiril Nesenko <knesenko@redhat.com> Gerrit-Reviewer: Ofer Schreiber <oschreib@redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo@redhat.com>