Merge ~pjdc/ubuntu-mirror-charm/+git/ubuntu-mirror-charm:enforce-perms-for-1-stage-sync into ubuntu-mirror-charm:master

Proposed by Paul Collins
Status: Merged
Approved by: Haw Loeung
Approved revision: 83ddea402429f8a20485a289de8e4aa7e15b73cf
Merged at revision: e5c2fcf7e82aa72aa38be73b688e1b2ecc0cfb97
Proposed branch: ~pjdc/ubuntu-mirror-charm/+git/ubuntu-mirror-charm:enforce-perms-for-1-stage-sync
Merge into: ubuntu-mirror-charm:master
Diff against target: 14 lines (+4/-0)
1 file modified
files/mirror-1stage.sh (+4/-0)
Reviewer Review Type Date Requested Status
Haw Loeung +1 Approve
Canonical IS Reviewers Pending
Review via email: mp+394608@code.launchpad.net

Commit message

import permission-correction from mirror-2stage.sh

To post a comment you must log in.
Revision history for this message
🤖 Canonical IS Merge Bot (canonical-is-mergebot) wrote :

This merge proposal is being monitored by mergebot. Change the status to Approved to merge.

Revision history for this message
Haw Loeung (hloeung) wrote :

LGTM

review: Approve (+1)
Revision history for this message
🤖 Canonical IS Merge Bot (canonical-is-mergebot) wrote :

Change successfully merged at revision e5c2fcf7e82aa72aa38be73b688e1b2ecc0cfb97

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/files/mirror-1stage.sh b/files/mirror-1stage.sh
2index f578059..4832970 100755
3--- a/files/mirror-1stage.sh
4+++ b/files/mirror-1stage.sh
5@@ -83,6 +83,10 @@ rsync --verbose --recursive --times --links --hard-links \
6 --exclude ".~tmp~" \
7 ${url} ${DEST_DIR} || fatal "Failed to rsync from ${url}."
8
9+log "== Making sure permissions are all correct =="
10+find ${DEST_DIR} -type d ! -perm /o=r,o=x -ls -exec chmod go+rx '{}' +
11+find ${DEST_DIR} -type f ! -perm /o=r -ls -exec chmod go+r '{}' +
12+
13 install -d "${DEST_DIR}/.trace/"
14 date -u > "${DEST_DIR}/.trace/${role}-${myhostname}"
15

Subscribers

People subscribed via source and target branches