OLD | NEW |
1 # coding: utf-8 | 1 # coding: utf-8 |
2 # Copyright (c) 2012 The Chromium Authors. All rights reserved. | 2 # Copyright (c) 2012 The Chromium Authors. All rights reserved. |
3 # Use of this source code is governed by a BSD-style license that can be | 3 # Use of this source code is governed by a BSD-style license that can be |
4 # found in the LICENSE file. | 4 # found in the LICENSE file. |
5 """Defines class Rietveld to easily access a rietveld instance. | 5 """Defines class Rietveld to easily access a rietveld instance. |
6 | 6 |
7 Security implications: | 7 Security implications: |
8 | 8 |
9 The following hypothesis are made: | 9 The following hypothesis are made: |
10 - Rietveld enforces: | 10 - Rietveld enforces: |
(...skipping 415 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
426 flake_codes = [500, 502, 503] | 426 flake_codes = [500, 502, 503] |
427 if retry_on_404: | 427 if retry_on_404: |
428 flake_codes.append(404) | 428 flake_codes.append(404) |
429 if e.code not in flake_codes: | 429 if e.code not in flake_codes: |
430 raise | 430 raise |
431 except urllib2.URLError, e: | 431 except urllib2.URLError, e: |
432 if retry >= (self._maxtries - 1): | 432 if retry >= (self._maxtries - 1): |
433 raise | 433 raise |
434 if (not 'Name or service not known' in e.reason and | 434 if (not 'Name or service not known' in e.reason and |
435 not 'EOF occurred in violation of protocol' in e.reason and | 435 not 'EOF occurred in violation of protocol' in e.reason and |
436 not 'Connection timed out' in e.reason and | 436 not 'timed out' in e.reason and |
437 not 'The handshake operation timed out' in e.reason and | 437 not 'The handshake operation timed out' in e.reason and |
438 # On windows we hit weird bug http://crbug.com/537417 | 438 # On windows we hit weird bug http://crbug.com/537417 |
439 # with message '[Errno 10060] A connection attempt failed...' | 439 # with message '[Errno 10060] A connection attempt failed...' |
440 not (sys.platform.startswith('win') and | 440 not (sys.platform.startswith('win') and |
441 isinstance(e.reason, socket.error) and | 441 isinstance(e.reason, socket.error) and |
442 e.reason.errno == errno.ETIMEDOUT | 442 e.reason.errno == errno.ETIMEDOUT |
443 ) | 443 ) |
444 ): | 444 ): |
445 # Usually internal GAE flakiness. | 445 # Usually internal GAE flakiness. |
446 raise | 446 raise |
(...skipping 285 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
732 self, issue, patchset, reason, clobber, revision, builders_and_tests, | 732 self, issue, patchset, reason, clobber, revision, builders_and_tests, |
733 master=None, category='cq'): | 733 master=None, category='cq'): |
734 logging.info('ReadOnlyRietveld: triggering try jobs %r for issue %d' % | 734 logging.info('ReadOnlyRietveld: triggering try jobs %r for issue %d' % |
735 (builders_and_tests, issue)) | 735 (builders_and_tests, issue)) |
736 | 736 |
737 def trigger_distributed_try_jobs( # pylint:disable=R0201 | 737 def trigger_distributed_try_jobs( # pylint:disable=R0201 |
738 self, issue, patchset, reason, clobber, revision, masters, | 738 self, issue, patchset, reason, clobber, revision, masters, |
739 category='cq'): | 739 category='cq'): |
740 logging.info('ReadOnlyRietveld: triggering try jobs %r for issue %d' % | 740 logging.info('ReadOnlyRietveld: triggering try jobs %r for issue %d' % |
741 (masters, issue)) | 741 (masters, issue)) |
OLD | NEW |