1417676 - Make browser-instrumentation run only on Nightlies

Status: RESOLVED FIXED

(Testing :: Mochitest, enhancement)

Description

[Brian Grinstead [:bgrins]]

See https://bugzilla.mozilla.org/show_bug.cgi?id=1376546#c44. We don't need this job to run on every push, once a day should be fine.

Comment 1

[Brian Grinstead [:bgrins]]

Attachment: Bug 1417676 - Make browser-instrumentation run only on Nightlies;

Review commit: https://reviewboard.mozilla.org/r/199978/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/199978/

Comment 2

[Brian Grinstead [:bgrins]]

One weird thing is that I don't see it show up as an option for ./mach try fuzzy anymore even when I added "try" to the default run-on-projects set, but that's probably OK.  Ideally we could find a way to always run on nightlies and still be able to run on try but only when it's explicitly requested.

Here's a try push with `./mach try -b o -p all -u all -t none` which should not include it once it finishes: https://treeherder.mozilla.org/#/jobs?repo=try&revision=67b5de7708ced3c1ff38212ffe8701fa271ea2d4

Comment 3

[Joel Maher ( :jmaher ) (UTC -8)]

Comment on attachment 8928715 [details]
Bug 1417676 - Make browser-instrumentation run only on Nightlies;

https://reviewboard.mozilla.org/r/199978/#review205104

this looks right, if we land it now we can revisit it tomorrow easily

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/3b17f971782a
Make browser-instrumentation run only on Nightlies;r=jmaher

Comment 5

[Raul Gurzau (:RaulG)]

https://hg.mozilla.org/mozilla-central/rev/3b17f971782a

Comment 6

[Joel Maher ( :jmaher ) (UTC -8)]

this seems to be working as planned:
https://treeherder.mozilla.org/#/jobs?repo=mozilla-central&filter-searchStr=inst&filter-tier=1&filter-tier=2&filter-tier=3

Comment 7

[Brian Grinstead [:bgrins]]

(In reply to Joel Maher ( :jmaher) (UTC-5) from comment #6)
> this seems to be working as planned:
> https://treeherder.mozilla.org/#/jobs?repo=mozilla-central&filter-
> searchStr=inst&filter-tier=1&filter-tier=2&filter-tier=3

Yep, and the artifacts look right - thanks!