Comment 5 for bug 1433390

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to puppet-swift (stable/juno)

Reviewed: https://review.openstack.org/167307
Committed: https://git.openstack.org/cgit/stackforge/puppet-swift/commit/?id=3a695cd7a0905773f58f7c4e347338d24841a47b
Submitter: Jenkins
Branch: stable/juno

commit 3a695cd7a0905773f58f7c4e347338d24841a47b
Author: David Moreau Simard <email address hidden>
Date: Tue Mar 17 22:05:53 2015 -0400

    Provide a mean to change the default rsync chmod

    puppet-rsync provides a default chmod of 0644.
    puppet-swift, until this commit, did not provide a way
    to change that default chmod.
    According to the experience in bug #1433390, it
    seemed possible for folders to be created in 0644, thus
    denying access to the folders unless you were root.

    Backwards compatibility is maintained through the same
    defaults.. It looks like the default is broken in
    the context of Swift (unless it runs as root?), however.
    We should consider putting 'Du=rwx,g=rx,o=rx,Fu=rw,g=r,o=r'
    which translates to 755 for folders and 644 for files.

    Change-Id: I7fcef6e4cdf478e808de2895168165989ff098e4
    Closes-bug: #1433390
    (cherry picked from commit 2caba862a8609528f10c975d7a956b246f2ca477)