Yuvipanda has submitted this change and it was merged.

Change subject: toollabs: Use hiera to figure out where cron runner host is
......................................................................


toollabs: Use hiera to figure out where cron runner host is

Bug: T123873
Change-Id: I0699455b2d6da425df8baba681d645cbc40298f7
---
M modules/toollabs/manifests/bastion.pp
1 file changed, 2 insertions(+), 3 deletions(-)

Approvals:
  Yuvipanda: Verified; Looks good to me, approved



diff --git a/modules/toollabs/manifests/bastion.pp 
b/modules/toollabs/manifests/bastion.pp
index 15fc282..73e002c 100644
--- a/modules/toollabs/manifests/bastion.pp
+++ b/modules/toollabs/manifests/bastion.pp
@@ -10,9 +10,7 @@
 #
 # Sample Usage:
 #
-class toollabs::bastion(
-    $cron_host = 'tools-submit',
-) inherits toollabs {
+class toollabs::bastion inherits toollabs {
 
     include gridengine::submit_host,
             gridengine::admin_host,
@@ -58,6 +56,7 @@
     include ldap::role::config::labs
     $ldapconfig = $ldap::role::config::labs::ldapconfig
 
+    $cron_host = hiera('active_cronrunner')
     file { '/usr/local/bin/crontab':
         ensure  => file,
         mode    => '0755',

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I0699455b2d6da425df8baba681d645cbc40298f7
Gerrit-PatchSet: 2
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Yuvipanda <yuvipa...@wikimedia.org>
Gerrit-Reviewer: Yuvipanda <yuvipa...@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