1191297 - Trychooser syntax builder doesn't provide a syntax for Android 4.0 API11+ opt R(1)

Status: RESOLVED FIXED

(Release Engineering :: General, defect)

Description

[Kartikaya Gupta (email:[e-mail])]

I want to do a try push that includes the Android 4.0 API11+ opt R(1) job. So I went to the syntax builder, checked the following:

Build type: opt only
Platforms: android api 11+
Restrict tests to platform(s): Android - 4.0 API11+ disabled when not explicity checked
Android-only Unittest Suites: plain-reftest-1

This gave me the syntax of:
try: -b o -p android-api-11 -u plain-reftest-1[Android -] -t none

Which didn't actually run the R(1) I was looking for. See https://treeherder.mozilla.org/#/jobs?repo=try&revision=bfe674f60284

Comment 1

[Kartikaya Gupta (email:[e-mail])]

I'm doing a new try push at https://treeherder.mozilla.org/#/jobs?repo=try&revision=715571158bf3 without the reftest restriction, hopefully that will work better.

Comment 2

[Kartikaya Gupta (email:[e-mail])]

Attachment: Patch

I think this should fix it. I have another try syntax from before which doesn't have the "-" following "Android" and that *does* run the 4.0 tests. I think the "-" is just a typo (as is the bug number linked there).

Comment 3

[Kartikaya Gupta (email:[e-mail])]

Attachment: Patch v2

Also remove the spurious trailing </label>. Three typos in a single line!

Comment 4

[Kartikaya Gupta (email:[e-mail])]

And indeed removing that spurious dash fixes the problem: https://treeherder.mozilla.org/#/jobs?repo=try&revision=001996148a06

Comment 5

[Kartikaya Gupta (email:[e-mail])]

https://hg.mozilla.org/build/tools/rev/32aeeb3ef438

Comment 6

[Kartikaya Gupta (email:[e-mail])]

Do you know how frequently changes get pushed out to trychooser.pub.build.m.o?

Comment 7

[Justin Wood (:Callek)]

When I reviewed, I thought this was auto-deploying, turns out my memory then was wrong. I just deployed this though.

(Its usually an "on demand" deploy)

Comment 8

[Kartikaya Gupta (email:[e-mail])]

Cool, thanks!