Commit c15d2a0a authored by tandrii@chromium.org's avatar tandrii@chromium.org

Rietveld: retry on win on "Errno 10060" socket error.

R=sergeyberezin@chromium.org
BUG=537417

Review URL: https://codereview.chromium.org/1388463002

git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@296995 0039d316-1c4b-4281-b951-d872f2087c98
parent 95a3849a
...@@ -15,10 +15,13 @@ The following hypothesis are made: ...@@ -15,10 +15,13 @@ The following hypothesis are made:
""" """
import copy import copy
import errno
import json import json
import logging import logging
import re import re
import socket
import ssl import ssl
import sys
import time import time
import urllib import urllib
import urllib2 import urllib2
...@@ -429,7 +432,14 @@ class Rietveld(object): ...@@ -429,7 +432,14 @@ class Rietveld(object):
if retry >= (self._maxtries - 1): if retry >= (self._maxtries - 1):
raise raise
if (not 'Name or service not known' in e.reason and if (not 'Name or service not known' in e.reason and
not 'EOF occurred in violation of protocol' in e.reason): not 'EOF occurred in violation of protocol' in e.reason and
# On windows we hit weird bug http://crbug.com/537417
# with message '[Errno 10060] A connection attempt failed...'
not (sys.platform.startswith('win') and
isinstance(e.reason, socket.error) and
e.reason.errno == errno.ETIMEDOUT
)
):
# Usually internal GAE flakiness. # Usually internal GAE flakiness.
raise raise
except ssl.SSLError, e: except ssl.SSLError, e:
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment