Add list of Skia trybots to run CLs against, along with existing lists of Chrome trybots

Add list of Skia trybots to run CLs against, along with existing lists of Chrome trybots
(SkipBuildbotRuns)
Committed: https://code.google.com/p/skia/source/detail?r=8522

[epoger, 2013-04-03 15:21:36]

Patchset 1 just moves the existing Chrome trybot lists into a "chrome" subdir,
to make room for the new Skia trybot lists.

NOTE TO SELF: At the same time that you commit this CL, update the documentation
pointing at these files to include the new subdirectory path!

[epoger, 2013-04-03 16:46:25]

Patchset 2 adds a complete list of *all* compile-only Skia trybots.  I will pare
that down in patchset 3, to a small number that we think are representative.

[epoger, 2013-04-03 16:59:37]

Ready to review at patchset 3.

https://codereview.chromium.org/13545002/diff/6002/trybots_to_run/skia/compile
File trybots_to_run/skia/compile (right):

https://codereview.chromium.org/13545002/diff/6002/trybots_to_run/skia/compil...
trybots_to_run/skia/compile:1: Skia_ChromeOS_Alex_Compile_Debug_32
Skia_Linux_NoGPU_Compile_Debug_32 Skia_MacMiniLion_Float_Compile_Debug_32
Skia_MacMiniLion_Float_Compile_Release_64
Skia_MacMini_10_8_Float_Compile_Debug_32
Skia_MacMini_10_8_Float_Compile_Release_64 Skia_Mac_Float_Compile_Debug_32
Skia_Mac_Float_Compile_Release_64 Skia_Nexus10_4-1_Float_Compile_Debug_32
Skia_Nexus4_4-1_Float_Compile_Release_32 Skia_RazrI_4-1_Float_Compile_Debug_32
Skia_Shuttle_Ubuntu12_ATI5770_Float_Compile_Debug_32
Skia_Shuttle_Ubuntu12_ATI5770_Float_Compile_Release_64
Skia_Shuttle_Ubuntu12_NaCl_Compile_Debug
Skia_Shuttle_Win7_Intel_Float_ANGLE_Compile_Debug_32
Skia_Shuttle_Win7_Intel_Float_Compile_Debug_32
Skia_Shuttle_Win7_Intel_Float_DirectWrite_Compile_Debug_32
Skia_iOS_Compile_Debug_32
patchset 3 reduces the list of compile-only bots... a smaller set, that we
believe will cover 99% of compile failures.

[borenet, 2013-04-03 17:06:59]

On 2013/04/03 16:59:37, epoger wrote:
> Ready to review at patchset 3.
> 
> https://codereview.chromium.org/13545002/diff/6002/trybots_to_run/skia/compile
> File trybots_to_run/skia/compile (right):
> 
>
https://codereview.chromium.org/13545002/diff/6002/trybots_to_run/skia/compil...
> trybots_to_run/skia/compile:1: Skia_ChromeOS_Alex_Compile_Debug_32
> Skia_Linux_NoGPU_Compile_Debug_32 Skia_MacMiniLion_Float_Compile_Debug_32
> Skia_MacMiniLion_Float_Compile_Release_64
> Skia_MacMini_10_8_Float_Compile_Debug_32
> Skia_MacMini_10_8_Float_Compile_Release_64 Skia_Mac_Float_Compile_Debug_32
> Skia_Mac_Float_Compile_Release_64 Skia_Nexus10_4-1_Float_Compile_Debug_32
> Skia_Nexus4_4-1_Float_Compile_Release_32 Skia_RazrI_4-1_Float_Compile_Debug_32
> Skia_Shuttle_Ubuntu12_ATI5770_Float_Compile_Debug_32
> Skia_Shuttle_Ubuntu12_ATI5770_Float_Compile_Release_64
> Skia_Shuttle_Ubuntu12_NaCl_Compile_Debug
> Skia_Shuttle_Win7_Intel_Float_ANGLE_Compile_Debug_32
> Skia_Shuttle_Win7_Intel_Float_Compile_Debug_32
> Skia_Shuttle_Win7_Intel_Float_DirectWrite_Compile_Debug_32
> Skia_iOS_Compile_Debug_32
> patchset 3 reduces the list of compile-only bots... a smaller set, that we
> believe will cover 99% of compile failures.

LGTM

[rmistry, 2013-04-03 17:09:38]

https://codereview.chromium.org/13545002/diff/6002/trybots_to_run/skia/compile
File trybots_to_run/skia/compile (right):

https://codereview.chromium.org/13545002/diff/6002/trybots_to_run/skia/compil...
trybots_to_run/skia/compile:1: Skia_ChromeOS_Alex_Compile_Debug_32
Skia_Linux_NoGPU_Compile_Debug_32 Skia_MacMiniLion_Float_Compile_Debug_32
Skia_MacMiniLion_Float_Compile_Release_64
Skia_MacMini_10_8_Float_Compile_Debug_32
Skia_MacMini_10_8_Float_Compile_Release_64 Skia_Mac_Float_Compile_Debug_32
Skia_Mac_Float_Compile_Release_64 Skia_Nexus10_4-1_Float_Compile_Debug_32
Skia_Nexus4_4-1_Float_Compile_Release_32 Skia_RazrI_4-1_Float_Compile_Debug_32
Skia_Shuttle_Ubuntu12_ATI5770_Float_Compile_Debug_32
Skia_Shuttle_Ubuntu12_ATI5770_Float_Compile_Release_64
Skia_Shuttle_Ubuntu12_NaCl_Compile_Debug
Skia_Shuttle_Win7_Intel_Float_ANGLE_Compile_Debug_32
Skia_Shuttle_Win7_Intel_Float_Compile_Debug_32
Skia_Shuttle_Win7_Intel_Float_DirectWrite_Compile_Debug_32
Skia_iOS_Compile_Debug_32
Dont they have to be comma separated?

[borenet, 2013-04-03 17:10:45]

https://codereview.chromium.org/13545002/diff/6002/trybots_to_run/skia/compile
File trybots_to_run/skia/compile (right):

https://codereview.chromium.org/13545002/diff/6002/trybots_to_run/skia/compil...
trybots_to_run/skia/compile:1: Skia_ChromeOS_Alex_Compile_Debug_32
Skia_Linux_NoGPU_Compile_Debug_32 Skia_MacMiniLion_Float_Compile_Debug_32
Skia_MacMiniLion_Float_Compile_Release_64
Skia_MacMini_10_8_Float_Compile_Debug_32
Skia_MacMini_10_8_Float_Compile_Release_64 Skia_Mac_Float_Compile_Debug_32
Skia_Mac_Float_Compile_Release_64 Skia_Nexus10_4-1_Float_Compile_Debug_32
Skia_Nexus4_4-1_Float_Compile_Release_32 Skia_RazrI_4-1_Float_Compile_Debug_32
Skia_Shuttle_Ubuntu12_ATI5770_Float_Compile_Debug_32
Skia_Shuttle_Ubuntu12_ATI5770_Float_Compile_Release_64
Skia_Shuttle_Ubuntu12_NaCl_Compile_Debug
Skia_Shuttle_Win7_Intel_Float_ANGLE_Compile_Debug_32
Skia_Shuttle_Win7_Intel_Float_Compile_Debug_32
Skia_Shuttle_Win7_Intel_Float_DirectWrite_Compile_Debug_32
Skia_iOS_Compile_Debug_32
On 2013/04/03 17:09:38, rmistry wrote:
> Dont they have to be comma separated?

No: https://sites.google.com/site/skiadocs/developer-documentation/skia-trybots

[rmistry, 2013-04-03 17:14:45]

On 2013/04/03 17:10:45, borenet wrote:
> https://codereview.chromium.org/13545002/diff/6002/trybots_to_run/skia/compile
> File trybots_to_run/skia/compile (right):
> 
>
https://codereview.chromium.org/13545002/diff/6002/trybots_to_run/skia/compil...
> trybots_to_run/skia/compile:1: Skia_ChromeOS_Alex_Compile_Debug_32
> Skia_Linux_NoGPU_Compile_Debug_32 Skia_MacMiniLion_Float_Compile_Debug_32
> Skia_MacMiniLion_Float_Compile_Release_64
> Skia_MacMini_10_8_Float_Compile_Debug_32
> Skia_MacMini_10_8_Float_Compile_Release_64 Skia_Mac_Float_Compile_Debug_32
> Skia_Mac_Float_Compile_Release_64 Skia_Nexus10_4-1_Float_Compile_Debug_32
> Skia_Nexus4_4-1_Float_Compile_Release_32 Skia_RazrI_4-1_Float_Compile_Debug_32
> Skia_Shuttle_Ubuntu12_ATI5770_Float_Compile_Debug_32
> Skia_Shuttle_Ubuntu12_ATI5770_Float_Compile_Release_64
> Skia_Shuttle_Ubuntu12_NaCl_Compile_Debug
> Skia_Shuttle_Win7_Intel_Float_ANGLE_Compile_Debug_32
> Skia_Shuttle_Win7_Intel_Float_Compile_Debug_32
> Skia_Shuttle_Win7_Intel_Float_DirectWrite_Compile_Debug_32
> Skia_iOS_Compile_Debug_32
> On 2013/04/03 17:09:38, rmistry wrote:
> > Dont they have to be comma separated?
> 
> No:
https://sites.google.com/site/skiadocs/developer-documentation/skia-trybots

This is interesting because the documentation for gcl try says:
Where <builders> is a comma-separated list of builders with no spaces

So submit_try requires spaces but gcl try requires commas?

[rmistry, 2013-04-03 17:15:53]

On 2013/04/03 17:14:45, rmistry wrote:
> On 2013/04/03 17:10:45, borenet wrote:
> >
https://codereview.chromium.org/13545002/diff/6002/trybots_to_run/skia/compile
> > File trybots_to_run/skia/compile (right):
> > 
> >
>
https://codereview.chromium.org/13545002/diff/6002/trybots_to_run/skia/compil...
> > trybots_to_run/skia/compile:1: Skia_ChromeOS_Alex_Compile_Debug_32
> > Skia_Linux_NoGPU_Compile_Debug_32 Skia_MacMiniLion_Float_Compile_Debug_32
> > Skia_MacMiniLion_Float_Compile_Release_64
> > Skia_MacMini_10_8_Float_Compile_Debug_32
> > Skia_MacMini_10_8_Float_Compile_Release_64 Skia_Mac_Float_Compile_Debug_32
> > Skia_Mac_Float_Compile_Release_64 Skia_Nexus10_4-1_Float_Compile_Debug_32
> > Skia_Nexus4_4-1_Float_Compile_Release_32
Skia_RazrI_4-1_Float_Compile_Debug_32
> > Skia_Shuttle_Ubuntu12_ATI5770_Float_Compile_Debug_32
> > Skia_Shuttle_Ubuntu12_ATI5770_Float_Compile_Release_64
> > Skia_Shuttle_Ubuntu12_NaCl_Compile_Debug
> > Skia_Shuttle_Win7_Intel_Float_ANGLE_Compile_Debug_32
> > Skia_Shuttle_Win7_Intel_Float_Compile_Debug_32
> > Skia_Shuttle_Win7_Intel_Float_DirectWrite_Compile_Debug_32
> > Skia_iOS_Compile_Debug_32
> > On 2013/04/03 17:09:38, rmistry wrote:
> > > Dont they have to be comma separated?
> > 
> > No:
> https://sites.google.com/site/skiadocs/developer-documentation/skia-trybots
> 
> This is interesting because the documentation for gcl try says:
> Where <builders> is a comma-separated list of builders with no spaces
> 
> So submit_try requires spaces but gcl try requires commas?

If this is the case then all the files in trybots_to_run should be space
separated and README.txt should say only use submit_try.

[epoger, 2013-04-03 17:16:13]

On 2013/04/03 17:14:45, rmistry wrote:
> On 2013/04/03 17:10:45, borenet wrote:
> >
https://codereview.chromium.org/13545002/diff/6002/trybots_to_run/skia/compile
> > File trybots_to_run/skia/compile (right):
> > 
> >
>
https://codereview.chromium.org/13545002/diff/6002/trybots_to_run/skia/compil...
> > trybots_to_run/skia/compile:1: Skia_ChromeOS_Alex_Compile_Debug_32
> > Skia_Linux_NoGPU_Compile_Debug_32 Skia_MacMiniLion_Float_Compile_Debug_32
> > Skia_MacMiniLion_Float_Compile_Release_64
> > Skia_MacMini_10_8_Float_Compile_Debug_32
> > Skia_MacMini_10_8_Float_Compile_Release_64 Skia_Mac_Float_Compile_Debug_32
> > Skia_Mac_Float_Compile_Release_64 Skia_Nexus10_4-1_Float_Compile_Debug_32
> > Skia_Nexus4_4-1_Float_Compile_Release_32
Skia_RazrI_4-1_Float_Compile_Debug_32
> > Skia_Shuttle_Ubuntu12_ATI5770_Float_Compile_Debug_32
> > Skia_Shuttle_Ubuntu12_ATI5770_Float_Compile_Release_64
> > Skia_Shuttle_Ubuntu12_NaCl_Compile_Debug
> > Skia_Shuttle_Win7_Intel_Float_ANGLE_Compile_Debug_32
> > Skia_Shuttle_Win7_Intel_Float_Compile_Debug_32
> > Skia_Shuttle_Win7_Intel_Float_DirectWrite_Compile_Debug_32
> > Skia_iOS_Compile_Debug_32
> > On 2013/04/03 17:09:38, rmistry wrote:
> > > Dont they have to be comma separated?
> > 
> > No:
> https://sites.google.com/site/skiadocs/developer-documentation/skia-trybots
> 
> This is interesting because the documentation for gcl try says:
> Where <builders> is a comma-separated list of builders with no spaces
> 
> So submit_try requires spaces but gcl try requires commas?

All I know is, I followed the directions at
https://sites.google.com/site/skiadocs/developer-documentation/skia-trybots and
tested this with submit_try.  I think probably either spaces or commas work...
we can discuss live...

[epoger, 2013-04-03 18:39:55]

https://codereview.chromium.org/13545002/diff/6002/trybots_to_run/skia/compile
File trybots_to_run/skia/compile (right):

https://codereview.chromium.org/13545002/diff/6002/trybots_to_run/skia/compil...
trybots_to_run/skia/compile:1: Skia_ChromeOS_Alex_Compile_Debug_32
Skia_Linux_NoGPU_Compile_Debug_32 Skia_MacMiniLion_Float_Compile_Debug_32
Skia_MacMiniLion_Float_Compile_Release_64
Skia_MacMini_10_8_Float_Compile_Debug_32
Skia_MacMini_10_8_Float_Compile_Release_64 Skia_Mac_Float_Compile_Debug_32
Skia_Mac_Float_Compile_Release_64 Skia_Nexus10_4-1_Float_Compile_Debug_32
Skia_Nexus4_4-1_Float_Compile_Release_32 Skia_RazrI_4-1_Float_Compile_Debug_32
Skia_Shuttle_Ubuntu12_ATI5770_Float_Compile_Debug_32
Skia_Shuttle_Ubuntu12_ATI5770_Float_Compile_Release_64
Skia_Shuttle_Ubuntu12_NaCl_Compile_Debug
Skia_Shuttle_Win7_Intel_Float_ANGLE_Compile_Debug_32
Skia_Shuttle_Win7_Intel_Float_Compile_Debug_32
Skia_Shuttle_Win7_Intel_Float_DirectWrite_Compile_Debug_32
Skia_iOS_Compile_Debug_32
On 2013/04/03 17:10:45, borenet wrote:
> On 2013/04/03 17:09:38, rmistry wrote:
> > Dont they have to be comma separated?
> 
> No:
https://sites.google.com/site/skiadocs/developer-documentation/skia-trybots

Patchset 4 uses commas instead of spaces as delimiters, because submit_try
supports either one as of https://code.google.com/p/skia/source/detail?r=8521 !

Unless I hear complaints, I will commit at patchset 4, and update documentation
accordingly.

[borenet, 2013-04-03 18:43:23]

On 2013/04/03 18:39:55, epoger wrote:
> https://codereview.chromium.org/13545002/diff/6002/trybots_to_run/skia/compile
> File trybots_to_run/skia/compile (right):
> 
>
https://codereview.chromium.org/13545002/diff/6002/trybots_to_run/skia/compil...
> trybots_to_run/skia/compile:1: Skia_ChromeOS_Alex_Compile_Debug_32
> Skia_Linux_NoGPU_Compile_Debug_32 Skia_MacMiniLion_Float_Compile_Debug_32
> Skia_MacMiniLion_Float_Compile_Release_64
> Skia_MacMini_10_8_Float_Compile_Debug_32
> Skia_MacMini_10_8_Float_Compile_Release_64 Skia_Mac_Float_Compile_Debug_32
> Skia_Mac_Float_Compile_Release_64 Skia_Nexus10_4-1_Float_Compile_Debug_32
> Skia_Nexus4_4-1_Float_Compile_Release_32 Skia_RazrI_4-1_Float_Compile_Debug_32
> Skia_Shuttle_Ubuntu12_ATI5770_Float_Compile_Debug_32
> Skia_Shuttle_Ubuntu12_ATI5770_Float_Compile_Release_64
> Skia_Shuttle_Ubuntu12_NaCl_Compile_Debug
> Skia_Shuttle_Win7_Intel_Float_ANGLE_Compile_Debug_32
> Skia_Shuttle_Win7_Intel_Float_Compile_Debug_32
> Skia_Shuttle_Win7_Intel_Float_DirectWrite_Compile_Debug_32
> Skia_iOS_Compile_Debug_32
> On 2013/04/03 17:10:45, borenet wrote:
> > On 2013/04/03 17:09:38, rmistry wrote:
> > > Dont they have to be comma separated?
> > 
> > No:
> https://sites.google.com/site/skiadocs/developer-documentation/skia-trybots
> 
> Patchset 4 uses commas instead of spaces as delimiters, because submit_try
> supports either one as of https://code.google.com/p/skia/source/detail?r=8521
!
> 
> Unless I hear complaints, I will commit at patchset 4, and update
documentation
> accordingly.

LGTM. Consistency wins.

[epoger, 2013-04-03 18:48:26]

Committed patchset #4 manually as r8522 (presubmit successful).

[epoger, 2013-04-04 17:38:17]

Check out this cool command line I can now run to make sure my CL won't break
any warnings-as-errors on various platforms:

tools/submit_try $CHANGENAME --bot $(cat trybots_to_run/skia/compile) -r $(svn
info | grep ^Revision | awk '{print $2}')

I just ran it for the first time on
https://codereview.chromium.org/13650002/#ps18002