ArielGlenn has submitted this change and it was merged.
Change subject: explicitly set perms on the empty directory for rsync deletes
......................................................................
explicitly set perms on the empty directory for rsync deletes
If we don't do this, we may wind up with overly restrictive
perms on the rsynced directories on the target filesystem.
Change-Id: Ia060d14c6c16d68ec207639637beb29c498b641b
---
M tools/mirrors/wmfdumpsmirror.py
1 file changed, 1 insertion(+), 1 deletion(-)
Approvals:
ArielGlenn: Verified; Looks good to me, approved
jenkins-bot: Verified
diff --git a/tools/mirrors/wmfdumpsmirror.py b/tools/mirrors/wmfdumpsmirror.py
index 4571e1f..17b11a4 100644
--- a/tools/mirrors/wmfdumpsmirror.py
+++ b/tools/mirrors/wmfdumpsmirror.py
@@ -273,7 +273,7 @@
if self.emptydirpath is None:
self.emptydirpath = maketempdir()
-
+ os.chmod(self.emptydirpath, 0755)
excludes = ["--exclude=/{0}/".format(dirname) for dirname in
project_dirs_rsynced]
command = ["/usr/bin/rsync", "-a", "--delete"]
command.extend(excludes)
--
To view, visit https://gerrit.wikimedia.org/r/275805
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ia060d14c6c16d68ec207639637beb29c498b641b
Gerrit-PatchSet: 2
Gerrit-Project: operations/dumps
Gerrit-Branch: ariel
Gerrit-Owner: ArielGlenn <[email protected]>
Gerrit-Reviewer: ArielGlenn <[email protected]>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
