Closed Bug 815680 Opened 11 years ago Closed 11 years ago

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


(Release Engineering :: General, defect)

Not set


(Not tracked)



(Reporter: armenzg, Assigned: armenzg)




(4 files, 1 obsolete file)

Once we use the whole pool again we should take the subset of slaves and re-image them like the rest of the pool.
Depends on: enable-8.0sdk
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:
which leads to:
Attached patch remove win64-metro platform (obsolete) — Splinter Review
Attachment #688003 - Flags: review?(coop)
Attachment #688004 - Flags: review?(coop)
Comment on attachment 688003 [details] [diff] [review]
remove win64-metro platform

wrong patch
Attachment #688003 - Flags: review?(coop)
Attachment #688003 - Attachment is obsolete: true
Attachment #688024 - Flags: review?(coop)
Blocks: metro-releng
No longer depends on: enable-8.0sdk
Attachment #688004 - Flags: review?(coop) → review+
Comment on attachment 688024 [details] [diff] [review]
remove win64-metro platform

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

r+ with the nit fixed.

::: mozilla/
@@ -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?
Attachment #688024 - Flags: review?(coop) → review+
Attachment #688024 - Flags: checked-in+
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.
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.
Attachment #689303 - Flags: review?(coop)
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?
Attachment #689303 - Flags: review?(coop) → review+
Comment on attachment 689303 [details] [diff] [review]
fix list of slaves

I completely missed it :S
Attachment #689303 - Flags: checked-in+
Attachment #688004 - Flags: checked-in+
Assignee: nobody → armenzg
Attachment #689723 - Flags: review?(catlee)
Attachment #689723 - Flags: review?(catlee) → review+
Attachment #689723 - Flags: checked-in+
Blocks: 819357
Closed: 11 years ago
Resolution: --- → FIXED
Product: → Release Engineering
Component: General Automation → General
You need to log in before you can comment on or make changes to this bug.