Bug 1506319 Change log levels for bouncer check failures r=mtabara
authorSimon Fraser <sfraser@mozilla.com>
Mon, 19 Nov 2018 11:31:28 +0000
changeset 503389 3ebe77971c61f5d535272ebfde2616d8b01ac696
parent 503388 2666875fcf32354c20da8e00fab923f301879a14
child 503390 df938148d900a6010c1a31471e7939d7282a53b5
push id10290
push userffxbld-merge
push dateMon, 03 Dec 2018 16:23:23 +0000
treeherdermozilla-beta@700bed2445e6 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersmtabara
bugs1506319
milestone65.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1506319 Change log levels for bouncer check failures r=mtabara Differential Revision: https://phabricator.services.mozilla.com/D12259
testing/mozharness/scripts/release/bouncer_check.py
--- a/testing/mozharness/scripts/release/bouncer_check.py
+++ b/testing/mozharness/scripts/release/bouncer_check.py
@@ -115,25 +115,25 @@ class BouncerCheck(BaseScript, Virtualen
         ]
 
         def do_check_url():
             self.log("Checking {}".format(url))
             r = session.head(url, verify=True, timeout=10, allow_redirects=True)
             try:
                 r.raise_for_status()
             except Exception:
-                self.warning("FAIL: {}, status: {}".format(url, r.status_code))
+                self.error("FAIL: {}, status: {}".format(url, r.status_code))
                 raise
 
             final_url = urlparse(r.url)
             if final_url.scheme != 'https':
-                self.warning('FAIL: URL scheme is not https: {}'.format(r.url))
+                self.error('FAIL: URL scheme is not https: {}'.format(r.url))
 
             if final_url.netloc not in mozilla_locations:
-                self.warning('FAIL: host not in allowed locations: {}'.format(r.url))
+                self.error('FAIL: host not in allowed locations: {}'.format(r.url))
 
         retry(do_check_url, sleeptime=3, max_sleeptime=10, attempts=3)
 
     def get_urls(self):
         for product in self.config["products"].values():
             if not product["check_uptake"]:
                 continue
             product_name = product["product-name"] % {"version": self.config["version"]}