815680 - Elm branch should use the whole pool of Win64 builders rather than a subset

Status: RESOLVED FIXED

(Release Engineering :: General, defect)

Description

[Armen [:armenzg]]

Once we use the whole pool again we should take the subset of slaves and re-image them like the rest of the pool.

Comment 1

[Armen [:armenzg]]

After speaking with dustin, we will push the re-imaging of the custom builders later on as we don't currently have a Windows guy.

For now, we will add post-imaging steps for win64 slaves:
https://wiki.mozilla.org/ReleaseEngineering/How_To/Set_Up_a_Freshly_Imaged_Slave#Windows_2008_64-bit_.28MDT_.26_unmanaged.29
which leads to:
https://wiki.mozilla.org/ReferencePlatforms/Win64#Post-reimaging_steps

Comment 2

[Armen [:armenzg]]

Attachment: remove win64-metro platform

Comment 3

[Armen [:armenzg]]

Attachment: puppet changes for win64-metro

Comment 4

[Armen [:armenzg]]

Comment on attachment 688003 [details] [diff] [review]
remove win64-metro platform

wrong patch

Comment 5

[Armen [:armenzg]]

Attachment: remove win64-metro platform

Comment 6

[Chris Cooper [:coop] (he/him)]

Comment on attachment 688024 [details] [diff] [review]
remove win64-metro platform

Review of attachment 688024 [details] [diff] [review]:
-----------------------------------------------------------------

r+ with the nit fixed.

::: mozilla/production_config.py
@@ -12,1 @@
>  MOCK_DL120G7   = ['bld-centos6-hp-%03d' % x for x in range(6,24)] # 5 staging, 17 prod, 17 try

Shouldn't we remove the WIN64_METRO definition from line 11 also?

Comment 7

[Jim Mathies [:jimm]]

We're getting some exception on the elm builders, wondering if this checkin might be the cause?

remoteFailed: [Failure instance: Traceback (failure with no frames): <class 'twisted.spread.pb.PBConnectionLost'>: [Failure instance: Traceback (failure with no frames): <class 'twisted.internet.error.ConnectionDone'>: Connection was closed cleanly.

https://tbpl.mozilla.org/?noignore=0&tree=Elm

Comment 8

[Armen [:armenzg]]

Attachment: fix list of slaves

The slaves that took the jobs where the known ones:
WIN64_METRO    = ['w64-ix-slave%02i' % x for x in [11,20,42,43]] 

Unfortunately without this patch those 5 slaves would get the jobs assigned but in reality they had no master to talk to.

They were living dead; caught between two realms.

I will shut them down until this patch makes it into production.

Comment 9

[Chris Cooper [:coop] (he/him)]

Comment on attachment 689303 [details] [diff] [review]
fix list of slaves

Review of attachment 689303 [details] [diff] [review]:
-----------------------------------------------------------------

Hrmmm, didn't I suggest this already with the previous patch in comment #6?

Comment 10

[Armen [:armenzg]]

Comment on attachment 689303 [details] [diff] [review]
fix list of slaves

I completely missed it :S

Comment 11

[Armen [:armenzg]]

Attachment: enable bundles for Elm