Ottomata has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/56395


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(-)


  git pull ssh://gerrit.wikimedia.org:29418/operations/puppet 
refs/changes/95/56395/1

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: newchange
Gerrit-Change-Id: Ib0e33a50b9acc0f3cf0bd971200ed8ae903a9339
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Ottomata <[email protected]>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to