ArielGlenn has submitted this change and it was merged. ( https://gerrit.wikimedia.org/r/402322 )
Change subject: snapshot hosts: empty nfs server name means no mount ...................................................................... snapshot hosts: empty nfs server name means no mount Change-Id: Iccf4de539955253e200f42d3c231f95014240705 --- M modules/snapshot/manifests/dumps/nfsmount.pp 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: ArielGlenn: Looks good to me, approved jenkins-bot: Verified diff --git a/modules/snapshot/manifests/dumps/nfsmount.pp b/modules/snapshot/manifests/dumps/nfsmount.pp index 92b910e..86c7f21 100644 --- a/modules/snapshot/manifests/dumps/nfsmount.pp +++ b/modules/snapshot/manifests/dumps/nfsmount.pp @@ -11,7 +11,7 @@ ensure => 'directory', } - if (defined('$server')) { + if (defined('$server') and $server != '') { mount { $mountpoint: ensure => 'mounted', device => "${server}:/data", -- To view, visit https://gerrit.wikimedia.org/r/402322 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iccf4de539955253e200f42d3c231f95014240705 Gerrit-PatchSet: 1 Gerrit-Project: operations/puppet Gerrit-Branch: production Gerrit-Owner: ArielGlenn <ar...@wikimedia.org> Gerrit-Reviewer: ArielGlenn <ar...@wikimedia.org> Gerrit-Reviewer: jenkins-bot <> _______________________________________________ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits