Create trychange_git.py, for starting tryjobs in a pure-git world.

Create trychange_git.py, for starting tryjobs in a pure-git world.

This is a companion CL to https://codereview.chromium.org/23112007/

This file takes your local feature branch, pushes it to a special try-ref in a
remote repo, and then posts a json blob to an appengine app that a buildbot
tryserver will be polling from. I'd like to integrate it into git_cl.py, but I
haven't figured out the best way to determine if a user wants to issue a tryjob
this way or the normal way via rietveld.

[cmp, 2013-08-20 05:52:29]

lgtm with comment and nit

But this should probably not be in depot_tools for now.  Things in there have a
way of becoming used even if they're only in prototype form.  Can land this in
the chromium-tryserver tools dir once that's in place to just store it somewhere
for now until we figure out what route we're headed.

https://codereview.chromium.org/23205010/diff/4001/trychange_git.py
File trychange_git.py (right):

https://codereview.chromium.org/23205010/diff/4001/trychange_git.py#newcode126
trychange_git.py:126: server = settings.get('TRYSERVER_HTTP_HOST')
It would be for the best to enforce that TRYSERVER_HTTP_HOST is set to a https
URL.  Either here or before line 110 in PostJob().

https://codereview.chromium.org/23205010/diff/4001/trychange_git.py#newcode138
trychange_git.py:138: if __name__ == "__main__":
single quotes here instead of double quotes

[agable, 2013-08-20 07:48:08]

Okay, when the appengine app review is done and we decide where to land that
(chrome/trunk/tools/chromium-jobserver? Robbie is opposed to having 'tryserver'
in the name since that is more closely tied to the buildbot instance) I'll land
this in the same place.

https://codereview.chromium.org/23205010/diff/4001/trychange_git.py
File trychange_git.py (right):

https://codereview.chromium.org/23205010/diff/4001/trychange_git.py#newcode126
trychange_git.py:126: server = settings.get('TRYSERVER_HTTP_HOST')
On 2013/08/20 05:52:29, cmp wrote:
> It would be for the best to enforce that TRYSERVER_HTTP_HOST is set to a https
> URL.  Either here or before line 110 in PostJob().

Done.

https://codereview.chromium.org/23205010/diff/4001/trychange_git.py#newcode138
trychange_git.py:138: if __name__ == "__main__":
On 2013/08/20 05:52:29, cmp wrote:
> single quotes here instead of double quotes

Done.

[agable, 2013-08-21 13:02:14]

On 2013/08/20 07:48:08, Aaron Gable wrote:
> Okay, when the appengine app review is done and we decide where to land that
> (chrome/trunk/tools/chromium-jobserver? Robbie is opposed to having
'tryserver'
> in the name since that is more closely tied to the buildbot instance) I'll
land
> this in the same place.
> 
> https://codereview.chromium.org/23205010/diff/4001/trychange_git.py
> File trychange_git.py (right):
> 
> https://codereview.chromium.org/23205010/diff/4001/trychange_git.py#newcode126
> trychange_git.py:126: server = settings.get('TRYSERVER_HTTP_HOST')
> On 2013/08/20 05:52:29, cmp wrote:
> > It would be for the best to enforce that TRYSERVER_HTTP_HOST is set to a
https
> > URL.  Either here or before line 110 in PostJob().
> 
> Done.
> 
> https://codereview.chromium.org/23205010/diff/4001/trychange_git.py#newcode138
> trychange_git.py:138: if __name__ == "__main__":
> On 2013/08/20 05:52:29, cmp wrote:
> > single quotes here instead of double quotes
> 
> Done.

Committed in appropriate directory as https://codereview.chromium.org/23242010/,
closing issue.

[cmp, 2013-08-22 04:06:45]

I agree about changing the name from tryserver to something else for the same
reason.  That SGTM.