Hello Chasemp, Merlijn van Deen, I'd like you to do a code review. Please visit
https://gerrit.wikimedia.org/r/339921 to review the following change. Change subject: Tools: Require gridengine-master for gridengine_resource ...................................................................... Tools: Require gridengine-master for gridengine_resource Bug: T127388 Change-Id: Ief54f613b2cdcb912215b22300f9a3494fdfd565 --- M modules/toollabs/manifests/master.pp 1 file changed, 3 insertions(+), 0 deletions(-) git pull ssh://gerrit.wikimedia.org:29418/operations/puppet refs/changes/21/339921/1 diff --git a/modules/toollabs/manifests/master.pp b/modules/toollabs/manifests/master.pp index ea8e58d..d21091c 100644 --- a/modules/toollabs/manifests/master.pp +++ b/modules/toollabs/manifests/master.pp @@ -11,6 +11,7 @@ ensure => present, requestable => 'FORCED', consumable => 'YES', + require => Service['gridengine-master'], } gridengine_resource { 'release': @@ -22,6 +23,7 @@ consumable => 'NO', default => 'NONE', urgency => '0', + require => Service['gridengine-master'], } gridengine_resource { 'user_slot': @@ -33,6 +35,7 @@ consumable => 'YES', default => '0', urgency => '0', + require => Service['gridengine-master'], } file { "${toollabs::collectors}/hostgroups": -- To view, visit https://gerrit.wikimedia.org/r/339921 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: newchange Gerrit-Change-Id: Ief54f613b2cdcb912215b22300f9a3494fdfd565 Gerrit-PatchSet: 1 Gerrit-Project: operations/puppet Gerrit-Branch: production Gerrit-Owner: Tim Landscheidt <t...@tim-landscheidt.de> Gerrit-Reviewer: Chasemp <r...@wikimedia.org> Gerrit-Reviewer: Merlijn van Deen <valhall...@arctus.nl> _______________________________________________ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits