Added transient 502 errors to 'git_retry' list.

Added transient 502 errors to 'git_retry' list.

BUG=chromium:430343
TEST=None

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

[dnj, 2014-11-05 06:24:18]

dnj@chromium.org changed reviewers:
+ davidjames@chromium.org, iannucci@chromium.org, vadimsh@chromium.org

[dnj, 2014-11-05 06:24:19]

PTAL.

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

davidjames@google.com changed reviewers:
+ davidjames@google.com

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

lgtm

https://codereview.chromium.org/701843004/diff/1/git_common.py
File git_common.py (right):

https://codereview.chromium.org/701843004/diff/1/git_common.py#newcode91
git_common.py:91: r'The requested URL returned error: 502',
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.

[dnj, 2014-11-05 06:37:49]

I am going to opt out of including internal URLs. New patch set groups the
internal server error regexes. I'll sync up w/ Chromite when the current dust
settles :)

[dnj, 2014-11-05 06:39:48]

The CQ bit was checked by dnj@chromium.org

[commit-bot: I haz the power, 2014-11-05 06:40:31]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/701843004/40001

[commit-bot: I haz the power, 2014-11-05 06:40:33]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-11-05 06:40:34]

No LGTM from a valid reviewer yet. Only full committers are accepted.
Even if an LGTM may have been provided, it was from a non-committer,
_not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[dnj, 2014-11-05 18:33:56]

dnj@chromium.org changed reviewers:
+ hinoka@chromium.org
- davidjames@chromium.org, iannucci@chromium.org, vadimsh@chromium.org

[dnj, 2014-11-05 18:33:57]

+hinoka@ PTAL :)

[dnj, 2014-11-06 17:57:04]

dnj@chromium.org changed reviewers:
+ agable@chromium.org
- davidjames@google.com

[dnj, 2014-11-06 17:57:04]

+agable@ The imminent problem has passed but we might as well get this change
in.

[agable, 2014-11-06 19:32:47]

On 2014/11/06 at 17:57:04, dnj wrote:
> +agable@ The imminent problem has passed but we might as well get this change
in.

rslgtm since davidjames already lgtm'd.

[dnj, 2014-11-06 19:33:45]

The CQ bit was checked by dnj@chromium.org

[commit-bot: I haz the power, 2014-11-06 19:34:03]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/701843004/40001

[commit-bot: I haz the power, 2014-11-06 19:36:44]

Committed patchset #3 (id:40001) as 292868