Sandro Bonazzola has submitted this change and it was merged. Change subject: upload: nfs: set correct umask while renaming ...................................................................... upload: nfs: set correct umask while renaming set the correct umask while renaming the temporary file to destination file. Avoid to have incorrect access bits on systems with not standard umask (like 077) Change-Id: I5b7401f91e5fb4ce604ff17209e7c1260dd57503 Bug-Url: https://bugzilla.redhat.com/988146 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com&gt; --- M src/__main__.py 1 file changed, 3 insertions(+), 1 deletion(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Alex Lourie: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/17411 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5b7401f91e5fb4ce604ff17209e7c1260dd57503 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-iso-uploader Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com&gt; Gerrit-Reviewer: Alex Lourie <alourie(a)redhat.com&gt; Gerrit-Reviewer: Keith Robertson <kroberts(a)redhat.com&gt; Gerrit-Reviewer: Kiril Nesenko <knesenko(a)redhat.com&gt; Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com&gt;