Allow presubmit_support to fetch issue data from Rietveld.

Allow presubmit_support to fetch issue data from Rietveld.

R=ilevy@chromium.org, maruel@chromium.org
BUG=

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

[Isaac (away), 2013-04-24 03:45:35]

lgtm

[commit-bot: I haz the power, 2013-04-24 03:54:53]

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

[commit-bot: I haz the power, 2013-04-24 03:56:27]

Presubmit check for 13966016-1 failed and returned exit status 1.

INFO:root:Found 1 file(s).
INFO:PRESUBMIT:Running pylint on 60 files
Running presubmit commit checks ...
Running /b/commit-queue/workdir/tools/depot_tools/PRESUBMIT.py
Running tests/gclient_utils_test.py
Running tests/watchlists_unittest.py
Running tests/checkout_test.py
Running tests/scm_unittest.py
Running tests/gcl_unittest.py
Running tests/patch_test.py
Running tests/fix_encoding_test.py
Running tests/presubmit_unittest.py
Running tests/trychange_unittest.py
Running tests/rietveld_test.py
Running tests/owners_unittest.py
Running tests/breakpad_unittest.py
Running tests/gclient_test.py
Running tests/gclient_smoketest.py
Running tests/subprocess2_test.py
Running tests/git_cl_test.py
Running tests/gclient_scm_test.py
INFO:root:/usr/bin/python /b/google_appengine/dev_appserver.py . --port 8080
--storage /tmp/rietveld_testqE68ul --clear_search_indexes
--skip_sdk_update_check
Setting up test upstream git repo...
Setting up test git repo...
TESTING: uploading to bogus server
 test | 1 +
 1 file changed, 1 insertion(+)
Using 50% similarity for rename/copy detection. Override with --similarity.
Loaded authentication cookies from /home/chrome-bot/.codereview_upload_cookies
Running presubmit upload checks ...

Presubmit checks passed.
Upload server: http://bogus.example.com:80 (change with -s/--server)
Loaded authentication cookies from /home/chrome-bot/.codereview_upload_cookies

Got exception while uploading -- saving description to
/home/chrome-bot/.git_cl_description_backup

TESTING: description was backed up
PASS
Setting up test upstream git repo...
Setting up test git repo...
TESTING: git-cl upload wants a server
TESTING: git-cl status has no issue
TESTING: upload succeeds (needs a server running on localhost)

WARNING: Use -t or --title to set the title of the patchset.
In the near future, -m or --message will send a message instead.
See http://goo.gl/JGg0Z for details.

TESTING: git-cl status now knows the issue
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed

  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--    
0
100    77    0     0  100    77      0   1427 --:--:-- --:--:-- --:--:--  1452
TESTING: Base URL contains branch name
TESTING: git-cl push ok
Using 50% similarity for rename/copy detection. Override with --similarity.
Loaded authentication cookies from /home/chrome-bot/.codereview_upload_cookies
Running presubmit commit checks ...

Presubmit checks passed.
Description: 'foo-quux\n\nReview URL: http://localhost:8080/5629499534213120'
Closing issue (you may be prompted for your codereview password)...
TESTING: committed code has proper description
TESTING: issue no longer has a branch
TESTING: upstream repo has our commit
PASS
Setting up test SVN repo...
Setting up test git-svn repo...
The --set-upstream flag is deprecated and will be removed. Consider using
--track or --set-upstream-to
Branch feature_branch set up to track local ref refs/remotes/trunk.
TESTING: Guessing upstream branch for refs/remotes/trunk
The --set-upstream flag is deprecated and will be removed. Consider using
--track or --set-upstream-to
Branch feature_branch set up to track local ref refs/remotes/some_branch.
TESTING: Guessing upstream branch for refs/remotes/some_branch
PASS
Setting up test SVN repo...
Setting up test git-svn repo...
TESTING: upload succeeds

WARNING: Use -t or --title to set the title of the patchset.
In the near future, -m or --message will send a message instead.
See http://goo.gl/JGg0Z for details.

TESTING: git-cl dcommits ok
Using 50% similarity for rename/copy detection. Override with --similarity.
Loaded authentication cookies from /home/chrome-bot/.codereview_upload_cookies
Running presubmit commit checks ...

Presubmit checks passed.
Description: 'test\n\nBUG=\n\nReview URL:
http://localhost:8080/5910974510923776'
Closing issue (you may be prompted for your codereview password)...
PASS
Setting up test SVN repo...
Setting up test git-svn repo...
TESTING: upload succeeds

WARNING: Use -t or --title to set the title of the patchset.
In the near future, -m or --message will send a message instead.
See http://goo.gl/JGg0Z for details.

Switched to branch 'master'
Deleted branch abandoned (was a5df396).
TESTING: git-cl status dropped abandoned branch
PASS
Setting up test SVN repo...
Setting up test remote git-svn-submodule repo...
Switched to branch 'master'
TESTING: dcommitted code
Switched to branch 'git-cl-cherry-pick'
Using 50% similarity for rename/copy detection. Override with --similarity.
Description: 'dcommit'
TESTING: svn got new revision
TESTING: svn diff is correct
TESTING: git svn fetch gets new svn revision
PASS
Setting up test upstream git repo...
Setting up test git repo...
TESTING: upload succeeds (needs a server running on localhost)

WARNING: Use -t or --title to set the title of the patchset.
In the near future, -m or --message will send a message instead.
See http://goo.gl/JGg0Z for details.

PASS
Setting up test SVN repo...
Setting up test git-svn repo...
TESTING: git-cl upload hook fails
Command "git config rietveld.server" failed.
Could not find settings file. You must configure your review setup by running
"git cl config".
TESTING: git-cl dcommit hook fails
Command "git config rietveld.server" failed.
Could not find settings file. You must configure your review setup by running
"git cl config".
PASS
Setting up test SVN repo...
Setting up test git-svn repo...
TESTING: dcommitted code
Using 50% similarity for rename/copy detection. Override with --similarity.
Description: 'dcommit'
TESTING: post-cl-dcommit hook executed
PASS
Setting up test upstream git repo...
Setting up test git repo...
TESTING: upload succeeds (needs a server running on localhost)

WARNING: Use -t or --title to set the title of the patchset.
In the near future, -m or --message will send a message instead.
See http://goo.gl/JGg0Z for details.

TESTING: description shouldn't contain unrelated commits
PASS
Setting up test SVN repo...
Setting up test git-svn repo...
TESTING: upload succeeds (needs a server running on localhost)

WARNING: Use -t or --title to set the title of the patchset.
In the near future, -m or --message will send a message instead.
See http://goo.gl/JGg0Z for details.

TESTING: git-cl status now knows the issue
TESTING: git cl patch 5981343255101440
PASS
Setting up test SVN repo...
Setting up test git-svn repo...
TESTING: git-cl upload wants a server
TESTING: git-cl status has no issue
TESTING: upload succeeds (needs a server running on localhost)

WARNING: Use -t or --title to set the title of the patchset.
In the near future, -m or --message will send a message instead.
See http://goo.gl/JGg0Z for details.

TESTING: git-cl status now knows the issue
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed

  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--    
0
100    77    0     0  100    77      0   1597 --:--:-- --:--:-- --:--:--  1638
TESTING: git-cl dcommits ok
Using 50% similarity for rename/copy detection. Override with --similarity.
Loaded authentication cookies from /home/chrome-bot/.codereview_upload_cookies
Running presubmit commit checks ...

Presubmit checks passed.
Description: 'foo-quux\n\nReview URL: http://localhost:8080/5840605766746112'
Closing issue (you may be prompted for your codereview password)...
TESTING: dcommitted code has proper description
TESTING: issue no longer has a branch
TESTING: upstream svn has our commit
PASS
Checking out rietveld...
Running save-description-on-failure.sh
Running push-basic.sh
Running upstream.sh
Running submit-from-new-dir.sh
Running abandon.sh
Running submodule-merge-test.sh
Running upload-local-tracking-branch.sh
Running hooks.sh
Running post-dcommit-hook-test.sh
Running upload-stale.sh
Running patch.sh
Running basic.sh

** Presubmit ERRORS **
Missing LGTM from an OWNER for these files:
    depot_tools/presubmit_support.py

Presubmit checks took 92.4s to calculate.

[iannucci, 2013-04-24 04:01:42]

*ahem* that was a bit cheeky of me... dpranke@ can you take a look?

Context: The presubmit trybots need to be able to get the description into
presubmit_support somehow. I figured that since presubmit_support literally has
everything necessary to get this information already, I might as well extend it
to optionally ping the api.

[Dirk Pranke, 2013-04-24 04:08:58]

lgtm, though a little weird. I'm gonna be curious to see why/how we're invoking
this script via the command line.

[iannucci, 2013-04-24 04:10:06]

On 2013/04/24 04:08:58, Dirk Pranke wrote:
> lgtm, though a little weird. I'm gonna be curious to see why/how we're
invoking
> this script via the command line.

(pre-fetch build recipe, but I think you get the idea)

http://src.chromium.org/viewvc/chrome/trunk/tools/build/scripts/slave/recipes...

[iannucci, 2013-04-24 04:11:37]

On 2013/04/24 04:10:06, iannucci wrote:
> On 2013/04/24 04:08:58, Dirk Pranke wrote:
> > lgtm, though a little weird. I'm gonna be curious to see why/how we're
> invoking
> > this script via the command line.
> 
> (pre-fetch build recipe, but I think you get the idea)
> 
>
http://src.chromium.org/viewvc/chrome/trunk/tools/build/scripts/slave/recipes...

And the CL to use it: https://chromiumcodereview.appspot.com/13910019/

[commit-bot: I haz the power, 2013-04-24 04:12:09]

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

[iannucci, 2013-04-24 04:12:31]

On 2013/04/24 04:11:37, iannucci wrote:
> On 2013/04/24 04:10:06, iannucci wrote:
> > On 2013/04/24 04:08:58, Dirk Pranke wrote:
> > > lgtm, though a little weird. I'm gonna be curious to see why/how we're
> > invoking
> > > this script via the command line.
> > 
> > (pre-fetch build recipe, but I think you get the idea)
> > 
> >
>
http://src.chromium.org/viewvc/chrome/trunk/tools/build/scripts/slave/recipes...
> 
> And the CL to use it: https://chromiumcodereview.appspot.com/13910019/

Also, if it makes you feel better, CQ already uses presubmit_support via the
command line :P

[commit-bot: I haz the power, 2013-04-24 04:13:51]

Change committed as 196023