Closed Bug 1217534 Opened 9 years ago Closed 9 years ago

linux64 talos is not running on mozilla-aurora or mozilla-beta

Categories

(Release Engineering :: General, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: jmaher, Assigned: kmoir)

Details

Attachments

(2 files, 2 obsolete files)

wlach pointed out that we don't run linux64 talos on aurora or beta.  This stopped on Oct 13th.
:Callek, can you help us figure out what went wrong here.  I looked at a few things and couldn't determine the root cause.
Flags: needinfo?(bugspam.Callek)
Assignee: nobody → kmoir
Attached patch bug1217534.patch (obsolete) — Splinter Review
The problem is that 6b1e88013eba added gtests after the get_talos part due to an ordering issue which changed the talos platform to ubuntu64_vm instead of ubuntu64_hw and there weren't any builders associated with it.  In any case, I don't really to wrestle that ordering issue again so I'll just explicitly set the talos platform for m-a and m-b.  I didn't see that when I was testing the other bug because I was testing on a master configured for linux64 vms, not linux64 hw to match our production systems.
Attached file builder.diff (obsolete) —
Attachment #8677679 - Flags: review?(jmaher)
Flags: needinfo?(bugspam.Callek)
Attachment #8677679 - Flags: review?(jmaher) → review+
Attachment #8677679 - Flags: checked-in+
Comment on attachment 8677679 [details] [diff] [review]
bug1217534.patch

That didn't work. Trying something different
Attachment #8677679 - Flags: checked-in+ → checked-in-
Would backing out the gtests patch be an option?
(and try re-landing it after using list_builder_differences.sh)
I'm surprised your patch didn't work from looking at the builder.diff! Good luck!
The problem with the previous patch is that it added m-a and m-b talos builders to masters with only vms attached, not talos machines and thus it failed due to the lack of matching slaves.  Testing new patches to address this but no luck yet.
Attachment #8677679 - Attachment is obsolete: true
Attachment #8677680 - Attachment is obsolete: true
Attached file bug1217534builder.diff
builder diff for a master with ubuntu64_hw slaves

diff for a master w ubuntu64_vm slaves is a noop

./test-masters.sh runs green
Attachment #8678304 - Attachment is patch: false
Attachment #8678303 - Flags: review?(armenzg)
Attachment #8678303 - Flags: review?(armenzg) → review+
Comment on attachment 8678303 [details] [diff] [review]
bug1217534-2.patch

and tests pass
Attachment #8678303 - Flags: checked-in+
I seen builder jobs on the appropriate masters after the reconfig.
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Component: General Automation → General
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: