bug 1381597 - switch bbb test to a warning until we smooth out the issues. r=callek
authorAki Sasaki <asasaki@mozilla.com>
Thu, 28 Sep 2017 17:11:17 -0700
changeset 383615 c37ef428d978f4dd447a8e0c546ff8537b20d9f5
parent 383614 f44332154b9489f4499dbcdf522c5cd0b40cc554
child 383616 772f1641588ac2fc20ff6ec5fc14df98314f4660
push id95596
push userasasaki@mozilla.com
push dateFri, 29 Sep 2017 00:11:26 +0000
treeherdermozilla-inbound@c37ef428d978 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewerscallek
bugs1381597
milestone58.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 1381597 - switch bbb test to a warning until we smooth out the issues. r=callek
taskcluster/taskgraph/util/verify.py
--- a/taskcluster/taskgraph/util/verify.py
+++ b/taskcluster/taskgraph/util/verify.py
@@ -1,22 +1,24 @@
 # -*- coding: utf-8 -*-
 # This Source Code Form is subject to the terms of the Mozilla Public
 # License, v. 2.0. If a copy of the MPL was not distributed with this
 # file, You can obtain one at http://mozilla.org/MPL/2.0/.
 
 from __future__ import absolute_import, print_function, unicode_literals
 
+import logging
 import re
 import os
 import sys
 
 from .. import GECKO
 from taskgraph.util.bbb_validation import valid_bbb_builders
 
+logger = logging.getLogger(__name__)
 base_path = os.path.join(GECKO, 'taskcluster', 'docs')
 
 
 class VerificationSequence(object):
     """
     Container for a sequence of verifications over a TaskGraph. Each
     verification is represented as a callable taking (task, taskgraph,
     scratch_pad), called for each task in the taskgraph, and one more
@@ -162,12 +164,12 @@ def verify_bbb_builders_valid(task, task
     if task is None:
         return
     valid_builders = valid_bbb_builders()
     if valid_builders is None:
         return
     if task.task.get('workerType') == 'buildbot-bridge':
         buildername = task.task['payload']['buildername']
         if buildername not in valid_builders:
-            raise Exception(
+            logger.warning(
                 '{} uses an invalid buildbot buildername ("{}") '
                 ' - contact #releng for help'
                 .format(task.label, buildername))