author Rob Wood <rwood@mozilla.com>
Tue, 29 Aug 2017 11:44:58 -0400
changeset 377960 b4ea43ebf3461ba307a3b673f3ac9261506cd2a4
parent 372256 4e799d2c0555423eae7690b14d8be8aad59d39ee
child 389255 349b9517cb9b7b70449ffec2ac1a3b280e4ff83a
permissions -rw-r--r--
Bug 1390084 - Fix enabling gecko profiling on try; r=jmaher MozReview-Commit-ID: 8Cm9zH7lxGF

#!/usr/bin/env python
# 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/.
# ***** END LICENSE BLOCK *****

import argparse
import os
import re
from collections import defaultdict

from mozharness.base.script import PostScriptAction
from mozharness.base.transfer import TransferMixin

try_config_options = [
     {"action": "store",
     "dest": "try_message",
     "default": None,
     "help": "try syntax string to select tests to run",

test_flavors = {
    'browser-chrome': {},
    'chrome': {},
    'devtools-chrome': {},
    'mochitest': {},
    'xpcshell' :{},
    'reftest': {
        "path": lambda x: os.path.join("tests", "reftest", "tests", x)
    'crashtest': {
        "path": lambda x: os.path.join("tests", "reftest", "tests", x)
    'web-platform-tests': {
        "path": lambda x: os.path.join("tests", x.split("testing" + os.path.sep)[1])
    'web-platform-tests-reftests': {
        "path": lambda x: os.path.join("tests", x.split("testing" + os.path.sep)[1])
    'web-platform-tests-wdspec': {
        "path": lambda x: os.path.join("tests", x.split("testing" + os.path.sep)[1])

class TryToolsMixin(TransferMixin):
    """Utility functions for an interface between try syntax and out test harnesses.
    Requires log and script mixins."""

    harness_extra_args = None
    try_test_paths = {}
    known_try_arguments = {
        '--tag': ({
            'action': 'append',
            'dest': 'tags',
            'default': None,
        }, (
        '--setenv': ({
            'action': 'append',
            'dest': 'setenv',
            'default': [],
            'metavar': 'NAME=VALUE',
        }, (

    def _extract_try_message(self):
        msg = None
        buildbot_config = self.buildbot_config or {}
        if "try_message" in self.config and self.config["try_message"]:
            msg = self.config["try_message"]
        elif 'TRY_COMMIT_MSG' in os.environ:
            msg = os.environ['TRY_COMMIT_MSG']
        elif self._is_try():
            if 'sourcestamp' in buildbot_config and buildbot_config['sourcestamp'].get('changes'):
                msg = buildbot_config['sourcestamp']['changes'][-1].get('comments')

            if msg is None or len(msg) == 1024:
                # This commit message was potentially truncated or not available in
                # buildbot_config (e.g. if running in TaskCluster), get the full message
                # from hg.
                props = buildbot_config.get('properties', {})
                repo_url = 'https://hg.mozilla.org/%s/'
                if 'revision' in props and 'repo_path' in props:
                    rev = props['revision']
                    repo_path = props['repo_path']
                    # In TaskCluster we have no buildbot props, rely on env vars instead
                    rev = os.environ.get('GECKO_HEAD_REV')
                    repo_path = self.config.get('branch')
                if repo_path:
                    repo_url = repo_url % repo_path
                    repo_url = os.environ.get('GECKO_HEAD_REPOSITORY',
                                              repo_url % 'try')
                if not repo_url.endswith('/'):
                    repo_url += '/'

                url = '{}json-pushes?changeset={}&full=1'.format(repo_url, rev)

                pushinfo = self.load_json_from_url(url)
                for k, v in pushinfo.items():
                    if isinstance(v, dict) and 'changesets' in v:
                        msg = v['changesets'][-1]['desc']

            if not msg and 'try_syntax' in buildbot_config.get('properties', {}):
                # If we don't find try syntax in the usual place, check for it in an
                # alternate property available to tools using self-serve.
                msg = buildbot_config['properties']['try_syntax']
        if not msg:
            self.warning('Try message not found.')
        return msg

    def _extract_try_args(self, msg):
        """ Returns a list of args from a try message, for parsing """
        if not msg:
            return None
        all_try_args = None
        for line in msg.splitlines():
            if 'try: ' in line:
                # Autoland adds quotes to try strings that will confuse our
                # args later on.
                if line.startswith('"') and line.endswith('"'):
                    line = line[1:-1]
                # Allow spaces inside of [filter expressions]
                try_message = line.strip().split('try: ', 1)
                all_try_args = re.findall(r'(?:\[.*?\]|\S)+', try_message[1])
        if not all_try_args:
            self.warning('Try syntax not found in: %s.' % msg )
        return all_try_args

    def try_message_has_flag(self, flag, message=None):
        Returns True if --`flag` is present in message.
        parser = argparse.ArgumentParser()
        parser.add_argument('--' + flag, action='store_true')
        message = message or self._extract_try_message()
        if not message:
            return False
        msg_list = self._extract_try_args(message)
        args, _ = parser.parse_known_args(msg_list)
        return getattr(args, flag, False)

    def _is_try(self):
        repo_path = None
        if self.buildbot_config and 'properties' in self.buildbot_config:
            repo_path = self.buildbot_config['properties'].get('branch')
        get_branch = self.config.get('branch', repo_path)
        if get_branch is not None:
            on_try = ('try' in get_branch or 'Try' in get_branch)
        elif os.environ is not None:
            on_try = ('TRY_COMMIT_MSG' in os.environ)
            on_try = False
        return on_try

    def set_extra_try_arguments(self, action, success=None):
        """Finds a commit message and parses it for extra arguments to pass to the test
        harness command line and test paths used to filter manifests.

        Extracting arguments from a commit message taken directly from the try_parser.
        if not self._is_try():

        msg = self._extract_try_message()
        if not msg:

        all_try_args = self._extract_try_args(msg)
        if not all_try_args:

        parser = argparse.ArgumentParser(
            description=('Parse an additional subset of arguments passed to try syntax'
                         ' and forward them to the underlying test harness command.'))

        label_dict = {}
        def label_from_val(val):
            if val in label_dict:
                return label_dict[val]
            return '--%s' % val.replace('_', '-')

        for label, (opts, _) in self.known_try_arguments.iteritems():
            if 'action' in opts and opts['action'] not in ('append', 'store',
                                                           'store_true', 'store_false'):
                self.fatal('Try syntax does not support passing custom or store_const '
                           'arguments to the harness process.')
            if 'dest' in opts:
                label_dict[opts['dest']] = label

            parser.add_argument(label, **opts)

        parser.add_argument('--try-test-paths', nargs='*')
        (args, _) = parser.parse_known_args(all_try_args)
        self.try_test_paths = self._group_test_paths(args.try_test_paths)
        del args.try_test_paths

        out_args = defaultdict(list)
        # This is a pretty hacky way to echo arguments down to the harness.
        # Hopefully this can be improved once we have a configuration system
        # in tree for harnesses that relies less on a command line.
        for arg, value in vars(args).iteritems():
            if value:
                label = label_from_val(arg)
                _, flavors = self.known_try_arguments[label]

                for f in flavors:
                    if isinstance(value, bool):
                        # A store_true or store_false argument.
                    elif isinstance(value, list):
                        out_args[f].extend(['%s=%s' % (label, el) for el in value])
                        out_args[f].append('%s=%s' % (label, value))

        self.harness_extra_args = dict(out_args)

    def _group_test_paths(self, args):
        rv = defaultdict(list)

        if args is None:
            return rv

        for item in args:
            suite, path = item.split(":", 1)
        return rv

    def try_args(self, flavor):
        """Get arguments, test_list derived from try syntax to apply to a command"""
        args = []
        if self.harness_extra_args:
            args = self.harness_extra_args.get(flavor, [])[:]

        if self.try_test_paths.get(flavor):
            self.info('TinderboxPrint: Tests will be run from the following '
                      'files: %s.' % ','.join(self.try_test_paths[flavor]))
            args.extend(['--this-chunk=1', '--total-chunks=1'])

            path_func = test_flavors[flavor].get("path", lambda x:x)
            tests = [path_func(os.path.normpath(item)) for item in self.try_test_paths[flavor]]
            tests = []

        if args or tests:
            self.info('TinderboxPrint: The following arguments were forwarded from mozharness '
                      'to the test command:\nTinderboxPrint: \t%s -- %s' %
                      (" ".join(args), " ".join(tests)))

        return args, tests