Closed
Bug 994267
Opened 11 years ago
Closed 11 years ago
remove machines that don't exist from buildbot-configs
Categories
(Release Engineering :: General, defect)
Release Engineering
General
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: bhearsum, Assigned: bhearsum)
References
Details
Attachments
(3 files)
6.77 KB,
patch
|
rail
:
review+
bhearsum
:
checked-in+
|
Details | Diff | Splinter Review |
5.71 KB,
patch
|
rail
:
review+
bhearsum
:
checked-in+
|
Details | Diff | Splinter Review |
3.20 KB,
patch
|
rail
:
review+
bhearsum
:
checked-in+
|
Details | Diff | Splinter Review |
The report that started running as part of bug 989492 found a ton of machines that exist in buildbot-configs, but aren't in inventory or AWS. That report is here:
https://secure.pub.build.mozilla.org/builddata/reports/reportor/daily/machine_sanity/machine_sanity.txt
We should sync up. This patch should do most of it, although there's a bunch of extra machines due to https://bugzilla.mozilla.org/show_bug.cgi?id=985625, and some off by one errors in AWS. I'd like to land this as a first step and then go from there. Once the huge swaths of AWS machines are cleared out of the report it will be easier to deal with the remaining machines.
I didn't go so far as to verify decomm in bugzilla, but nothing being removed here exists in DNS.
Attachment #8404159 -
Flags: review?(rail)
Updated•11 years ago
|
Attachment #8404159 -
Flags: review?(rail) → review+
Assignee | ||
Updated•11 years ago
|
Attachment #8404159 -
Flags: checked-in+
Comment 1•11 years ago
|
||
Hi Ben,
This caused a jenkins failure (http://10.134.48.37:8080/job/buildbot-configs_tests/1914/) - ok for me to back out?
Thanks,
Pete
Flags: needinfo?(bhearsum)
Comment 2•11 years ago
|
||
Erm, or maybe it didn't - let me double check. It might be another commit...
Comment 3•11 years ago
|
||
False alarm! Was another commit.
Updated•11 years ago
|
Flags: needinfo?(bhearsum)
Assignee | ||
Comment 4•11 years ago
|
||
This is in preparation of removing a bunch of dead tegras/pandas from the slavelist. It's no-op, just reorganization.
Comment 5•11 years ago
|
||
Comment on attachment 8404762 [details] [diff] [review]
make test slave lists more consistent, readable
0 diff!
Attachment #8404762 -
Flags: review?(rail) → review+
Assignee | ||
Updated•11 years ago
|
Attachment #8404762 -
Flags: checked-in+
Assignee | ||
Comment 6•11 years ago
|
||
I verified all of these with slave health/bugs.
Attachment #8404875 -
Flags: review?(rail)
Comment 7•11 years ago
|
||
Comment on attachment 8404875 [details] [diff] [review]
remove decommed tegras and pandas
stamp
Attachment #8404875 -
Flags: review?(rail) → review+
Assignee | ||
Updated•11 years ago
|
Attachment #8404875 -
Flags: checked-in+
Assignee | ||
Comment 8•11 years ago
|
||
Pretty sure we're done here now...
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
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
•