Ryan VanderMeulen [:RyanVM] Assignee
	Description • 11 years ago

The test ordering is a bit random at the moment. We have Cpp/Jit tests in the middle of a bunch of others, Jetpack tacked on to the end, etc. I'm going to reorder the test suites so that we at least have some sort of alphabetical order. On a high level, I'll enforce a Builds->Unit Tests->Perf Tests ordering.

	Ryan VanderMeulen [:RyanVM] Assignee
	Comment 1 • 11 years ago

This is a straight-up alphabetization of the unit test jobs and making sure they're all ahead of talos. The one break from this patter is that the Mozbase tests come before other unit tests as they're kind of a "bridge" test suite since many others derive from it and bustage to that is likely to hit others as well.

This leaves the unknown jobs at the end as that makes the most sense from a visibility standpoint IMO.

	Ryan VanderMeulen [:RyanVM] Assignee
	Comment 2 • 11 years ago

Scratching one more itch, this fixes the SM ordering.

	Ryan VanderMeulen [:RyanVM] Assignee
	Comment 3 • 11 years ago

Now with the right patch attached...

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

Comment on attachment 8379137 [details] [diff] [review]
alphabetize unit tests

\o/ :-)

	Ryan VanderMeulen [:RyanVM] Assignee
	Comment 5 • 11 years ago

https://hg.mozilla.org/webtools/tbpl/rev/b0d6324ccb98

	Ryan VanderMeulen [:RyanVM] Assignee
	Comment 6 • 11 years ago

In production :)

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

I'm wondering if we could tweak the tbpl job ordering back slightly so that the mochitest jobs are lined up properly again - since it's visually quite distracting now with some (but crucially not all) rows having cpp etc bump the mochitest jobs along.

Perhaps:
B  M ( ...) R (...) anything-loose   T (...)

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

Comment on attachment 8381597 [details] [diff] [review]
put mochitests and reftests first

Maybe also add a comment/heading, similar to the generic ones already in Data.js? (Just thinking of when other people add rows).

	Ryan VanderMeulen [:RyanVM] Assignee
	Comment 10 • 11 years ago

https://hg.mozilla.org/webtools/tbpl/rev/e32cb5042e86

	Ryan VanderMeulen [:RyanVM] Assignee
	Comment 11 • 11 years ago

In production.