Re: Freeze Break Request: Allow basset to use up to 6 workers

2016-06-07 Thread Ricky Elrod
+1

-re

On 06/07/2016 06:02 PM, Patrick Uiterwijk wrote:
> Hi,
> 
> Currently the Nagios config allows Basset to run only exactly 1 worker
> before it starts complaining, but sometimes I start extra workers to catch
> up on big queues.
> Can I get +1s on the following patch to make sure that this is no longer
> reason for Nagios to start complaining?
> 
> Patrick
> 
> 
> commit 13b1dbddec43e351489fe639592160bd6bdf3238
> Author: Patrick Uiterwijk 
> Date:   Tue Jun 7 22:01:18 2016 +
> 
> Allow Basset to go up to 6 workers
> 
> Signed-off-by: Patrick Uiterwijk 
> 
> diff --git a/roles/nagios/client/templates/check_basset.cfg.j2
> b/roles/nagios/client/templates/check_basset.cfg.j2
> index 6b179ff..4da827b 100644
> --- a/roles/nagios/client/templates/check_basset.cfg.j2
> +++ b/roles/nagios/client/templates/check_basset.cfg.j2
> @@ -1,4 +1,4 @@
>  command[check_mongo_proc]={{ libdir }}/nagios/plugins/check_procs -s
> RSD -u mongodb -C mongod -c 1:1
>  command[check_rabbitmq_proc]={{ libdir }}/nagios/plugins/check_procs
> -s RSD -u rabbitmq -C beam.smp -c 1:1
> -command[check_worker_proc]={{ libdir }}/nagios/plugins/check_procs -s
> RSD -u basset-worker -C basset-worker -c 1:1
> +command[check_worker_proc]={{ libdir }}/nagios/plugins/check_procs -s
> RSD -u basset-worker -C basset-worker -c 1:6
>  command[check_basset_queue]={{ libdir
> }}/nagios/plugins/check_rabbitmq_size check_submission
> ___
> infrastructure mailing list
> infrastructure@lists.fedoraproject.org
> https://lists.fedoraproject.org/admin/lists/infrastructure@lists.fedoraproject.org
> 




signature.asc
Description: OpenPGP digital signature
___
infrastructure mailing list
infrastructure@lists.fedoraproject.org
https://lists.fedoraproject.org/admin/lists/infrastructure@lists.fedoraproject.org


Re: Freeze Break Request: Allow basset to use up to 6 workers

2016-06-07 Thread Kevin Fenzi
+1 here. 

Less alerts is good. 

kevin


pgpYpTqXxUP7H.pgp
Description: OpenPGP digital signature
___
infrastructure mailing list
infrastructure@lists.fedoraproject.org
https://lists.fedoraproject.org/admin/lists/infrastructure@lists.fedoraproject.org


Freeze Break Request: Allow basset to use up to 6 workers

2016-06-07 Thread Patrick Uiterwijk
Hi,

Currently the Nagios config allows Basset to run only exactly 1 worker
before it starts complaining, but sometimes I start extra workers to catch
up on big queues.
Can I get +1s on the following patch to make sure that this is no longer
reason for Nagios to start complaining?

Patrick


commit 13b1dbddec43e351489fe639592160bd6bdf3238
Author: Patrick Uiterwijk 
Date:   Tue Jun 7 22:01:18 2016 +

Allow Basset to go up to 6 workers

Signed-off-by: Patrick Uiterwijk 

diff --git a/roles/nagios/client/templates/check_basset.cfg.j2
b/roles/nagios/client/templates/check_basset.cfg.j2
index 6b179ff..4da827b 100644
--- a/roles/nagios/client/templates/check_basset.cfg.j2
+++ b/roles/nagios/client/templates/check_basset.cfg.j2
@@ -1,4 +1,4 @@
 command[check_mongo_proc]={{ libdir }}/nagios/plugins/check_procs -s
RSD -u mongodb -C mongod -c 1:1
 command[check_rabbitmq_proc]={{ libdir }}/nagios/plugins/check_procs
-s RSD -u rabbitmq -C beam.smp -c 1:1
-command[check_worker_proc]={{ libdir }}/nagios/plugins/check_procs -s
RSD -u basset-worker -C basset-worker -c 1:1
+command[check_worker_proc]={{ libdir }}/nagios/plugins/check_procs -s
RSD -u basset-worker -C basset-worker -c 1:6
 command[check_basset_queue]={{ libdir
}}/nagios/plugins/check_rabbitmq_size check_submission
___
infrastructure mailing list
infrastructure@lists.fedoraproject.org
https://lists.fedoraproject.org/admin/lists/infrastructure@lists.fedoraproject.org