Added transient 502 errors to 'git_retry' list.

git_common.py

 # Copyright 2014 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 # Monkeypatch IMapIterator so that Ctrl-C can kill everything properly.
 # Derived from https://gist.github.com/aljungberg/626518
 import multiprocessing.pool
 from multiprocessing.pool import IMapIterator
 def wrapper(func):
   def wrap(self, timeout=None):
@@ skipping 68 matching lines @@
     r'TLS packet with unexpected length was received',
 
     # crbug.com/187444
     r'RPC failed; result=\d+, HTTP code = \d+',
 
     # crbug.com/315421
     r'The requested URL returned error: 500 while accessing',
 
     # crbug.com/388876
     r'Connection timed out',
+
+    # crbug.com/430343
+    r'The requested URL returned error: 502',

[David James, 2014/11/05 06:30:15]

Can you merge this with the one above, like this?

https://chromium-review.googlesource.com/#/c/227513/

Maybe also add a comment about keeping the two in sync since they are
copy/pasted :(

Just "The requested URL returned error: 5" should be enough.

 )
 
 GIT_TRANSIENT_ERRORS_RE = re.compile('|'.join(GIT_TRANSIENT_ERRORS),
                                      re.IGNORECASE)
 
 # First version where the for-each-ref command's format string supported the
 # upstream:track token.
 MIN_UPSTREAM_TRACK_GIT_VERSION = (1, 9)
 
 class BadCommitRefException(Exception):
@@ skipping 662 matching lines @@
         hash=branch_hash, upstream=upstream_branch, ahead=ahead, behind=behind)
 
   # Set None for upstreams which are not branches (e.g empty upstream, remotes
   # and deleted upstream branches).
   missing_upstreams = {}
   for info in info_map.values():
     if info.upstream not in info_map and info.upstream not in missing_upstreams:
       missing_upstreams[info.upstream] = None
 
   return dict(info_map.items() + missing_upstreams.items())