[MediaWiki-commits] [Gerrit] Fixing lint issues - change (operations/puppet)

2016-06-08 Thread Gehel (Code Review) 
Gehel has submitted this change and it was merged.

Change subject: Fixing lint issues
..


Fixing lint issues

those issue were probably introduced while Jenkins was taking a nap...

Change-Id: I98ebd391b1ca29d8023e0bbaf37a83e14ea8bca7
---
M modules/role/manifests/ci/master.pp
M modules/role/manifests/zookeeper/client.pp
2 files changed, 6 insertions(+), 6 deletions(-)

Approvals:
  Gehel: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/modules/role/manifests/ci/master.pp 
b/modules/role/manifests/ci/master.pp
index 3b351ab..0bbd5e3 100644
--- a/modules/role/manifests/ci/master.pp
+++ b/modules/role/manifests/ci/master.pp
@@ -71,10 +71,10 @@
 require contint::master_dir
 
 file { '/srv/ssd/jenkins':
-ensure  => 'directory',
-owner   => 'jenkins',
-group   => 'jenkins',
-mode=> '2775',  # group sticky bit
+ensure => 'directory',
+owner  => 'jenkins',
+group  => 'jenkins',
+mode   => '2775',  # group sticky bit
 }
 
 # Master does not run job anymore since June 2013. But better safe than
diff --git a/modules/role/manifests/zookeeper/client.pp 
b/modules/role/manifests/zookeeper/client.pp
index f0adf1e..66a0a53 100644
--- a/modules/role/manifests/zookeeper/client.pp
+++ b/modules/role/manifests/zookeeper/client.pp
@@ -29,8 +29,8 @@
 
 require_package('openjdk-7-jdk')
 class { '::zookeeper':
-hosts  => $hosts,
-version=> $version,
+hosts   => $hosts,
+version => $version,
 }
 }
 

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I98ebd391b1ca29d8023e0bbaf37a83e14ea8bca7
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Gehel 
Gerrit-Reviewer: Gehel 
Gerrit-Reviewer: jenkins-bot <>

___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

[MediaWiki-commits] [Gerrit] Fixing lint issues - change (operations/puppet)

2016-06-08 Thread Gehel (Code Review) 
Gehel has uploaded a new change for review.

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

Change subject: Fixing lint issues
..

Fixing lint issues

those issue were probably introduced while Jenkins was taking a nap...

Change-Id: I98ebd391b1ca29d8023e0bbaf37a83e14ea8bca7
---
M modules/role/manifests/ci/master.pp
M modules/role/manifests/zookeeper/client.pp
2 files changed, 6 insertions(+), 6 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/operations/puppet 
refs/changes/77/293377/1

diff --git a/modules/role/manifests/ci/master.pp 
b/modules/role/manifests/ci/master.pp
index 3b351ab..0bbd5e3 100644
--- a/modules/role/manifests/ci/master.pp
+++ b/modules/role/manifests/ci/master.pp
@@ -71,10 +71,10 @@
 require contint::master_dir
 
 file { '/srv/ssd/jenkins':
-ensure  => 'directory',
-owner   => 'jenkins',
-group   => 'jenkins',
-mode=> '2775',  # group sticky bit
+ensure => 'directory',
+owner  => 'jenkins',
+group  => 'jenkins',
+mode   => '2775',  # group sticky bit
 }
 
 # Master does not run job anymore since June 2013. But better safe than
diff --git a/modules/role/manifests/zookeeper/client.pp 
b/modules/role/manifests/zookeeper/client.pp
index f0adf1e..66a0a53 100644
--- a/modules/role/manifests/zookeeper/client.pp
+++ b/modules/role/manifests/zookeeper/client.pp
@@ -29,8 +29,8 @@
 
 require_package('openjdk-7-jdk')
 class { '::zookeeper':
-hosts  => $hosts,
-version=> $version,
+hosts   => $hosts,
+version => $version,
 }
 }
 

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I98ebd391b1ca29d8023e0bbaf37a83e14ea8bca7
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Gehel 

___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits