coren has submitted this change and it was merged.
Change subject: Break up our 'sudo' commands into separate args.
......................................................................
Break up our 'sudo' commands into separate args.
This makes 'sudo' happier.
Change-Id: Iaead542d633eda99d94d43facb02a25fc857d549
---
M files/ldap/scripts/manage-nfs-volumes-daemon
1 file changed, 2 insertions(+), 2 deletions(-)
Approvals:
coren: Looks good to me, approved
jenkins-bot: Verified
diff --git a/files/ldap/scripts/manage-nfs-volumes-daemon
b/files/ldap/scripts/manage-nfs-volumes-daemon
index 9bf4b8a..e75b136 100755
--- a/files/ldap/scripts/manage-nfs-volumes-daemon
+++ b/files/ldap/scripts/manage-nfs-volumes-daemon
@@ -234,8 +234,8 @@
# We need to clean up from partially-failed runs. rmdir is pretty
conservative; it will
# only clean up empty dirs.
- cmds1 = ['sudo', 'rmdir %s' % voldir]
- cmds2 = ['sudo', 'mkdir -p %s' % voldir]
+ cmds1 = ['sudo', 'rmdir', voldir]
+ cmds2 = ['sudo', 'mkdir', '-p', voldir]
if self.dry_run:
print "Exec %s" % cmds1
--
To view, visit https://gerrit.wikimedia.org/r/60226
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Iaead542d633eda99d94d43facb02a25fc857d549
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Andrew Bogott <[email protected]>
Gerrit-Reviewer: coren <[email protected]>
Gerrit-Reviewer: jenkins-bot
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
