Closed Bug 1173458 Opened 9 years ago Closed 9 years ago

Add nagios checks for prod signing workers

Categories

(Release Engineering :: General, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: rail, Assigned: rail)

References

Details

Attachments

(1 file, 1 obsolete file)

assigning to myself for now
Attached patch signing-workers-nagios.diff (obsolete) (deleted) — Splinter Review
something like this?
Attachment #8620930 - Flags: review?(arich)
Comment on attachment 8620930 [details] [diff] [review] signing-workers-nagios.diff Confusingly, the use1.pp and usw2.pp files are only used to check connectivity between regions and scl3. You want to put the host definitions in scl3.pp. And for consistency sake, I would give the service description as 'procs - signing-worker' to match what we have for other checks (though I see a bit of inconsistency has bled in, I should go back and clean that up). Otherwise, I think this looks good.
Attachment #8620930 - Flags: review?(arich) → review-
Attached patch signing-workers-nagios.diff (deleted) — Splinter Review
Attachment #8620930 - Attachment is obsolete: true
Attachment #8620935 - Flags: review?(arich)
Comment on attachment 8620935 [details] [diff] [review] signing-workers-nagios.diff Change the service description to 'procs - signing-worker' and ship it. :}
Attachment #8620935 - Flags: review?(arich) → review+
ah, I misread the initial comment. Fixing, pushing.
Comment on attachment 8620935 [details] [diff] [review] signing-workers-nagios.diff Sending hostgroups.pp Sending scl3.pp Sending services.pp Transmitting file data ... Committed revision 105311.
Attachment #8620935 - Flags: checked-in+
Fixed the regexp from /builds/signing-worker/bin/signing-worker to /builds/signingworker/bin/signing-worker in revision 105313.
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Component: General Automation → General
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: