Ottomata has submitted this change and it was merged. Change subject: Fixing README and comments ......................................................................
Fixing README and comments Change-Id: Ib0e33a50b9acc0f3cf0bd971200ed8ae903a9339 --- M modules/limn/README.md M modules/limn/manifests/init.pp 2 files changed, 5 insertions(+), 11 deletions(-) Approvals: Ottomata: Verified; Looks good to me, approved diff --git a/modules/limn/README.md b/modules/limn/README.md index c2b5477..6abbbcb 100644 --- a/modules/limn/README.md +++ b/modules/limn/README.md @@ -11,18 +11,15 @@ ``` -The above will start a Limn instance listening on port 8081, with an -apache VirtualHost reverse proxying to it on port 80. If the defaults -for the VirtualHost don't work for you, you can set proxy => false -on the instance and set it up yourself or using the limn::instance::proxy -define. - Note that the limn class by default does not attempt to install a limn package. There is not currently a supported limn .deb package, so we recommend cloning the [limn repository][https://github.com/wikimedia/limn] directly into /usr/local/share/limn (or elsewhere if you set base_directory to something non default on limn::instance). Or deploy limn to wherever you see fit. + +```limn::instance::proxy``` sets up an Apache VirtualHost reverse proxying on port 80. + ## Example: ```bash diff --git a/modules/limn/manifests/init.pp b/modules/limn/manifests/init.pp index 84d1b14..61d01d9 100644 --- a/modules/limn/manifests/init.pp +++ b/modules/limn/manifests/init.pp @@ -1,5 +1,5 @@ # == Class limn -# Installs limn. +# Sets up limn. # To spawn up a limn server instance, use the limn::instance define. # # NOTE: This does not install limn. You must do that youself. @@ -9,13 +9,10 @@ # == Parameters # $var_directory - Default path to Limn var directory. This will also be limn user's home directory. Default: /var/lib/limn # $log_directory - Default path to Limn server logs. Default: /var/log/limn -# $user - Limn user. Default: limn -# $group - Limn group. Default: limn # class limn( $var_directory = '/var/lib/limn', - $log_directory = '/var/log/limn', - $install = false) + $log_directory = '/var/log/limn') { $user = 'limn' $group = 'limn' -- To view, visit https://gerrit.wikimedia.org/r/56395 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib0e33a50b9acc0f3cf0bd971200ed8ae903a9339 Gerrit-PatchSet: 1 Gerrit-Project: operations/puppet Gerrit-Branch: production Gerrit-Owner: Ottomata <[email protected]> Gerrit-Reviewer: Ottomata <[email protected]> _______________________________________________ MediaWiki-commits mailing list [email protected] https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
