887641 - Make it possible for mortals to get static analysis builds on Try

Status: RESOLVED FIXED

(Release Engineering :: General, defect)

Description

[Phil Ringnalda (:philor)]

Three problems:

* it's 'try_by_default': False, which was right when it was only running hidden on m-c, but isn't right now that it's on all trees and wants to be unhidden

* it took me most of an hour to figure out how to get it to run on Try, that being how long it took for the realization that linux64-dbg-st-an is an opt build, and will only run from -b o -p linux64-dbg-st-an, because debug builds are builds whose name ends in -debug, not builds which have the letters d, b, and g somewhere within their names

* it needs to be added to trychooser

Comment 1

[Joshua Cranmer [:jcranmer]]

What would need to be done to fix these things? The first thing looks like deleting a line in buildbot-configs, the second I have no idea how to fix, and the third may require fixing the second.

Comment 2

[Phil Ringnalda (:philor)]

Aye, the name's the rub. If there turns out not to be a character limit you'll run over, then it's just a matter of changing all of http://mxr.mozilla.org/build/search?string=linux64-dbg-st-an to linux64-st-an-debug; if there is, then it's either linux64-st-debug (and the horrible linux64-as-debug for its next-door-neighbor, linux64-dbg-asan, which also needs to change), or the much less fun job of fixing every instance of "-debug" in http://mxr.mozilla.org/build/source/buildbotcustom/try_parser.py.

Personally, I tend to make that my reviewer's problem, and just write a patch changing it to what seems like it ought to work, linux64-st-an-debug, and make sure to use the word "untested" while attaching the patch.

Comment 3

[Phil Ringnalda (:philor)]

Attachment: [configs] rename st-an and asan

Oh, yeah, character limit comes after the name gets shortened, among other things by replacing "debug" with "dbg", so these are the same names by the time that comes up.

Who knows, might even work.

Comment 4

[Chris AtLee [:catlee]]

Comment on attachment 791758 [details] [diff] [review]
[configs] rename st-an and asan

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

Naming things with "-debug" has the fun side effect of disabling nightly builds unless you also specify "enable_nightly" in the platforms you care about. I think this means that linux64-asan-debug and linux64-st-an-debug need enable_nightly to be True if you want to continue having nightly builds of those types.
http://hg.mozilla.org/build/buildbotcustom/file/default/misc.py#l1258

Comment 5

[Joshua Cranmer [:jcranmer]]

(In reply to Chris AtLee [:catlee] from comment #4)
> Naming things with "-debug" has the fun side effect of disabling nightly
> builds unless you also specify "enable_nightly" in the platforms you care
> about. I think this means that linux64-asan-debug and linux64-st-an-debug
> need enable_nightly to be True if you want to continue having nightly builds
> of those types.
> http://hg.mozilla.org/build/buildbotcustom/file/default/misc.py#l1258

For static analysis at least, nightly builds are likely to be a waste of resources.

Comment 6

[Phil Ringnalda (:philor)]

Yeah, it's only a problem because I scope-creeped, who knew that would be a bad idea?

Comment 7

[Phil Ringnalda (:philor)]

Attachment: [configs] rename st-an and asan with explicit enable_nightly

Well, I know less than I did before (I've got no idea whether or not http://hg.mozilla.org/build/buildbot-configs/file/88a3f11a4a02/mozilla/config.py#l1560 is true, though I suspect it is not, and I have no idea why, in terms of policy, we choose to do debug asan nightlies when we don't do debug anything else nightlies, and I have absolutely no explanation for why, as a matter of code, we are not currently unintentionally creating static-analysis nightlies, since they don't end in -debug and don't say at the platform level to not do nightlies and do say at the branch level to do nightlies), but, perhaps this will do the job.

Comment 8

[Phil Ringnalda (:philor)]

Comment on attachment 792421 [details] [diff] [review]
[configs] rename st-an and asan with explicit enable_nightly

https://hg.mozilla.org/build/buildbot-configs/rev/8f2865b76d55

Comment 9

[Phil Ringnalda (:philor)]

Comment on attachment 792421 [details] [diff] [review]
[configs] rename st-an and asan with explicit enable_nightly

And backed out in https://hg.mozilla.org/build/buildbot-configs/rev/8bdc04480977 for utterly incomprehensible mute test failure.

Comment 10

[Phil Ringnalda (:philor)]

I got nothin' here, empty failure logs locally with and without my patch, and empty failures logs on Jenkins with it.

Comment 11

[John Hopkins (:jhopkins)]

Backout is in production

Comment 12

[Phil Ringnalda (:philor)]

And now that Jenkins is a tiny bit more verbose, https://hg.mozilla.org/build/buildbot-configs/rev/e59e0fa1faf8 and then https://hg.mozilla.org/build/buildbot-configs/rev/1bcd9ace6472 because the problem was that I screwed up rebasing.

Comment 13

[John Hopkins (:jhopkins)]

In production.

Comment 14

[Phil Ringnalda (:philor)]

Attachment: [tools] Add linux64-st-an to trychooser

On top of bug 912345 and bug 847973.

Comment 15

[Ed Morley [:emorley]]

Comment on attachment 799950 [details] [diff] [review]
[tools] Add linux64-st-an to trychooser

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

Lgtm & works well locally :-)

Comment 16

[Phil Ringnalda (:philor)]

http://hg.mozilla.org/build/tools/rev/835ed7190428 with a typoed bug number, and deployed.