Closed Bug 1231367 Opened 10 years ago Closed 10 years ago

Kill Jacuzzis

Categories

(Release Engineering :: General, defect)

Product:
Release Engineering
Component:
General
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: catlee, Assigned: catlee)

Details

Attachments

(4 files)

bb-configs.diff
3.50 KB, patch
catlee
: review+
rail
: checked-in+
Details | Diff | Splinter Review
remove jacuzzis from buildbotcustom
11.64 KB, patch
rail
: review+
catlee
: checked-in+
Details | Diff | Splinter Review
[slave_health] v1
29.92 KB, patch
coop
: review+
Details | Diff | Splinter Review
dejacuzzi-puppet.diff
14.46 KB, patch
rail
: review+
catlee
: checked-in+
Details | Diff | Splinter Review
We're going to kill off jacuzzis. This bug tracks work involved.
Attached patch bb-configs.diff — — Splinter Review
Attachment #8697003 - Flags: review?(catlee)
Attachment #8697005 - Flags: review?(rail)
Attachment #8697003 - Flags: review?(catlee) → review+
callek@Centaurus:~/mozilla/hg/slave_health$ hg diff --stat .hgignore | 3 - css/slave_health.css | 28 ++---- index.html | 2 +- jacuzzis.html | 376 ++-------------------------------------------------------------------------------------------- js/slave_health.js | 154 -------------------------------------- scripts/slave_health.py | 1 - slave.html | 7 - slavetype.html | 2 +- 8 files changed, 21 insertions(+), 552 deletions(-)
Attachment #8697007 - Flags: review?(coop)
Attachment #8697005 - Flags: review?(rail) → review+
Attachment #8697005 - Flags: checked-in+
Comment on attachment 8697007 [details] [diff] [review] [slave_health] v1 Review of attachment 8697007 [details] [diff] [review]: ----------------------------------------------------------------- ::: jacuzzis.html @@ +1,3 @@ > <html> > + <head> > + <title>The Tudors</title> Huh? ::: scripts/slave_health.py @@ +372,1 @@ > if slave_types["all_slaves"].has_key(slave_name): Do we still need all_slaves at all?
Attachment #8697007 - Flags: review?(coop) → review+
(In reply to Chris Cooper [:coop] from comment #6) > Comment on attachment 8697007 [details] [diff] [review] > [slave_health] v1 > > Review of attachment 8697007 [details] [diff] [review]: > ----------------------------------------------------------------- > > ::: jacuzzis.html > @@ +1,3 @@ > > <html> > > + <head> > > + <title>The Tudors</title> > > Huh? Haha, copy/paste fail (from a meta redirect example I found online). I just fixed. > ::: scripts/slave_health.py > @@ +372,1 @@ > > if slave_types["all_slaves"].has_key(slave_name): > > Do we still need all_slaves at all? Honestly, not sure. I *think* something else (outside of slave health) uses it, but I didn't want to gamble atm and just leave it in for now. We can remove and risk it though, at your (or others) discretion. Pushed the existing patch (+ title fix) with: https://hg.mozilla.org/build/slave_health/rev/a5715dfec263
Remove jacuzzi support in puppet. I'm not sure where this file actually gets used...
Attachment #8697030 - Flags: review?(rail)
Comment on attachment 8697030 [details] [diff] [review] dejacuzzi-puppet.diff Review of attachment 8697030 [details] [diff] [review]: ----------------------------------------------------------------- lgtm
Attachment #8697030 - Flags: review?(rail) → review+
Just realized we may have to do something to keep thunderbird l10n jobs from consuming the OSX build pool in the morning
Attachment #8697030 - Flags: checked-in+
Assignee: nobody → catlee
Dustin, I need some help disabling this on relengwebadm.private.scl3. Ideally we disable the cronjob, wait a few days, and then remove everything. Is that hard to do?
Flags: needinfo?(dustin)
I removed the cronjob from puppet and then manually deleted it.
Flags: needinfo?(dustin)
No ill effects detected. Can we purge the jacuzzi stuff from relengwebadm.private.scl3 now?
Flags: needinfo?(dustin)
OK, I moved /data/releng/src/jacuzzi-allocator{,.disabled}, deleted the CNAME, and removed all puppet config for the site.
Flags: needinfo?(dustin)
I think we're done here?
Status: NEW → RESOLVED
Closed: 10 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

Creator:
Created:
Updated:
Size: