964589 - try -u all does not trigger Android x86 S4

Status: RESOLVED FIXED

(Infrastructure & Operations Graveyard :: CIDuty, task)

Description

[Geoff Brown [:gbrown]]

In https://tbpl.mozilla.org/?tree=Try&rev=1399e9c332d9, I specified:

try: -b o -p linux64,android,android-x86,android-armv6 -u all -t all

but no Android x86 tests were scheduled. I expected S4 to run.


In contrast, try pushes seem to work fine for Android x86 if a particular test job is specified. For example, https://tbpl.mozilla.org/?tree=Try&rev=0431c24939d4 specified:

try: -b o -p linux64,android,android-x86 -u mochitest-1,robocop-1,reftest,plain-reftest-1 -t none

and Android x86 S4 ran, as I would expect.

Comment 1

[Armen [:armenzg]]

I will look into this.

Comment 2

[Armen [:armenzg]]

Attachment: trychooser.diff

This is only to bring trychooser up-to-date.

It does not address the issue filed.

Comment 3

[Armen [:armenzg]]

I should try to read bug 915870 to see how I worked on this.

Comment 4

[Geoff Brown [:gbrown]]

Comment on attachment 8368672 [details] [diff] [review]
trychooser.diff

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

BTW, I like the "only x86" info but I do not see much value to the "not x86" info.

Comment 5

[Phil Ringnalda (:philor)]

Once we realize just how incredibly sloppy we've gotten about enforcing the rather minimal visibility requirements, this bug is a very likely place that we'll start.

Comment 6

[Armen [:armenzg]]

I'm working on the side something for bug 983802.

Comment 7

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

Pinged Armen in bug 983802.

Comment 9

[James Graham [:jgraham]]

So, as stated in the dupe, I got backed out because of this. Is there a reason this doesn't run by default?

Comment 10

[Chris Manchester (limited bugmail, email directly)]

Attachment: bb.patch

I've been backed out for this before, as well.

Comment 11

[Chris Manchester (limited bugmail, email directly)]

http://hg.mozilla.org/build/buildbotcustom/rev/b7f3c09d04d1

Comment 12

[Chris Manchester (limited bugmail, email directly)]

I'm seeing S4 on some recent -u all runs, this is fixed.