Closed Bug 1033507 Opened 6 years ago Closed 6 years ago

disable tegras on 32 to ride the trains


(Infrastructure & Operations Graveyard :: CIDuty, task)

Not set


(Not tracked)



(Reporter: kmoir, Assigned: kmoir)




(6 files, 3 obsolete files)

bye bye tegras
Assignee: nobody → kmoir
Blocks: 1016453
Christmas has come early! :-D
Depends on: 1034221
Attached patch bug1033507.patch (obsolete) — Splinter Review
Attached patch bug1033507builder.diff (obsolete) — Splinter Review
builder diff
Attached file bug1033507builder.diff (obsolete) —
Attachment #8451129 - Attachment is obsolete: true
As I mentioned before in bug

I don't know how to disable talos on a per branch basis until we turn off all Tegra tests.  

Actually now that we have all tests off on certain bramches if I turn off builds on those branches, no talos tests will be invoked.  Let me write a patch for that.
Attachment #8451117 - Attachment is obsolete: true
Attachment #8451689 - Flags: review?(bugspam.Callek)
builder diff
Attachment #8451130 - Attachment is obsolete: true
Callek: I looked at disabling the builds but of course the armv6 apks are still used for 2.3 tests and the android apks are used for 4.0.  So I don't know a good way to disable the associated talos tests for tegras.

If you have suggestions, I'd appreciate them.

See comment #5.
Attachment #8451692 - Attachment is patch: true
Attachment #8451692 - Attachment mime type: text/x-patch → text/plain
Comment on attachment 8451689 [details] [diff] [review]

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

r- purely because its removing them from aurora as well per the builder diff.

I'd also love a grep of "Tegra" in the full (after) builder list, to verify what branches are left, while we're at it.

To be clear, we *do* want to remove on try, effectively defeating the ability to mobile test beta/etc patches that land?
Attachment #8451689 - Flags: review?(bugspam.Callek) → review-
My understanding is that it's supposed to be removed from the aurora because it will be removed from 32. In the mobile meeting it was mentioned that there would be a lack of test coverage during this change.  The same with try. I have already disabled selected tests on 32 in bug 1017599 which is reflected on Try and m-a.

I'll attach a grep of Tegra in the after builder list.
Attached file Tegra.jul7
List of builders with Tegra after this change
Comment on attachment 8451689 [details] [diff] [review]

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

Ok then. If be interested in knowing if the b2g* branches still want or need tegra as well
Attachment #8451689 - Flags: review- → review+
Attachment #8451689 - Flags: checked-in+
Callek: who should I ask re the b2g branches, not sure if the mobile test meeting is the best place.
In production
From #ateam
blassey	go ahead and turn off talos tegras
kmoir	gbrown: I can needinfo blassey on that bug and see if we can disable all talos on tegras
jmaher	blassey++
kmoir	blassey++

So I'll attach a patch to disable all talos testing on tegras
(In reply to Kim Moir [:kmoir] from comment #13)
> Callek: who should I ask re the b2g branches, not sure if the mobile test
> meeting is the best place.

I'm not sure, n-i to :aki to redirect.

For enumeration, kim, which branches are running which tests, with which builds against tegras?
Flags: needinfo?(aki)
b2g30 has none:
I'm going to follow that pattern for b2g32.
Flags: needinfo?(aki)
patch to disable Talos tests on Tegra across all branches
Attachment #8454595 - Flags: review?(bugspam.Callek)
Attached file byebyeTegrasTalos
builder diff
Attached file remainingTegrajobs
List of remaining Tegra jobs after disabling Talos
Comment on attachment 8454595 [details] [diff] [review]

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

::: mozilla-tests/
@@ +119,1 @@
>  ANDROID_NOT_PANDA = [slave_plat for slave_plat in ANDROID if 'panda' not in slave_plat]

ANDROID_NOT_PANDA will now be an empty list, followup fodder.

@@ +1658,5 @@
>              if not branch['platforms'][platform][slave_plat]['debug_unittest_suites']:
>                  continue
>              else:
>                  branch['platforms'][platform]['enable_debug_unittests'] = False
> +A

accidental insertion
Attachment #8454595 - Flags: review?(bugspam.Callek) → review+
verified on tbpl
Closed: 6 years ago
Resolution: --- → FIXED
No longer depends on: 1034221
Duplicate of this bug: 1034221
Blocks: 1033126
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.