Bug 1303668 - Failing to find a symbols url should be a warning instead of a exception. r=whimboo
Failing to find symbols in this case should be turned into a warning rather than dumping the traceback
since we're going to rely on mozcrash doing the right thing later on.
This will reduce unnecessary reporting of symbols not being available.
MozReview-Commit-ID: GXO01B7vzGV
--- a/testing/mozharness/mozharness/mozilla/testing/testbase.py
+++ b/testing/mozharness/mozharness/mozilla/testing/testbase.py
@@ -195,17 +195,17 @@ class TestingMixin(VirtualenvMixin, Buil
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
try:
if symbols_url:
self._urlopen(symbols_url, timeout=120)
self.symbols_url = symbols_url
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)
+ self.warning("Can't figure out symbols_url from installer_url: %s!" % self.installer_url)
# 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