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 <thomasmulhall...@yahoo.com> Gerrit-Reviewer: Dzahn <dz...@wikimedia.org> Gerrit-Reviewer: Ottomata <o...@wikimedia.org> Gerrit-Reviewer: jenkins-bot <> _______________________________________________ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits