1036014 - Try alias for Android 2.3 xpcshell tests

Status: RESOLVED FIXED

(Release Engineering :: General, defect)

Description

[Geoff Brown [:gbrown]]

+++ This bug was initially created as a clone of Bug #969991 +++

Try pushes with -u xpcshell do not trigger Android 2.3's xpcshell-1, xpcshell-2, and xpcshell-3, sometimes resulting in confusion and disappointment. RyanVM suggested an alias and pointed me at https://hg.mozilla.org/build/buildbotcustom/rev/e9a52c460141.

See https://tbpl.mozilla.org/?tree=Try&rev=88d5ab0797e3&showall=1 for an example.

I wonder if there are other Android cases that could be improved? 

-u robocop?

Comment 1

[Justin Wood (:Callek)]

Attachment: [buildbotcustom] v1

this was a quick patch while I saw the IRC discussion. Helps make devs lives easier.

Comment 2

[Ryan VanderMeulen [:RyanVM]]

Comment on attachment 8452575 [details] [diff] [review]
[buildbotcustom] v1

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

::: try_parser.py
@@ +19,4 @@
>          return v.startswith('jittest')
>      elif u in ('mochitest-o', 'mochitest-chrome', 'mochitest-a11y'):
>          return re.search('mochitest-o', v)
> +    elif u in ('xpcshells', 'xpcshell'):

Trychooser has an explicit "jittests" option on it. It doesn't have an "xpcshells" option that I'm aware of, so I don't think we need to support this.

Comment 3

[Justin Wood (:Callek)]

Pushed with nit addressed: https://hg.mozilla.org/build/buildbotcustom/rev/c695faf1f43d

Comment 4

[Pete Moore [:pmoore][:pete]]

In production.

Comment 5

[Geoff Brown [:gbrown]]

Works great: https://tbpl.mozilla.org/?tree=Try&rev=e52bcf945dcd