Merge lp:~allenap/maas/maas-set-correct-file-permissions into lp:~maas-committers/maas/trunk
Proposed by
Gavin Panella
Status: | Merged |
---|---|
Approved by: | Andres Rodriguez |
Approved revision: | no longer in the source branch. |
Merged at revision: | 956 |
Proposed branch: | lp:~allenap/maas/maas-set-correct-file-permissions |
Merge into: | lp:~maas-committers/maas/trunk |
Diff against target: |
83 lines (+26/-7) 3 files modified
scripts/maas-import-ephemerals (+0/-7) src/provisioningserver/pxe/install_image.py (+8/-0) src/provisioningserver/pxe/tests/test_install_image.py (+18/-0) |
To merge this branch: | bzr merge lp:~allenap/maas/maas-set-correct-file-permissions |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Scott Moser (community) | Approve | ||
Review via email:
|
Commit message
Normalise permissions of all files and directories installed via install-pxe-image.
Previously this was not done, and restrictive permissions could cause failures later on the TFTP server.
This also reverts revision 947, an earlier attempt to fix the same bug.
Description of the change
This changes install_dir() to normalize
To post a comment you must log in.
its curious that you're doing chmod after you've put the files in place, given the big in comments there about a race condition. (which I honestly do not think that is a large concern as this will be very small window and not a common operation).
But why wouldn't you fix permissions *before* you move it into place?