Alexandros Kosiaris has submitted this change and it was merged.
Change subject: Fixed spacing and alignment
......................................................................
Fixed spacing and alignment
This fixes lint 'two-space soft tabs not used'.
Change-Id: I78aae2a826ba663711964faaea9040247cd0022e
---
M manifests/role/coredb.pp
M manifests/role/elasticsearch.pp
M manifests/role/grafana.pp
M manifests/role/logstash.pp
M manifests/role/phabricator.pp
M manifests/role/pybal_config.pp
M manifests/role/wikimetrics.pp
M manifests/site.pp
M modules/bacula/manifests/client/job.pp
M modules/diamond/manifests/init.pp
M modules/install-server/manifests/dhcp-server.pp
M modules/labs_lvm/manifests/volume.pp
M modules/postgresql/manifests/spatialdb.pp
M modules/postgresql/manifests/user.pp
M modules/puppet_compiler/manifests/packages.pp
15 files changed, 65 insertions(+), 63 deletions(-)
Approvals:
Alexandros Kosiaris: Looks good to me, approved
jenkins-bot: Verified
diff --git a/manifests/role/coredb.pp b/manifests/role/coredb.pp
index 33b8a13..d6b148d 100644
--- a/manifests/role/coredb.pp
+++ b/manifests/role/coredb.pp
@@ -58,7 +58,7 @@
},
'x1' => {
'hosts' => {
- 'eqiad' => [ 'db1029', 'db1031' ] },
+ 'eqiad' => [ 'db1029', 'db1031' ] },
'primary_site' => $::mw_primary,
'masters' => { 'eqiad' => 'db1029' },
'snapshot' => [ 'db1031' ],
@@ -66,7 +66,7 @@
},
'm1' => {
'hosts' => {
- 'eqiad' => ['db1001', 'db1016'] },
+ 'eqiad' => ['db1001', 'db1016'] },
'primary_site' => $::mw_primary,
'masters' => { 'eqiad' => 'db1001' },
'snapshot' => ['db1016' ],
diff --git a/manifests/role/elasticsearch.pp b/manifests/role/elasticsearch.pp
index 92e1f86..41b4c0a 100644
--- a/manifests/role/elasticsearch.pp
+++ b/manifests/role/elasticsearch.pp
@@ -91,7 +91,7 @@
}
if ($rack == 'Unknown') {
- fail("Don't know rack for $::host")
+ fail("Don't know rack for $::host")
}
$row = regsubst($rack, '^(.).$', '\1' )
diff --git a/manifests/role/grafana.pp b/manifests/role/grafana.pp
index 32f758e..6a51a5f 100644
--- a/manifests/role/grafana.pp
+++ b/manifests/role/grafana.pp
@@ -18,10 +18,10 @@
render_method => 'GET',
},
elasticsearch => {
- type => 'elasticsearch',
- url => "//grafana.${domain_suffix}",
- index => 'grafana-dashboards',
- grafanaDB => true,
+ type => 'elasticsearch',
+ url => "//grafana.${domain_suffix}",
+ index => 'grafana-dashboards',
+ grafanaDB => true,
},
},
},
diff --git a/manifests/role/logstash.pp b/manifests/role/logstash.pp
index 0539da2..d467244 100644
--- a/manifests/role/logstash.pp
+++ b/manifests/role/logstash.pp
@@ -49,11 +49,11 @@
include ::redis::ganglia
class { '::logstash':
- heap_memory_mb => 128,
- # TODO: the multiline filter that is used in several places in the
- # current configuration isn't thread safe and can cause crashes or
- # garbled output when used with more than one thread worker.
- filter_workers => 1,
+ heap_memory_mb => 128,
+ # TODO: the multiline filter that is used in several places in the
+ # current configuration isn't thread safe and can cause crashes or
+ # garbled output when used with more than one thread worker.
+ filter_workers => 1,
}
logstash::input::udp2log { 'mediawiki':
diff --git a/manifests/role/phabricator.pp b/manifests/role/phabricator.pp
index 3994505..026172f 100644
--- a/manifests/role/phabricator.pp
+++ b/manifests/role/phabricator.pp
@@ -32,9 +32,9 @@
'mysql.host' => 'm3-master.eqiad.wmnet',
'storage.default-namespace' => 'phlegal',
'phpmailer.smtp-host' =>
- inline_template('<%= @mail_smarthost.join(";") %>'),
+ inline_template('<%= @mail_smarthost.join(";") %>'),
'metamta.default-address' =>
- '[email protected]',
+ '[email protected]',
'metamta.domain' => 'legalpad.wikimedia.org',
},
}
diff --git a/manifests/role/pybal_config.pp b/manifests/role/pybal_config.pp
index f1a5247..19c715e 100644
--- a/manifests/role/pybal_config.pp
+++ b/manifests/role/pybal_config.pp
@@ -3,11 +3,11 @@
class { '::pybal::web':
vhostnames => [
- 'config-master.eqiad.wmnet',
- 'config-master.codfw.wmnet',
- 'config-master.esams.wmnet',
- 'config-master.ulsfo.wmnet',
- 'config-master.wikimedia.org',
- ]
+ 'config-master.eqiad.wmnet',
+ 'config-master.codfw.wmnet',
+ 'config-master.esams.wmnet',
+ 'config-master.ulsfo.wmnet',
+ 'config-master.wikimedia.org',
+ ]
}
}
diff --git a/manifests/role/wikimetrics.pp b/manifests/role/wikimetrics.pp
index 7ba1e13..5371771 100644
--- a/manifests/role/wikimetrics.pp
+++ b/manifests/role/wikimetrics.pp
@@ -34,7 +34,7 @@
# Otherwise, backup gets turned off.
#
class role::wikimetrics {
- # wikimetrics does not yet run via puppet in production
+ # wikimetrics does not yet run via puppet in production
if $::realm == 'production' {
fail('Cannot include role::wikimetrics in production (yet).')
}
diff --git a/manifests/site.pp b/manifests/site.pp
index c42a68d..ddd843d 100644
--- a/manifests/site.pp
+++ b/manifests/site.pp
@@ -413,9 +413,9 @@
include admin
interface::add_ip6_mapped { 'main': }
rsyslog::conf { 'remote_logstash':
- content => "*.* @logstash1002.eqiad.wmnet:10514",
- priority => 32,
- }
+ content => "*.* @logstash1002.eqiad.wmnet:10514",
+ priority => 32,
+ }
}
node 'beryllium.wikimedia.org' {
@@ -581,9 +581,9 @@
include admin
interface::add_ip6_mapped { 'main': }
rsyslog::conf { 'remote_logstash':
- content => "*.* @logstash1002.eqiad.wmnet:10514",
- priority => 32,
- }
+ content => "*.* @logstash1002.eqiad.wmnet:10514",
+ priority => 32,
+ }
}
node /^cp30(0[3-9]|10|1[5-8])\.esams\.(wikimedia\.org|wmnet)$/ {
diff --git a/modules/bacula/manifests/client/job.pp
b/modules/bacula/manifests/client/job.pp
index 698ad35..4663249 100644
--- a/modules/bacula/manifests/client/job.pp
+++ b/modules/bacula/manifests/client/job.pp
@@ -25,9 +25,9 @@
# }
#
define bacula::client::job(
- $fileset,
- $jobdefaults,
- $extras=undef) {
+ $fileset,
+ $jobdefaults,
+ $extras=undef) {
$director = $::bacula::client::director
diff --git a/modules/diamond/manifests/init.pp
b/modules/diamond/manifests/init.pp
index 3926fcd..42e43c0 100644
--- a/modules/diamond/manifests/init.pp
+++ b/modules/diamond/manifests/init.pp
@@ -58,8 +58,9 @@
$path_prefix = 'servers',
$keep_logs_for = '5',
$service = running,
- $settings = { host => 'localhost',
- port => '8125',
+ $settings = {
+ host => 'localhost',
+ port => '8125',
},
) {
if os_version('debian >= jessie || ubuntu >= precise') {
diff --git a/modules/install-server/manifests/dhcp-server.pp
b/modules/install-server/manifests/dhcp-server.pp
index b1fbac8..64daaa6 100644
--- a/modules/install-server/manifests/dhcp-server.pp
+++ b/modules/install-server/manifests/dhcp-server.pp
@@ -38,9 +38,10 @@
service { $service_name:
ensure => running,
- require => [ Package[$package_name],
- File[$confdir]
- ],
+ require => [
+ Package[$package_name],
+ File[$confdir]
+ ],
subscribe => File[$confdir],
}
}
diff --git a/modules/labs_lvm/manifests/volume.pp
b/modules/labs_lvm/manifests/volume.pp
index 7b4cc4a..89d0a0a 100644
--- a/modules/labs_lvm/manifests/volume.pp
+++ b/modules/labs_lvm/manifests/volume.pp
@@ -42,9 +42,9 @@
unless => "/bin/mountpoint -q '$mountat'",
logoutput => 'on_failure',
require => [
- File['/usr/local/sbin/make-instance-vol'],
- Exec['create-volume-group']
- ],
+ File['/usr/local/sbin/make-instance-vol'],
+ Exec['create-volume-group']
+ ],
command => "/usr/local/sbin/make-instance-vol '$volname' '$size'
'$fstype' $mkfs_opt",
}
@@ -62,9 +62,9 @@
options => $options,
fstype => $fstype,
require => [
- Exec["create-vd-$volname"],
- File[$mountat],
- ],
+ Exec["create-vd-$volname"],
+ File[$mountat],
+ ],
}
labs_lvm::extend { $mountat:
diff --git a/modules/postgresql/manifests/spatialdb.pp
b/modules/postgresql/manifests/spatialdb.pp
index 9000b04..a06ae15 100644
--- a/modules/postgresql/manifests/spatialdb.pp
+++ b/modules/postgresql/manifests/spatialdb.pp
@@ -17,7 +17,7 @@
define postgresql::spatialdb(
$ensure = 'present',
$pg_version = '9.1',
- ) {
+ ) {
# Check if our db exists and store it
$db_exists = "/usr/bin/psql --tuples-only -c \'SELECT datname FROM
pg_catalog.pg_database;\' | /bin/grep \'^ ${name}\'"
diff --git a/modules/postgresql/manifests/user.pp
b/modules/postgresql/manifests/user.pp
index 59dd3f6..c48f89f 100644
--- a/modules/postgresql/manifests/user.pp
+++ b/modules/postgresql/manifests/user.pp
@@ -25,16 +25,16 @@
# Based upon https://github.com/uggedal/puppet-module-postgresql
#
define postgresql::user(
- $user,
- $password,
- $database = 'template1',
- $type = 'host',
- $method = 'md5',
- $cidr = '127.0.0.1/32',
- $pgversion = '9.1',
- $attrs = '',
- $ensure = 'present'
- ) {
+ $user,
+ $password,
+ $database = 'template1',
+ $type = 'host',
+ $method = 'md5',
+ $cidr = '127.0.0.1/32',
+ $pgversion = '9.1',
+ $attrs = '',
+ $ensure = 'present'
+ ) {
# Check if our user exists and store it
$userexists = "/usr/bin/psql --tuples-only -c \'SELECT rolname FROM
pg_catalog.pg_roles;\' | /bin/grep \'^ ${user}\'"
diff --git a/modules/puppet_compiler/manifests/packages.pp
b/modules/puppet_compiler/manifests/packages.pp
index 09344dd..56a6088 100644
--- a/modules/puppet_compiler/manifests/packages.pp
+++ b/modules/puppet_compiler/manifests/packages.pp
@@ -1,16 +1,16 @@
class puppet_compiler::packages($ensure = $puppet_compiler::ensure) {
include stdlib
$list = [
- 'curl',
- 'python-dev',
- 'rubygems',
- 'ruby-bundler',
- 'ruby1.8-dev',
- 'mysql-server',
- 'mysql-client',
- 'ruby-mysql',
- 'ruby-bcrypt',
- 'nginx'
- ]
+ 'curl',
+ 'python-dev',
+ 'rubygems',
+ 'ruby-bundler',
+ 'ruby1.8-dev',
+ 'mysql-server',
+ 'mysql-client',
+ 'ruby-mysql',
+ 'ruby-bcrypt',
+ 'nginx'
+ ]
ensure_packages([$list])
}
--
To view, visit https://gerrit.wikimedia.org/r/181404
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I78aae2a826ba663711964faaea9040247cd0022e
Gerrit-PatchSet: 4
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: KartikMistry <[email protected]>
Gerrit-Reviewer: Alexandros Kosiaris <[email protected]>
Gerrit-Reviewer: Dzahn <[email protected]>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
