--- /dev/null Thu Jan 01 00:00:00 1970 +0000
+++ b/WebKitTools/Scripts/webkitpy/tool/bot/sheriff.py Fri Sep 17 09:02:29 2010 +0300
@@ -0,0 +1,131 @@
+# 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.")