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) — 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) —
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.patchSplinter Review
Attachment #8666680 - Flags: review?(jmaher)
Attachment #8666118 - Attachment is obsolete: true
Attachment #8666118 - Flags: review?(jlund)
Assignee: nobody → kmoir
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) — Splinter Review
add new range of windows test machines and decomm ubuntu32_hw class of machines
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
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: