Closed Bug 1208449 Opened 9 years ago Closed 9 years ago

disable linux32 talos testing and reimage machines for windows testing

Categories

(Infrastructure & Operations Graveyard :: CIDuty, task)

task
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: kmoir, Assigned: kmoir)

References

Details

Attachments

(6 files, 3 obsolete files)

No description provided.
Attached patch bug1208449.patch (obsolete) (deleted) — Splinter Review
patch to disable linux32 talos builders. I'll clean up the configs to remove that machine class on a subsequent patch/reconfig.
Attached file bug1208449builder.diff (obsolete) (deleted) —
builder diff
Attachment #8666118 - Flags: review?(jlund)
From email conversation Armen suggests a pool allocation for the reimaged machines ====== I would say 70% win7 and 30% win8. Current backlog (which looks like this most of the time): win7-ix (2029) win8-ix (1198) winxp-ix (86) ======
Depends on: 1204920
Comment on attachment 8666118 [details] [diff] [review] bug1208449.patch Review of attachment 8666118 [details] [diff] [review]: ----------------------------------------------------------------- ::: mozilla-tests/config.py @@ -207,2 @@ > NO_WINXP = [platform for platform in ALL_TALOS_PLATFORMS if platform != 'xp-ix'] > NO_OSX = get_talos_slave_platforms(PLATFORMS, platforms=('linux', 'linux64', 'win32', 'win64')) we have another instance of 'linux' here.
Attached patch bug1208449.patch (deleted) — Splinter Review
Attachment #8666680 - Flags: review?(jmaher)
Attachment #8666118 - Attachment is obsolete: true
Attachment #8666118 - Flags: review?(jlund)
Assignee: nobody → kmoir
Attached patch bug1208449builder.diff (deleted) — Splinter Review
builder diff
Attachment #8666119 - Attachment is obsolete: true
Comment on attachment 8666680 [details] [diff] [review] bug1208449.patch Review of attachment 8666680 [details] [diff] [review]: ----------------------------------------------------------------- Leaving the r? on joel, since I'd like a "written down" comment from someone on ateam in support of this plan. But it doesn't have to be joel, and doesn't have to be on this bug/patch. (so may already exist) The code itself however, looks great.
Attachment #8666680 - Flags: review+
r+ from me as well! We are all sorting this out together!
Attachment #8666680 - Flags: review?(jmaher) → review+
Attachment #8666680 - Flags: checked-in+
Depends on: 1209093
in production since last reconfig at 8am PT. Will disable machines in slavealloc next
the 44 machines that were enabled are now disabled in slavealloc
Depends on: 1209492
Attached patch bug1208449bb.patch (obsolete) (deleted) — Splinter Review
add new range of windows test machines and decomm ubuntu32_hw class of machines
Attached patch bug1208449tools.patch (deleted) — Splinter Review
Attached patch bug1208449puppet.patch (deleted) — Splinter Review
Attached patch bug1208449bb.patch (deleted) — Splinter Review
add new range of windows test machines and decomm ubuntu32_hw class of machines
Attachment #8667490 - Attachment is obsolete: true
Attachment #8667492 - Flags: review?(bugspam.Callek)
Attachment #8667494 - Flags: review?(bugspam.Callek)
Attachment #8667497 - Flags: review?(bugspam.Callek)
Attachment #8667492 - Flags: review?(bugspam.Callek) → review+
Attachment #8667494 - Flags: review?(bugspam.Callek) → review+
Comment on attachment 8667497 [details] [diff] [review] bug1208449bb.patch Review of attachment 8667497 [details] [diff] [review]: ----------------------------------------------------------------- r+ because config is sane, but there is a gap here I think. ::: mozilla-tests/production_config.py @@ -16,5 @@ > > for i in range(1, 173): > SLAVES['xp-ix']['t-xp32-ix-%03i' % i] = {} > > -for i in range(1, 173): count(new_machines) == count(old_machines) + 31 @@ -21,3 @@ > SLAVES['win7-ix']['t-w732-ix-%03i' % i] = {} > > -for i in range(1, 102) + range(103, 181): # Omit 102 for win10 // Bug 1191481 count(new_machines) == count(old_machines) + 14 @@ -58,5 @@ > for i in range(1, 2100): > SLAVES['ubuntu64_vm']['tst-linux64-spot-%03i' % i] = {} > > -for i in range(1, 56): > - SLAVES['ubuntu32_hw']['talos-linux32-ix-%03i' % i] = {} count(removed_machines) > count(added_machines) What happens to the extra 11 machines?
Attachment #8667497 - Flags: review?(bugspam.Callek) → review+
Attached file bug1208449db.txt (deleted) —
machines to add vi dbimport
Comment on attachment 8667518 [details] bug1208449db.txt added to slavealloc, disabled
re comment #16. there aren't an extra 11 machines. I suspect the range wasn't cleaned up when we last moved things around. According to slave health there were only 45 machines in production until yesterday.
Attachment #8667497 - Flags: checked-in+
Attachment #8667492 - Flags: checked-in+
Attachment #8667494 - Flags: checked-in+
This can be closed, there are only a few machines left to be brought up in bug 1209492 (Q is working through reimaging issues)
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Component: Platform Support → Buildduty
Product: Release Engineering → Infrastructure & Operations
Product: Infrastructure & Operations → Infrastructure & Operations Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: