1140123 - enable graphene builds on try

Status: RESOLVED FIXED

(Release Engineering :: General, defect)

Description

[[e-mail] (:bhearsum)]

There's no reason we can't run them there. We should make them default off though.

Try chooser probably needs an update too.

Comment 1

[[e-mail] (:bhearsum)]

Attachment: enable graphene on try

I'm pretty sure try_by_default is false, so this should be the only thing we need in buildbot-configs....

Comment 2

[[e-mail] (:bhearsum)]

Attachment: add graphene options to trychooser

I'm not 100% sure if the platform values are correct. The platforms in b2g_config.py match these, but the builder names are different. Eg:  graphene_larch_linux64 build

I think you're the resident expert here, Steve :).

Comment 3

[Steve Fink [:sfink] [:s:]]

Comment on attachment 8573975 [details] [diff] [review]
add graphene options to trychooser

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

Looks right to me, assuming I'm reading b2g_config.py correctly. It seems to say -p linux64_graphene is the way to get that build.

Comment 4

[Jordan Lund (:jlund)]

Comment on attachment 8573931 [details] [diff] [review]
enable graphene on try

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

this should almost do it but, iiuc, we need a couple more things.

try_by_default is itself, defaulted to True if we don't define it: http://mxr.mozilla.org/build/search?string=try_by_default&find=misc&findi=&filter=%5E%5B%5E%5C0%5D*%24&hitlimit=&tree=build

so I think we need to add it and set it to false in the graphene pf config in a similar fashion to hazard builds: http://mxr.mozilla.org/build/source/buildbot-configs/mozilla/config.py#676

also, unfortunately, since try has its own slavepool, we need to explicitly tell try about this platform like we do for these: http://mxr.mozilla.org/build/source/buildbot-configs/mozilla/config.py#2741

r- for now till patch is updated or I'm proven to be inaccurate in my review :)

Comment 5

[[e-mail] (:bhearsum)]

(In reply to Jordan Lund (:jlund) from comment #4)
> Comment on attachment 8573931 [details] [diff] [review]
> enable graphene on try
> 
> Review of attachment 8573931 [details] [diff] [review]:
> -----------------------------------------------------------------
> 
> this should almost do it but, iiuc, we need a couple more things.
> 
> try_by_default is itself, defaulted to True if we don't define it:
> http://mxr.mozilla.org/build/
> search?string=try_by_default&find=misc&findi=&filter=%5E%5B%5E%5C0%5D*%24&hit
> limit=&tree=build

That link seems to prove what you say, but I'm confused. Doesn't that mean pushes will get a whole bunch of things run by default that we don't want?

> also, unfortunately, since try has its own slavepool, we need to explicitly
> tell try about this platform like we do for these:
> http://mxr.mozilla.org/build/source/buildbot-configs/mozilla/config.py#2741

Ah, good catch. Thanks!

Comment 6

[Jordan Lund (:jlund)]

(In reply to Ben Hearsum [:bhearsum] from comment #5)
> (In reply to Jordan Lund (:jlund) from comment #4)
> > Comment on attachment 8573931 [details] [diff] [review]
> > enable graphene on try
> > 
> > Review of attachment 8573931 [details] [diff] [review]:
> > -----------------------------------------------------------------
> > 
> > this should almost do it but, iiuc, we need a couple more things.
> > 
> > try_by_default is itself, defaulted to True if we don't define it:
> > http://mxr.mozilla.org/build/
> > search?string=try_by_default&find=misc&findi=&filter=%5E%5B%5E%5C0%5D*%24&hit
> > limit=&tree=build
> 
> That link seems to prove what you say, but I'm confused. Doesn't that mean
> pushes will get a whole bunch of things run by default that we don't want?

I'm pretty sure that if a dev uses 'try: -b do -p all' the only try build builders they won't get are the False guys from here(linux64-sh-haz, linux64-cc, and win64-debug): http://mxr.mozilla.org/build/search?string=try_by_default&find=mozilla%2F&findi=&filter=%5E%5B%5E%5C0%5D*%24&hitlimit=&tree=build

Comment 7

[[e-mail] (:bhearsum)]

Attachment: graphene try, #2

I think I fixed both of the issues you identified - thanks!

Comment 8

[Steve Fink [:sfink] [:s:]]

Doh! Sorry, I should have read the initial comments in this bug. I just assumed you wanted these to run on try by default.

> > try_by_default is itself, defaulted to True if we don't define it:
> > http://mxr.mozilla.org/build/
> > search?string=try_by_default&find=misc&findi=&filter=%5E%5B%5E%5C0%5D*%24&hit
> > limit=&tree=build
> 
> That link seems to prove what you say, but I'm confused. Doesn't that mean
> pushes will get a whole bunch of things run by default that we don't want?

There aren't very many things that are off by default on try. And it's weird how that works, too. eg the various spidermonkey builds are associated with regular platforms, but the hazard builds and graphene builds are their own platforms. So -p linux64 will get all of the nondefault spidermonkey builds, whether you wanted them or not. On the other hand, the weirdo platforms feel like a misuse of the data model and are hard to discover.

I'm still surprised trychooser hasn't collapsed under its own weight yet. :-)

Comment 9

[Jordan Lund (:jlund)]

Comment on attachment 8574814 [details] [diff] [review]
graphene try, #2

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

afaict, lgtm :)

Comment 10

[[e-mail] (:bhearsum)]

I had to land a silly typo fix for try chooser, but it's updated now: https://hg.mozilla.org/build/tools/rev/9633d1651564

Comment 11

[[e-mail] (:bhearsum)]

Needed a bustage fix for buildbot-configs too, because try has an explicit list of platforms: https://hg.mozilla.org/build/buildbot-configs/rev/a0eac491d154 and http://hg.mozilla.org/build/buildbot-configs/rev/87f23e9b7073

Comment 12

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

In production: https://hg.mozilla.org/build/buildbot-configs/rev/1c2da9cfccf8

Comment 13

[[e-mail] (:bhearsum)]

These builders are on try now, and can be run with try syntax like "try: -b o -p linux64_graphene,macosx64_graphene,win64_graphene -u none -t none", which is selectable with http://trychooser.pub.build.mozilla.org/