Closed
Bug 1173458
Opened 9 years ago
Closed 9 years ago
Add nagios checks for prod signing workers
Categories
(Release Engineering :: General, defect)
Release Engineering
General
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: rail, Assigned: rail)
References
Details
Attachments
(1 file, 1 obsolete file)
(deleted),
patch
|
arich
:
review+
rail
:
checked-in+
|
Details | Diff | Splinter Review |
assigning to myself for now
Comment 2•9 years ago
|
||
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-
Assignee | ||
Comment 3•9 years ago
|
||
Attachment #8620930 -
Attachment is obsolete: true
Attachment #8620935 -
Flags: review?(arich)
Comment 4•9 years ago
|
||
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+
Assignee | ||
Comment 5•9 years ago
|
||
ah, I misread the initial comment. Fixing, pushing.
Assignee | ||
Comment 6•9 years ago
|
||
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+
Assignee | ||
Comment 7•9 years ago
|
||
Fixed the regexp from /builds/signing-worker/bin/signing-worker to /builds/signingworker/bin/signing-worker in revision 105313.
Assignee | ||
Updated•9 years ago
|
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Updated•8 years ago
|
Updated•7 years ago
|
Component: General Automation → General
You need to log in
before you can comment on or make changes to this bug.
Description
•