Phil Ringnalda (:philor) Assignee
	Description • 11 years ago

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

	Joshua Cranmer [:jcranmer]
	Comment 1 • 11 years ago

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.

	Phil Ringnalda (:philor) Assignee
	Comment 2 • 11 years ago

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.

	Phil Ringnalda (:philor) Assignee
	Comment 3 • 11 years ago

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.

	Chris AtLee [:catlee]
	Comment 4 • 11 years ago

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

	Joshua Cranmer [:jcranmer]
	Comment 5 • 11 years ago

(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.

	Phil Ringnalda (:philor) Assignee
	Comment 6 • 11 years ago

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

	Phil Ringnalda (:philor) Assignee
	Comment 7 • 11 years ago

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.

	Phil Ringnalda (:philor) Assignee
	Comment 8 • 11 years ago

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

	Phil Ringnalda (:philor) Assignee
	Comment 9 • 11 years ago

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.

	Phil Ringnalda (:philor) Assignee
	Comment 10 • 11 years ago

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

	John Hopkins (:jhopkins)
	Comment 11 • 11 years ago

Backout is in production

	Phil Ringnalda (:philor) Assignee
	Comment 12 • 11 years ago

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.

	John Hopkins (:jhopkins)
	Comment 13 • 11 years ago

In production.

	Phil Ringnalda (:philor) Assignee
	Comment 14 • 11 years ago

On top of bug 912345 and bug 847973.

	Ed Morley [:emorley]
	Comment 15 • 11 years ago

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 :-)

	Phil Ringnalda (:philor) Assignee
	Comment 16 • 11 years ago

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