885164 - update nagios buildbot check for bld-lion-r5*

Status: RESOLVED FIXED

(Infrastructure & Operations Graveyard :: CIDuty, task)

Description

[Amy Rich [:arr] [:arich]]

The nagios check for buildbot looks for a process named puppet.  The new executable is called puppet2.7.  This needs to be updated in /etc/nagios/nrpe.d/check_buildbot.cfg.  The hosts are downtimed for 24 hours right now.

Comment 1

[Justin Wood (:Callek)]

(In reply to Amy Rich [:arich] [:arr] from comment #0)
> The nagios check for buildbot looks for a process named puppet.  The new
> executable is called puppet2.7.  This needs to be updated in

Puppet is on your mind, rephrased properly:

The nagios check for buildbot looks for a process named python.  The new
executable is called python2.7.  This needs to be updated in

Comment 2

[Amy Rich [:arr] [:arich]]

Wow, yeah, long day.

Comment 3

[Chris Cooper [:coop] (he/him)]

Can we stop checking for |-C python| and just match on |--argument-array=buildbot.tac| in the interim?

https://hg.mozilla.org/build/puppet/file/9a17bf8e40fb/modules/nrpe/manifests/check/buildbot.pp#l13

Comment 4

[Dustin J. Mitchell [:dustin] (he/him)]

I suspect the -C is required, but maybe.  I'm not going to get to this in the next few hours, if you want to steal it and try that out.

Comment 5

[Chris Cooper [:coop] (he/him)]

I've downtimed the nagios checks for bld-lion* until Monday.

Comment 6

[Dustin J. Mitchell [:dustin] (he/him)]

Attachment: bug885164.patch

You were right: -C is optional