Closed Bug 1295196 Opened 8 years ago Closed 8 years ago

add nagios checks for signing scriptworkers

Categories

(Release Engineering :: General, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: mozilla, Assigned: mozilla)

References

Details

Attachments

(2 files)

Puppet creates the nagios config on the signing scriptworkers, but we need nagios-side config changes to actually add the checks. Per bug 1173458 and bug 1186882, it looks like nagios configs are/were in subversion.
:arr, should I follow https://mana.mozilla.org/wiki/pages/viewpage.action?pageId=5734910 and ask for that svn access, or am I looking at the wrong place?
Flags: needinfo?(arich)
You're in the authz file already, but I bet that IT needs to enable svn access for you in LDAP.
Flags: needinfo?(arich)
This is the main portion. I'll write a followup patch when we spin up more signing scriptworkers. (Once we port the old tasks over from the existing signingworkers, I could also see a rename. But for now they're signing scriptworkers.)
Attachment #8781723 - Flags: review?(arich)
Attachment #8781723 - Flags: review?(arich) → review+
Comment on attachment 8781723 [details] [diff] [review] signing_scriptworker_nagios.diff (root) akimoz:/src/svn/nagios [16:54:37] 745$ svn ci Sending manifests/releng/hostgroups.pp Sending manifests/releng/scl3.pp Sending manifests/releng/services.pp Transmitting file data ... Committed revision 120853.
Attachment #8781723 - Flags: checked-in+
Keeping this bug open for the next 3? signing scriptworkers.
This should close out this bug.
Attachment #8783649 - Flags: review?(arich)
Comment on attachment 8783649 [details] [diff] [review] add_signing-linux-2-4.diff Review of attachment 8783649 [details] [diff] [review]: ----------------------------------------------------------------- One thing I noticed here is that there's no parent definition (I missed that on the first one). They should all have the firewall as their parent. parents => 'fw1.private.releng.scl3.mozilla.net',
Attachment #8783649 - Flags: review?(arich) → review+
Comment on attachment 8783649 [details] [diff] [review] add_signing-linux-2-4.diff Checked in, with parents set for signing-linux-{1..4} Sending manifests/releng/scl3.pp Transmitting file data . Committed revision 121008.
Attachment #8783649 - Flags: checked-in+
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
See Also: → 1321513
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: