Change in ovirt-iso-uploader[master]: upload: nfs: handle rename errors

sbonazzo at redhat.com sbonazzo at redhat.com
Thu Sep 12 14:02:07 UTC 2013


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 at 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 at redhat.com>
Gerrit-Reviewer: Alex Lourie <alourie at redhat.com>
Gerrit-Reviewer: Keith Robertson <kroberts at redhat.com>
Gerrit-Reviewer: Kiril Nesenko <knesenko at redhat.com>
Gerrit-Reviewer: Ofer Schreiber <oschreib at redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo at redhat.com>



More information about the Engine-commits mailing list