1018979 - Try server: builds triggered for platforms that weren't specified in the commit message

Status: RESOLVED DUPLICATE of bug 942167

(Release Engineering :: General, defect)

Description

[Benjamin Bouvier [:bbouvier] (inactive)]

In the try commit message, I only put MacOS 10.6/10.8, and builds were also triggered on B2G emulators.

See for instance https://tbpl.mozilla.org/?tree=Try&rev=4f702beeb85f

try: -b do -e -p macosx64 -u jittests[10.6,10.8] -t none

Comment 1

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

Note to whoever picks this up: this is where the code is:

bhearsum
15:55:35 https://github.com/mozilla/build-buildbotcustom/blob/master/try_parser.py
pmoore|buildduty
15:55:44 ah, perfect!
15:55:45 thanks ben
bhearsum
15:55:50 it's got tests, too: https://github.com/mozilla/build-buildbotcustom/blob/master/test/test_try_parser.py

Comment 2

[Phil Ringnalda (:philor)]

I don't see anything other than Mac builds and Mac jittests on that push. Did you by any chance, in whatever push you actually did see it, catch the mistaken periodic nonunified emulator builds that bug 942167 comment 76 says we need to get rid of, because come on, periodic builds on try?

Comment 3

[Benjamin Bouvier [:bbouvier] (inactive)]

(In reply to Phil Ringnalda (:philor) from comment #2)
> I don't see anything other than Mac builds and Mac jittests on that push.
> Did you by any chance, in whatever push you actually did see it, catch the
> mistaken periodic nonunified emulator builds that bug 942167 comment 76 says
> we need to get rid of, because come on, periodic builds on try?

Ha, sorry, I may have cancelled the builds as they were unexpected, in order to spare some CPU energy. I remember seeing that the emulator builds were nonunified, so i might have triggered the periodic builds, indeed.

Comment 4

[Phil Ringnalda (:philor)]

Indeed, https://secure.pub.build.mozilla.org/buildapi/self-serve/try/rev/4f702beeb85f.