Bug 1332255 - [followup] fix buildername for linux64-pgo talos. r=dustin

For correct platform name in TH, the buildername must contain the pgo
string in the buildername. We work a bit more generic by matching any
build variant.
This commit is contained in:
Wander Lairson Costa 2017-01-23 14:14:00 -05:00
Родитель 26d5dbe35c
Коммит d63bdf19ad
1 изменённых файлов: 19 добавлений и 3 удалений

Просмотреть файл

@ -35,6 +35,7 @@ from voluptuous import (
import copy
import logging
import os.path
import re
ARTIFACT_URL = 'https://queue.taskcluster.net/v1/task/{}/artifacts/{}'
WORKER_TYPE = {
@ -551,6 +552,17 @@ def set_retry_exit_status(config, tests):
test['retry-exit-status'] = 4
yield test
@transforms.add
def remove_linux_pgo_try_talos(config, tests):
"""linux64-pgo talos tests don't run on try."""
def predicate(test):
return not(
test['test-platform'] == 'linux64-pgo/opt'
and test['suite'] == 'talos'
and config.params['project'] == 'try'
)
for test in filter(predicate, tests):
yield test
@transforms.add
def make_task_description(config, tests):
@ -625,10 +637,8 @@ def make_task_description(config, tests):
# yield only the task description, discarding the test description
yield taskdesc
worker_setup_functions = {}
def worker_setup_function(name):
def wrap(func):
worker_setup_functions[name] = func
@ -964,9 +974,15 @@ def buildbot_bridge_setup(config, test, taskdesc):
taskdesc['worker-type'] = 'buildbot-bridge/buildbot-bridge'
if test.get('suite', '') == 'talos':
buildername = '{} {} talos {}'.format(
# on linux64-<variant>/<build>, we add the variant to the buildername
m = re.match(r'\w+-([^/]+)/.*', test['test-platform'])
variant = ''
if m and m.group(1):
variant = m.group(1) + ' '
buildername = '{} {} {}talos {}'.format(
BUILDER_NAME_PREFIX[platform],
branch,
variant,
test_name
)
if buildername.startswith('Ubuntu'):