1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
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.")
|
