[MediaWiki-commits] [Gerrit] Fix various typos - change (operations/puppet)

2014-01-13 Thread Tim Landscheidt (Code Review)
Tim Landscheidt has uploaded a new change for review.

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


Change subject: Fix various typos
..

Fix various typos

Change-Id: I2aa6e65e4eac8b920bf39fb51089cfaea9b0956f
---
M files/nfs/upstart-nfs-noidmap.conf
M manifests/ganglia.pp
M modules/toollabs/manifests/init.pp
3 files changed, 15 insertions(+), 16 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/operations/puppet 
refs/changes/65/107165/1

diff --git a/files/nfs/upstart-nfs-noidmap.conf 
b/files/nfs/upstart-nfs-noidmap.conf
index 61fb5cf..91dc558 100644
--- a/files/nfs/upstart-nfs-noidmap.conf
+++ b/files/nfs/upstart-nfs-noidmap.conf
@@ -1,10 +1,10 @@
 # nfs-noidmap - disable idmap for cross-project NFS
 #
-# This task disables the kernel's automatic idmap for NFSv4
-# to allow conflicting group names accross projects
+# This task disables the kernel's automatic idmap for NFSv4 to allow
+# conflicting group names across projects.
 #
-# Made into an upstart task because that setting is annoyingly
-# not made available through sysctl.
+# Made into an upstart task because that setting is annoyingly not
+# made available through sysctl.
 
 description disable idmap for cross-project NFS
 
diff --git a/manifests/ganglia.pp b/manifests/ganglia.pp
index 865d869..81d5d78 100644
--- a/manifests/ganglia.pp
+++ b/manifests/ganglia.pp
@@ -563,27 +563,27 @@
 }
 
 
-# == Define ganlia::view
+# == Define ganglia::view
 # Defines a Ganglia view JSON file.
 # See 
http://sourceforge.net/apps/trac/ganglia/wiki/ganglia-web-2#JSONdefinitionforviews
 # for documentation on Ganglia view JSON format.
 #
 # == Parameters:
-# $graphs   - Shortcut for of describing items that represent 
aggregate_graphs.
+# $graphs   - Shortcut for describing items that represent 
aggregate_graphs.
 # $items- Should match exactly the JSON structure expected by Ganglia 
for views.
 # $view_type- If you are using aggregate_graphs, this must be set to 
'standard'.
 # 'regex' will allow you to use non-aggregate graphs and match 
hostnames by regex.
 # Default: 'standard'.
 # $default_size - Default size for graphs.  Default: 'large'.
 # $conf_dir - Path to directory where ganglia view JSON files should live.
-# Defaults to the appropriate directory based on WMF $::realm. 
 Default: to /var/lib/ganglia/conf
-# $template - The ERb template to use for the JSON file.  Only change this 
if you need to do fancier things than this define allows.
+# Defaults to the appropriate directory based on WMF $::realm. 
 Default: /var/lib/ganglia/conf.
+# $template - The ERB template to use for the JSON file.  Only change this 
if you need to do fancier things than this define allows.
 #
 # == Examples:
-# # A 'regex' (non aggregate graph) view:
+# # A 'regex' (non-aggregate graph) view:
 # # Note that no aggregate_graphs are used.
 # # This will add 4 graphs to the 'cpu' view.
-# # (i.e. cpu_user and cpu_system for each myhost and myhost1)
+# # (i.e. cpu_user and cpu_system for each myhost0 and myhost1)
 # $host_regex = 'myhost[01]'
 # ganglia::view { 'cpu':
 #   view_type = 'regex',
@@ -651,10 +651,10 @@
 # $plugins - the plugin name (ex: 'diskstat'), will install the Python file
 # located in files/ganglia/plugins/${name}.py and expand the template from
 # templates/ganglia/plugins/${name}.pyconf.erb.
-# Defaults to $title as a convenience
+# Defaults to $title as a convenience.
 #
-# $opts - optional hash which can be used in the template. The defaults are
-# hardcoded in the templates. Default to {}
+# $opts - optional hash which can be used in the template.  The
+# defaults are hardcoded in the templates. Defaults to {}.
 #
 # == Examples:
 #
diff --git a/modules/toollabs/manifests/init.pp 
b/modules/toollabs/manifests/init.pp
index a125e35..48190e5 100644
--- a/modules/toollabs/manifests/init.pp
+++ b/modules/toollabs/manifests/init.pp
@@ -74,7 +74,7 @@
 # Tool Labs is enduser-facing, so we want to control the motd
 # properly (most things make no sense for community users: they
 # don't care that packages need updating, or that filesystems
-# will be checked, for instance)
+# will be checked, for instance).
 
 file { '/etc/update-motd.d':
 ensure  = directory,
@@ -86,7 +86,7 @@
 purge   = true,
 }
 
-# We keep a project-locat apt repo where we stuff packages we build
+# We keep a project-local apt repo where we stuff packages we build
 # that are intended to be local to the project.  By keeping it on the
 # shared storage, we have no need to set up a server to use it.
 
@@ -126,4 +126,3 @@
 target = ${store}/mail,
 }
 }
-

-- 
To view, visit https://gerrit.wikimedia.org/r/107165
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: 

[MediaWiki-commits] [Gerrit] Fix various typos - change (operations/puppet)

2014-01-13 Thread Faidon Liambotis (Code Review)
Faidon Liambotis has submitted this change and it was merged.

Change subject: Fix various typos
..


Fix various typos

Change-Id: I2aa6e65e4eac8b920bf39fb51089cfaea9b0956f
---
M files/nfs/upstart-nfs-noidmap.conf
M manifests/ganglia.pp
M modules/toollabs/manifests/init.pp
3 files changed, 15 insertions(+), 16 deletions(-)

Approvals:
  Faidon Liambotis: Verified; Looks good to me, approved



diff --git a/files/nfs/upstart-nfs-noidmap.conf 
b/files/nfs/upstart-nfs-noidmap.conf
index 61fb5cf..91dc558 100644
--- a/files/nfs/upstart-nfs-noidmap.conf
+++ b/files/nfs/upstart-nfs-noidmap.conf
@@ -1,10 +1,10 @@
 # nfs-noidmap - disable idmap for cross-project NFS
 #
-# This task disables the kernel's automatic idmap for NFSv4
-# to allow conflicting group names accross projects
+# This task disables the kernel's automatic idmap for NFSv4 to allow
+# conflicting group names across projects.
 #
-# Made into an upstart task because that setting is annoyingly
-# not made available through sysctl.
+# Made into an upstart task because that setting is annoyingly not
+# made available through sysctl.
 
 description disable idmap for cross-project NFS
 
diff --git a/manifests/ganglia.pp b/manifests/ganglia.pp
index 1a08f35..fa06976 100644
--- a/manifests/ganglia.pp
+++ b/manifests/ganglia.pp
@@ -574,27 +574,27 @@
 }
 
 
-# == Define ganlia::view
+# == Define ganglia::view
 # Defines a Ganglia view JSON file.
 # See 
http://sourceforge.net/apps/trac/ganglia/wiki/ganglia-web-2#JSONdefinitionforviews
 # for documentation on Ganglia view JSON format.
 #
 # == Parameters:
-# $graphs   - Shortcut for of describing items that represent 
aggregate_graphs.
+# $graphs   - Shortcut for describing items that represent 
aggregate_graphs.
 # $items- Should match exactly the JSON structure expected by Ganglia 
for views.
 # $view_type- If you are using aggregate_graphs, this must be set to 
'standard'.
 # 'regex' will allow you to use non-aggregate graphs and match 
hostnames by regex.
 # Default: 'standard'.
 # $default_size - Default size for graphs.  Default: 'large'.
 # $conf_dir - Path to directory where ganglia view JSON files should live.
-# Defaults to the appropriate directory based on WMF $::realm. 
 Default: to /var/lib/ganglia/conf
-# $template - The ERb template to use for the JSON file.  Only change this 
if you need to do fancier things than this define allows.
+# Defaults to the appropriate directory based on WMF $::realm. 
 Default: /var/lib/ganglia/conf.
+# $template - The ERB template to use for the JSON file.  Only change this 
if you need to do fancier things than this define allows.
 #
 # == Examples:
-# # A 'regex' (non aggregate graph) view:
+# # A 'regex' (non-aggregate graph) view:
 # # Note that no aggregate_graphs are used.
 # # This will add 4 graphs to the 'cpu' view.
-# # (i.e. cpu_user and cpu_system for each myhost and myhost1)
+# # (i.e. cpu_user and cpu_system for each myhost0 and myhost1)
 # $host_regex = 'myhost[01]'
 # ganglia::view { 'cpu':
 #   view_type = 'regex',
@@ -662,10 +662,10 @@
 # $plugins - the plugin name (ex: 'diskstat'), will install the Python file
 # located in files/ganglia/plugins/${name}.py and expand the template from
 # templates/ganglia/plugins/${name}.pyconf.erb.
-# Defaults to $title as a convenience
+# Defaults to $title as a convenience.
 #
-# $opts - optional hash which can be used in the template. The defaults are
-# hardcoded in the templates. Default to {}
+# $opts - optional hash which can be used in the template.  The
+# defaults are hardcoded in the templates. Defaults to {}.
 #
 # == Examples:
 #
diff --git a/modules/toollabs/manifests/init.pp 
b/modules/toollabs/manifests/init.pp
index a125e35..48190e5 100644
--- a/modules/toollabs/manifests/init.pp
+++ b/modules/toollabs/manifests/init.pp
@@ -74,7 +74,7 @@
 # Tool Labs is enduser-facing, so we want to control the motd
 # properly (most things make no sense for community users: they
 # don't care that packages need updating, or that filesystems
-# will be checked, for instance)
+# will be checked, for instance).
 
 file { '/etc/update-motd.d':
 ensure  = directory,
@@ -86,7 +86,7 @@
 purge   = true,
 }
 
-# We keep a project-locat apt repo where we stuff packages we build
+# We keep a project-local apt repo where we stuff packages we build
 # that are intended to be local to the project.  By keeping it on the
 # shared storage, we have no need to set up a server to use it.
 
@@ -126,4 +126,3 @@
 target = ${store}/mail,
 }
 }
-

-- 
To view, visit https://gerrit.wikimedia.org/r/107165
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I2aa6e65e4eac8b920bf39fb51089cfaea9b0956f
Gerrit-PatchSet: 2
Gerrit-Project: