Ottomata has submitted this change and it was merged.

Change subject: burrow: fix optional parameter listed before required parameter
......................................................................


burrow: fix optional parameter listed before required parameter

Change-Id: I7d06b34f58490b362bfc3ad52ce27c4d28105356
---
M modules/burrow/manifests/init.pp
1 file changed, 4 insertions(+), 4 deletions(-)

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



diff --git a/modules/burrow/manifests/init.pp b/modules/burrow/manifests/init.pp
index daf37a8..869902b 100644
--- a/modules/burrow/manifests/init.pp
+++ b/modules/burrow/manifests/init.pp
@@ -20,10 +20,6 @@
 # $email_template     - The name of the email template to use for Burrow's 
alerts
 
 class burrow (
-    $ensure = 'present',
-    $client_id = 'burrow-client',
-    $httpserver_port = 8000,
-    $lagcheck_intervals = 10,
     $zookeeper_hosts,
     $zookeeper_path,
     $kafka_cluster_name,
@@ -32,6 +28,10 @@
     $smtp_server,
     $from_email,
     $to_emails,
+    $ensure = 'present',
+    $client_id = 'burrow-client',
+    $httpserver_port = 8000,
+    $lagcheck_intervals = 10,
     $email_template = 'burrow/email.tmpl.erb'
 
 )

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I7d06b34f58490b362bfc3ad52ce27c4d28105356
Gerrit-PatchSet: 3
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Paladox <thomasmulhall...@yahoo.com>
Gerrit-Reviewer: Alexandros Kosiaris <akosia...@wikimedia.org>
Gerrit-Reviewer: Dzahn <dz...@wikimedia.org>
Gerrit-Reviewer: Hashar <has...@free.fr>
Gerrit-Reviewer: Madhuvishy <mviswanat...@wikimedia.org>
Gerrit-Reviewer: Ottomata <o...@wikimedia.org>
Gerrit-Reviewer: jenkins-bot <>

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

Reply via email to