Add crypto pre-submit that will add the openssl builder to the default try-bot list.

Add crypto pre-submit that will add the openssl builder to the default try-bot list.

BUG=None
TEST=git try should run a linux_redux try job too.


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=119094

[Ryan Sleevi, 2012-01-25 17:11:11]

Are we saying OpenSSL try failures should be fixed before committing?

If so, should we move the OpenSSL bot onto the main waterfall?

https://chromiumcodereview.appspot.com/9235031/diff/1/crypto/PRESUBMIT.py
File crypto/PRESUBMIT.py (right):

https://chromiumcodereview.appspot.com/9235031/diff/1/crypto/PRESUBMIT.py#new...
crypto/PRESUBMIT.py:1: # Copyright (c) 2011 The Chromium Authors. All rights
reserved.
2011 -> 2012

https://chromiumcodereview.appspot.com/9235031/diff/1/crypto/PRESUBMIT.py#new...
crypto/PRESUBMIT.py:5: """Chromium presubmit script for src/net.
nit: src/crypto

[Ryan Hamilton, 2012-01-25 17:11:50]

By only returning linux_redux, does that mean it'll run on *just* that try
slave, or will that be appended to the default list?

Thanks for doing this...

[Ryan Sleevi, 2012-01-25 17:17:51]

On 2012/01/25 17:11:50, Ryan Hamilton wrote:
> By only returning linux_redux, does that mean it'll run on *just* that try
> slave, or will that be appended to the default list?
> 
> Thanks for doing this...

It appends.

LGTM, btw

[joth, 2012-01-25 17:21:16]

On 25 January 2012 17:11, <rsleevi@chromium.org> wrote:

> Are we saying OpenSSL try failures should be fixed before committing?
>
> ideally :) (it's been having that affect in net/*nss for the last few
months)


> If so, should we move the OpenSSL bot onto the main waterfall?
>
> yeah.... we're still resource constrained for number of slaves available
for it, but this is the aim.
Having reliable trybots seems to be one pre-requisite for getting to main
waterfall, so I see this as a step toward that. I think?


> https://chromiumcodereview.**appspot.com/9235031/diff/1/**
>
crypto/PRESUBMIT.py<https://chromiumcodereview.appspot.com/9235031/diff/1/crypto/PRESUBMIT.py>
> File crypto/PRESUBMIT.py (right):
>
> https://chromiumcodereview.**appspot.com/9235031/diff/1/**
>
crypto/PRESUBMIT.py#newcode1<https://chromiumcodereview.appspot.com/9235031/diff/1/crypto/PRESUBMIT.py#newcode1>
> crypto/PRESUBMIT.py:1: # Copyright (c) 2011 The Chromium Authors. All
> rights reserved.
> 2011 -> 2012
>
> https://chromiumcodereview.**appspot.com/9235031/diff/1/**
>
crypto/PRESUBMIT.py#newcode5<https://chromiumcodereview.appspot.com/9235031/diff/1/crypto/PRESUBMIT.py#newcode5>
> crypto/PRESUBMIT.py:5: """Chromium presubmit script for src/net.
> nit: src/crypto
>
>
https://chromiumcodereview.**appspot.com/9235031/<https://chromiumcodereview....
>

[commit-bot: I haz the power, 2012-01-25 17:21:22]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/joth@chromium.org/9235031/1

[joth, 2012-01-25 17:22:49]

On 25 January 2012 17:17, <rsleevi@chromium.org> wrote:

> On 2012/01/25 17:11:50, Ryan Hamilton wrote:
>
>> By only returning linux_redux, does that mean it'll run on *just* that try
>> slave, or will that be appended to the default list?
>>
>
>  Thanks for doing this...
>>
>
> It appends.
>
> LGTM, btw
>
> thanks for the quick review :)


>
https://chromiumcodereview.**appspot.com/9235031/<https://chromiumcodereview....
>

[Ryan Hamilton, 2012-01-25 17:28:19]

On 2012/01/25 17:17:51, Ryan Sleevi wrote:
> On 2012/01/25 17:11:50, Ryan Hamilton wrote:
> > By only returning linux_redux, does that mean it'll run on *just* that try
> > slave, or will that be appended to the default list?
> 
> It appends.

LGTM

I appreciate this feature so I don't h0ze myself again :>

[commit-bot: I haz the power, 2012-01-25 18:50:49]

Change committed as 119094