Asher has submitted this change and it was merged. Change subject: Fixing undefined variable error for Redis module when $redis_replication was not set. ......................................................................
Fixing undefined variable error for Redis module when $redis_replication was not set. This adds a $redis_replication parameter to the redis class. Previously this was a global variable referenced from the redis.conf.erb template. It now must be passed in as a class parameter. The .erb template now checks to see if this variable is defined before attempting to configure redis replication. See RT 4257 Change-Id: I751fd2a1f1fc425f4958753b33b90150eb3215cf --- M manifests/site.pp M modules/redis/manifests/init.pp M modules/redis/templates/redis.conf.erb 3 files changed, 6 insertions(+), 3 deletions(-) Approvals: Asher: Verified; Looks good to me, approved jenkins-bot: Verified -- To view, visit https://gerrit.wikimedia.org/r/42566 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: merged Gerrit-Change-Id: I751fd2a1f1fc425f4958753b33b90150eb3215cf Gerrit-PatchSet: 1 Gerrit-Project: operations/puppet Gerrit-Branch: production Gerrit-Owner: Ottomata <[email protected]> Gerrit-Reviewer: Asher <[email protected]> Gerrit-Reviewer: jenkins-bot _______________________________________________ MediaWiki-commits mailing list [email protected] https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
