Dzahn has submitted this change and it was merged.
Change subject: rsync: Fix variable contains an uppercase letter
..
rsync: Fix variable contains an uppercase letter
Bug: T93645
Change-Id: I60a8dcd232dac9d9b563c97285cb58534e962b73
---
M modules/rsync/manifests/get.pp
1 file changed, 8 insertions(+), 8 deletions(-)
Approvals:
jenkins-bot: Verified
Dzahn: Looks good to me, approved
diff --git a/modules/rsync/manifests/get.pp b/modules/rsync/manifests/get.pp
index 1f3ff28..b2e80ff 100644
--- a/modules/rsync/manifests/get.pp
+++ b/modules/rsync/manifests/get.pp
@@ -35,30 +35,30 @@
) {
if $keyfile {
-$Mykeyfile = $keyfile
+$mykeyfile = $keyfile
} else {
-$Mykeyfile = "/home/${user}/.ssh/id_rsa"
+$mykeyfile = "/home/${user}/.ssh/id_rsa"
}
if $user {
-$MyUser = "-e 'ssh -i ${Mykeyfile} -l ${user}' ${user}@"
+$myuser = "-e 'ssh -i ${mykeyfile} -l ${user}' ${user}@"
}
if $purge {
-$MyPurge = '--delete'
+$mypurge = '--delete'
}
if $exclude {
-$MyExclude = "--exclude=${exclude}"
+$myexclude = "--exclude=${exclude}"
}
if $path {
-$MyPath = $path
+$mypath = $path
} else {
-$MyPath = $name
+$mypath = $name
}
- $rsync_options = "-a ${MyPurge} ${MyExclude} ${MyUser}${source} ${MyPath}"
+ $rsync_options = "-a ${mypurge} ${myexclude} ${myuser}${source} ${mypath}"
exec { "rsync ${name}":
command => "rsync -q ${rsync_options}",
--
To view, visit https://gerrit.wikimedia.org/r/308350
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I60a8dcd232dac9d9b563c97285cb58534e962b73
Gerrit-PatchSet: 4
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Paladox
Gerrit-Reviewer: Dzahn
Gerrit-Reviewer: Ottomata
Gerrit-Reviewer: jenkins-bot <>
___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
