WebKitTools/Scripts/webkitpy/tool/bot/sheriff.py
author Dremov Kirill (Nokia-D-MSW/Tampere) <kirill.dremov@nokia.com>
Mon, 04 Oct 2010 01:32:07 +0300
changeset 2 303757a437d3
parent 0 4f2f89ce4247
permissions -rw-r--r--
Revision: 201037 Kit: 201039

# Copyright (c) 2010 Google Inc. All rights reserved.
#
# Redistribution and use in source and binary forms, with or without
# modification, are permitted provided that the following conditions are
# met:
#
#     * Redistributions of source code must retain the above copyright
# notice, this list of conditions and the following disclaimer.
#     * Redistributions in binary form must reproduce the above
# copyright notice, this list of conditions and the following disclaimer
# in the documentation and/or other materials provided with the
# distribution.
#     * Neither the name of Google Inc. nor the names of its
# contributors may be used to endorse or promote products derived from
# this software without specific prior written permission.
#
# THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
# "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
# LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
# A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
# OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
# SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
# LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
# DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
# THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
# (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
# OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.

from webkitpy.common.checkout.changelog import view_source_url
from webkitpy.common.net.bugzilla import parse_bug_id
from webkitpy.common.system.deprecated_logging import log
from webkitpy.common.system.executive import ScriptError
from webkitpy.tool.grammar import join_with_separators


class Sheriff(object):
    def __init__(self, tool, sheriffbot):
        self._tool = tool
        self._sheriffbot = sheriffbot

    def post_irc_warning(self, commit_info, builders):
        irc_nicknames = sorted([party.irc_nickname for
                                party in commit_info.responsible_parties()
                                if party.irc_nickname])
        irc_prefix = ": " if irc_nicknames else ""
        irc_message = "%s%s%s might have broken %s" % (
            ", ".join(irc_nicknames),
            irc_prefix,
            view_source_url(commit_info.revision()),
            join_with_separators([builder.name() for builder in builders]))

        self._tool.irc().post(irc_message)

    def post_rollout_patch(self, svn_revision, rollout_reason):
        # Ensure that svn_revision is a number (and not an option to
        # create-rollout).
        try:
            svn_revision = int(svn_revision)
        except:
            raise ScriptError(message="Invalid svn revision number \"%s\"."
                              % svn_revision)

        if rollout_reason.startswith("-"):
            raise ScriptError(message="The rollout reason may not begin "
                              "with - (\"%s\")." % rollout_reason)

        output = self._sheriffbot.run_webkit_patch([
            "create-rollout",
            "--force-clean",
            # In principle, we should pass --non-interactive here, but it
            # turns out that create-rollout doesn't need it yet.  We can't
            # pass it prophylactically because we reject unrecognized command
            # line switches.
            "--parent-command=sheriff-bot",
            svn_revision,
            rollout_reason,
        ])
        return parse_bug_id(output)

    def _rollout_reason(self, builders):
        # FIXME: This should explain which layout tests failed
        # however, that would require Build objects here, either passed
        # in through failure_info, or through Builder.latest_build.
        names = [builder.name() for builder in builders]
        return "Caused builders %s to fail." % join_with_separators(names)

    def post_automatic_rollout_patch(self, commit_info, builders):
        # For now we're only posting rollout patches for commit-queue patches.
        commit_bot_email = "eseidel@chromium.org"
        if commit_bot_email == commit_info.committer_email():
            try:
                self.post_rollout_patch(commit_info.revision(),
                                        self._rollout_reason(builders))
            except ScriptError, e:
                log("Failed to create-rollout.")

    def post_blame_comment_on_bug(self, commit_info, builders, blame_list):
        if not commit_info.bug_id():
            return
        comment = "%s might have broken %s" % (
            view_source_url(commit_info.revision()),
            join_with_separators([builder.name() for builder in builders]))
        if len(blame_list) > 1:
            comment += "\nThe following changes are on the blame list:\n"
            comment += "\n".join(map(view_source_url, blame_list))
        self._tool.bugs.post_comment_to_bug(commit_info.bug_id(),
                                            comment,
                                            cc=self._sheriffbot.watchers)

    # FIXME: Should some of this logic be on BuildBot?
    def provoke_flaky_builders(self, revisions_causing_failures):
        # We force_build builders that are red but have not "failed" (i.e.,
        # been red twice). We do this to avoid a deadlock situation where a
        # flaky test blocks the commit-queue and there aren't any other
        # patches being landed to re-spin the builder.
        failed_builders = sum([revisions_causing_failures[key] for
                               key in revisions_causing_failures.keys()], [])
        failed_builder_names = \
            set([builder.name() for builder in failed_builders])
        idle_red_builder_names = \
            set([builder["name"]
                 for builder in self._tool.buildbot.idle_red_core_builders()])

        # We only want to provoke these builders if they are idle and have not
        # yet "failed" (i.e., been red twice) to avoid overloading the bots.
        flaky_builder_names = idle_red_builder_names - failed_builder_names

        for name in flaky_builder_names:
            flaky_builder = self._tool.buildbot.builder_with_name(name)
            flaky_builder.force_build(username=self._sheriffbot.name,
                                      comments="Probe for flakiness.")