Bug 1300663 - Catch HTTPError when query for symbols url. r=jlund, a=test-only
☠☠ backed out by 5eb0a600c391 ☠ ☠
authorArmen Zambrano Gasparnian <armenzg@mozilla.com>
Fri, 23 Sep 2016 13:37:58 -0400
changeset 348366 90a578222d6c254a19c4adc48737cb2d611be197
parent 348365 d94b9b11046950a28fa3232507f6addc68ae4992
child 348367 149e6651b5c8d9cfdca8379357784b30e062890d
push id6431
push userryanvm@gmail.com
push dateTue, 27 Sep 2016 21:51:52 +0000
treeherdermozilla-beta@0c6ccccaee82 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersjlund, test-only
Bug 1300663 - Catch HTTPError when query for symbols url. r=jlund, a=test-only If we don't catch HTTPError, the whole job fails since we get an uncaught exception. MozReview-Commit-ID: 8jwW7ZSieyC
--- a/testing/mozharness/mozharness/mozilla/testing/testbase.py
+++ b/testing/mozharness/mozharness/mozilla/testing/testbase.py
@@ -194,17 +194,17 @@ class TestingMixin(VirtualenvMixin, Buil
         elif self.installer_url:
             symbols_url = self.query_prefixed_build_dir_url('.crashreporter-symbols.zip')
             # Check if the URL exists. If not, use none to allow mozcrash to auto-check for symbols
                 if symbols_url:
                     self._urlopen(symbols_url, timeout=120)
                     self.symbols_url = symbols_url
-            except (urllib2.URLError, socket.error, socket.timeout):
+            except (urllib2.HTTPError, urllib2.URLError, socket.error, socket.timeout):
                 self.exception("Can't figure out symbols_url from installer_url: %s!" % self.installer_url, level=WARNING)
         # If no symbols URL can be determined let minidump_stackwalk query the symbols.
         # As of now this only works for Nightly and release builds.
         if not self.symbols_url:
             self.warning("No symbols_url found. Let minidump_stackwalk query for symbols.")
         return self.symbols_url