ikarienator@chromium.org | faf3fdf | 2013-09-20 02:11:48 +0000 | [diff] [blame] | 1 | # Copyright 2013 The Chromium Authors. All rights reserved. |
| 2 | # Use of this source code is governed by a BSD-style license that can be |
| 3 | # found in the LICENSE file. |
| 4 | |
| 5 | """Interactive tool for finding reviewers/owners for a change.""" |
| 6 | |
Raul Tambre | 80ee78e | 2019-05-06 22:41:05 +0000 | [diff] [blame] | 7 | from __future__ import print_function |
| 8 | |
ikarienator@chromium.org | faf3fdf | 2013-09-20 02:11:48 +0000 | [diff] [blame] | 9 | import os |
| 10 | import copy |
| 11 | import owners as owners_module |
Bruce Dawson | 37740e2 | 2019-11-14 00:27:44 +0000 | [diff] [blame] | 12 | import random |
ikarienator@chromium.org | faf3fdf | 2013-09-20 02:11:48 +0000 | [diff] [blame] | 13 | |
| 14 | |
| 15 | def first(iterable): |
| 16 | for element in iterable: |
| 17 | return element |
| 18 | |
| 19 | |
| 20 | class OwnersFinder(object): |
| 21 | COLOR_LINK = '\033[4m' |
| 22 | COLOR_BOLD = '\033[1;32m' |
| 23 | COLOR_GREY = '\033[0;37m' |
| 24 | COLOR_RESET = '\033[0m' |
| 25 | |
| 26 | indentation = 0 |
| 27 | |
Edward Lemur | 707d70b | 2018-02-07 00:50:14 +0100 | [diff] [blame] | 28 | def __init__(self, files, local_root, author, reviewers, |
dtu | 944b605 | 2016-07-14 14:48:21 -0700 | [diff] [blame] | 29 | fopen, os_path, |
ikarienator@chromium.org | faf3fdf | 2013-09-20 02:11:48 +0000 | [diff] [blame] | 30 | email_postfix='@chromium.org', |
Jochen Eisinger | d0573ec | 2017-04-13 10:55:06 +0200 | [diff] [blame] | 31 | disable_color=False, |
Sylvain Defresne | b1f865d | 2019-02-12 12:38:22 +0000 | [diff] [blame] | 32 | override_files=None, |
| 33 | ignore_author=False): |
ikarienator@chromium.org | faf3fdf | 2013-09-20 02:11:48 +0000 | [diff] [blame] | 34 | self.email_postfix = email_postfix |
| 35 | |
| 36 | if os.name == 'nt' or disable_color: |
| 37 | self.COLOR_LINK = '' |
| 38 | self.COLOR_BOLD = '' |
| 39 | self.COLOR_GREY = '' |
| 40 | self.COLOR_RESET = '' |
| 41 | |
Jochen Eisinger | eb74476 | 2017-04-05 11:00:05 +0200 | [diff] [blame] | 42 | self.db = owners_module.Database(local_root, fopen, os_path) |
Jochen Eisinger | d0573ec | 2017-04-13 10:55:06 +0200 | [diff] [blame] | 43 | self.db.override_files = override_files or {} |
ikarienator@chromium.org | faf3fdf | 2013-09-20 02:11:48 +0000 | [diff] [blame] | 44 | self.db.load_data_needed_for(files) |
| 45 | |
| 46 | self.os_path = os_path |
| 47 | |
| 48 | self.author = author |
| 49 | |
| 50 | filtered_files = files |
| 51 | |
Edward Lemur | 707d70b | 2018-02-07 00:50:14 +0100 | [diff] [blame] | 52 | reviewers = list(reviewers) |
Sylvain Defresne | b1f865d | 2019-02-12 12:38:22 +0000 | [diff] [blame] | 53 | if author and not ignore_author: |
Edward Lemur | 707d70b | 2018-02-07 00:50:14 +0100 | [diff] [blame] | 54 | reviewers.append(author) |
| 55 | |
| 56 | # Eliminate files that existing reviewers can review. |
dtu | 944b605 | 2016-07-14 14:48:21 -0700 | [diff] [blame] | 57 | filtered_files = list(self.db.files_not_covered_by( |
Edward Lemur | 707d70b | 2018-02-07 00:50:14 +0100 | [diff] [blame] | 58 | filtered_files, reviewers)) |
ikarienator@chromium.org | faf3fdf | 2013-09-20 02:11:48 +0000 | [diff] [blame] | 59 | |
| 60 | # If some files are eliminated. |
| 61 | if len(filtered_files) != len(files): |
| 62 | files = filtered_files |
| 63 | # Reload the database. |
Jochen Eisinger | eb74476 | 2017-04-05 11:00:05 +0200 | [diff] [blame] | 64 | self.db = owners_module.Database(local_root, fopen, os_path) |
Jochen Eisinger | d0573ec | 2017-04-13 10:55:06 +0200 | [diff] [blame] | 65 | self.db.override_files = override_files or {} |
ikarienator@chromium.org | faf3fdf | 2013-09-20 02:11:48 +0000 | [diff] [blame] | 66 | self.db.load_data_needed_for(files) |
| 67 | |
| 68 | self.all_possible_owners = self.db.all_possible_owners(files, None) |
Sylvain Defresne | b1f865d | 2019-02-12 12:38:22 +0000 | [diff] [blame] | 69 | if author and author in self.all_possible_owners: |
| 70 | del self.all_possible_owners[author] |
ikarienator@chromium.org | faf3fdf | 2013-09-20 02:11:48 +0000 | [diff] [blame] | 71 | |
| 72 | self.owners_to_files = {} |
| 73 | self._map_owners_to_files(files) |
| 74 | |
| 75 | self.files_to_owners = {} |
| 76 | self._map_files_to_owners() |
| 77 | |
| 78 | self.owners_score = self.db.total_costs_by_owner( |
| 79 | self.all_possible_owners, files) |
| 80 | |
| 81 | self.original_files_to_owners = copy.deepcopy(self.files_to_owners) |
| 82 | self.comments = self.db.comments |
| 83 | |
| 84 | # This is the queue that will be shown in the interactive questions. |
| 85 | # It is initially sorted by the score in descending order. In the |
| 86 | # interactive questions a user can choose to "defer" its decision, then the |
| 87 | # owner will be put to the end of the queue and shown later. |
| 88 | self.owners_queue = [] |
| 89 | |
| 90 | self.unreviewed_files = set() |
| 91 | self.reviewed_by = {} |
| 92 | self.selected_owners = set() |
| 93 | self.deselected_owners = set() |
| 94 | self.reset() |
| 95 | |
| 96 | def run(self): |
| 97 | self.reset() |
| 98 | while self.owners_queue and self.unreviewed_files: |
| 99 | owner = self.owners_queue[0] |
| 100 | |
| 101 | if (owner in self.selected_owners) or (owner in self.deselected_owners): |
| 102 | continue |
| 103 | |
| 104 | if not any((file_name in self.unreviewed_files) |
| 105 | for file_name in self.owners_to_files[owner]): |
| 106 | self.deselect_owner(owner) |
| 107 | continue |
| 108 | |
| 109 | self.print_info(owner) |
| 110 | |
| 111 | while True: |
| 112 | inp = self.input_command(owner) |
| 113 | if inp == 'y' or inp == 'yes': |
| 114 | self.select_owner(owner) |
| 115 | break |
| 116 | elif inp == 'n' or inp == 'no': |
| 117 | self.deselect_owner(owner) |
| 118 | break |
| 119 | elif inp == '' or inp == 'd' or inp == 'defer': |
| 120 | self.owners_queue.append(self.owners_queue.pop(0)) |
| 121 | break |
| 122 | elif inp == 'f' or inp == 'files': |
| 123 | self.list_files() |
| 124 | break |
| 125 | elif inp == 'o' or inp == 'owners': |
| 126 | self.list_owners(self.owners_queue) |
| 127 | break |
| 128 | elif inp == 'p' or inp == 'pick': |
| 129 | self.pick_owner(raw_input('Pick an owner: ')) |
| 130 | break |
| 131 | elif inp.startswith('p ') or inp.startswith('pick '): |
| 132 | self.pick_owner(inp.split(' ', 2)[1].strip()) |
| 133 | break |
| 134 | elif inp == 'r' or inp == 'restart': |
| 135 | self.reset() |
| 136 | break |
| 137 | elif inp == 'q' or inp == 'quit': |
| 138 | # Exit with error |
| 139 | return 1 |
| 140 | |
| 141 | self.print_result() |
| 142 | return 0 |
| 143 | |
| 144 | def _map_owners_to_files(self, files): |
| 145 | for owner in self.all_possible_owners: |
| 146 | for dir_name, _ in self.all_possible_owners[owner]: |
| 147 | for file_name in files: |
| 148 | if file_name.startswith(dir_name): |
| 149 | self.owners_to_files.setdefault(owner, set()) |
| 150 | self.owners_to_files[owner].add(file_name) |
| 151 | |
| 152 | def _map_files_to_owners(self): |
| 153 | for owner in self.owners_to_files: |
| 154 | for file_name in self.owners_to_files[owner]: |
| 155 | self.files_to_owners.setdefault(file_name, set()) |
| 156 | self.files_to_owners[file_name].add(owner) |
| 157 | |
| 158 | def reset(self): |
| 159 | self.files_to_owners = copy.deepcopy(self.original_files_to_owners) |
| 160 | self.unreviewed_files = set(self.files_to_owners.keys()) |
| 161 | self.reviewed_by = {} |
| 162 | self.selected_owners = set() |
| 163 | self.deselected_owners = set() |
| 164 | |
Bruce Dawson | 37740e2 | 2019-11-14 00:27:44 +0000 | [diff] [blame] | 165 | # Randomize owners' names so that if many reviewers have identical scores |
| 166 | # they will be randomly ordered to avoid bias. |
| 167 | owners = list(self.owners_to_files.keys()) |
| 168 | random.shuffle(owners) |
| 169 | self.owners_queue = sorted(owners, |
| 170 | key=lambda owner: self.owners_score[owner]) |
ikarienator@chromium.org | faf3fdf | 2013-09-20 02:11:48 +0000 | [diff] [blame] | 171 | self.find_mandatory_owners() |
| 172 | |
| 173 | def select_owner(self, owner, findMandatoryOwners=True): |
| 174 | if owner in self.selected_owners or owner in self.deselected_owners\ |
| 175 | or not (owner in self.owners_queue): |
| 176 | return |
| 177 | self.writeln('Selected: ' + owner) |
| 178 | self.owners_queue.remove(owner) |
| 179 | self.selected_owners.add(owner) |
| 180 | for file_name in filter( |
| 181 | lambda file_name: file_name in self.unreviewed_files, |
| 182 | self.owners_to_files[owner]): |
| 183 | self.unreviewed_files.remove(file_name) |
| 184 | self.reviewed_by[file_name] = owner |
| 185 | if findMandatoryOwners: |
| 186 | self.find_mandatory_owners() |
| 187 | |
| 188 | def deselect_owner(self, owner, findMandatoryOwners=True): |
| 189 | if owner in self.selected_owners or owner in self.deselected_owners\ |
| 190 | or not (owner in self.owners_queue): |
| 191 | return |
| 192 | self.writeln('Deselected: ' + owner) |
| 193 | self.owners_queue.remove(owner) |
| 194 | self.deselected_owners.add(owner) |
| 195 | for file_name in self.owners_to_files[owner] & self.unreviewed_files: |
| 196 | self.files_to_owners[file_name].remove(owner) |
| 197 | if findMandatoryOwners: |
| 198 | self.find_mandatory_owners() |
| 199 | |
| 200 | def find_mandatory_owners(self): |
| 201 | continues = True |
| 202 | for owner in self.owners_queue: |
| 203 | if owner in self.selected_owners: |
| 204 | continue |
| 205 | if owner in self.deselected_owners: |
| 206 | continue |
| 207 | if len(self.owners_to_files[owner] & self.unreviewed_files) == 0: |
| 208 | self.deselect_owner(owner, False) |
| 209 | |
| 210 | while continues: |
| 211 | continues = False |
| 212 | for file_name in filter( |
| 213 | lambda file_name: len(self.files_to_owners[file_name]) == 1, |
| 214 | self.unreviewed_files): |
| 215 | owner = first(self.files_to_owners[file_name]) |
| 216 | self.select_owner(owner, False) |
| 217 | continues = True |
| 218 | break |
| 219 | |
| 220 | def print_comments(self, owner): |
| 221 | if owner not in self.comments: |
| 222 | self.writeln(self.bold_name(owner)) |
| 223 | else: |
| 224 | self.writeln(self.bold_name(owner) + ' is commented as:') |
| 225 | self.indent() |
Jochen Eisinger | 72606f8 | 2017-04-04 10:44:18 +0200 | [diff] [blame] | 226 | if owners_module.GLOBAL_STATUS in self.comments[owner]: |
| 227 | self.writeln( |
| 228 | self.greyed(self.comments[owner][owners_module.GLOBAL_STATUS]) + |
| 229 | ' (global status)') |
| 230 | if len(self.comments[owner]) == 1: |
| 231 | self.unindent() |
| 232 | return |
ikarienator@chromium.org | faf3fdf | 2013-09-20 02:11:48 +0000 | [diff] [blame] | 233 | for path in self.comments[owner]: |
Jochen Eisinger | 72606f8 | 2017-04-04 10:44:18 +0200 | [diff] [blame] | 234 | if path == owners_module.GLOBAL_STATUS: |
| 235 | continue |
| 236 | elif len(self.comments[owner][path]) > 0: |
ikarienator@chromium.org | faf3fdf | 2013-09-20 02:11:48 +0000 | [diff] [blame] | 237 | self.writeln(self.greyed(self.comments[owner][path]) + |
| 238 | ' (at ' + self.bold(path or '<root>') + ')') |
| 239 | else: |
| 240 | self.writeln(self.greyed('[No comment] ') + ' (at ' + |
| 241 | self.bold(path or '<root>') + ')') |
| 242 | self.unindent() |
| 243 | |
| 244 | def print_file_info(self, file_name, except_owner=''): |
| 245 | if file_name not in self.unreviewed_files: |
| 246 | self.writeln(self.greyed(file_name + |
| 247 | ' (by ' + |
| 248 | self.bold_name(self.reviewed_by[file_name]) + |
| 249 | ')')) |
| 250 | else: |
| 251 | if len(self.files_to_owners[file_name]) <= 3: |
| 252 | other_owners = [] |
| 253 | for ow in self.files_to_owners[file_name]: |
| 254 | if ow != except_owner: |
| 255 | other_owners.append(self.bold_name(ow)) |
| 256 | self.writeln(file_name + |
| 257 | ' [' + (', '.join(other_owners)) + ']') |
| 258 | else: |
| 259 | self.writeln(file_name + ' [' + |
| 260 | self.bold(str(len(self.files_to_owners[file_name]))) + |
| 261 | ']') |
| 262 | |
| 263 | def print_file_info_detailed(self, file_name): |
| 264 | self.writeln(file_name) |
| 265 | self.indent() |
| 266 | for ow in sorted(self.files_to_owners[file_name]): |
| 267 | if ow in self.deselected_owners: |
| 268 | self.writeln(self.bold_name(self.greyed(ow))) |
| 269 | elif ow in self.selected_owners: |
| 270 | self.writeln(self.bold_name(self.greyed(ow))) |
| 271 | else: |
| 272 | self.writeln(self.bold_name(ow)) |
| 273 | self.unindent() |
| 274 | |
| 275 | def print_owned_files_for(self, owner): |
| 276 | # Print owned files |
| 277 | self.print_comments(owner) |
| 278 | self.writeln(self.bold_name(owner) + ' owns ' + |
| 279 | str(len(self.owners_to_files[owner])) + ' file(s):') |
| 280 | self.indent() |
| 281 | for file_name in sorted(self.owners_to_files[owner]): |
| 282 | self.print_file_info(file_name, owner) |
| 283 | self.unindent() |
| 284 | self.writeln() |
| 285 | |
| 286 | def list_owners(self, owners_queue): |
| 287 | if (len(self.owners_to_files) - len(self.deselected_owners) - |
| 288 | len(self.selected_owners)) > 3: |
| 289 | for ow in owners_queue: |
| 290 | if ow not in self.deselected_owners and ow not in self.selected_owners: |
| 291 | self.print_comments(ow) |
| 292 | else: |
| 293 | for ow in owners_queue: |
| 294 | if ow not in self.deselected_owners and ow not in self.selected_owners: |
| 295 | self.writeln() |
| 296 | self.print_owned_files_for(ow) |
| 297 | |
| 298 | def list_files(self): |
| 299 | self.indent() |
| 300 | if len(self.unreviewed_files) > 5: |
| 301 | for file_name in sorted(self.unreviewed_files): |
| 302 | self.print_file_info(file_name) |
| 303 | else: |
| 304 | for file_name in self.unreviewed_files: |
| 305 | self.print_file_info_detailed(file_name) |
| 306 | self.unindent() |
| 307 | |
| 308 | def pick_owner(self, ow): |
| 309 | # Allowing to omit domain suffixes |
| 310 | if ow not in self.owners_to_files: |
| 311 | if ow + self.email_postfix in self.owners_to_files: |
| 312 | ow += self.email_postfix |
| 313 | |
| 314 | if ow not in self.owners_to_files: |
| 315 | self.writeln('You cannot pick ' + self.bold_name(ow) + ' manually. ' + |
| 316 | 'It\'s an invalid name or not related to the change list.') |
| 317 | return False |
| 318 | elif ow in self.selected_owners: |
| 319 | self.writeln('You cannot pick ' + self.bold_name(ow) + ' manually. ' + |
| 320 | 'It\'s already selected.') |
| 321 | return False |
| 322 | elif ow in self.deselected_owners: |
| 323 | self.writeln('You cannot pick ' + self.bold_name(ow) + ' manually.' + |
| 324 | 'It\'s already unselected.') |
| 325 | return False |
| 326 | |
| 327 | self.select_owner(ow) |
| 328 | return True |
| 329 | |
| 330 | def print_result(self): |
| 331 | # Print results |
| 332 | self.writeln() |
| 333 | self.writeln() |
| 334 | self.writeln('** You selected these owners **') |
| 335 | self.writeln() |
| 336 | for owner in self.selected_owners: |
| 337 | self.writeln(self.bold_name(owner) + ':') |
| 338 | self.indent() |
| 339 | for file_name in sorted(self.owners_to_files[owner]): |
| 340 | self.writeln(file_name) |
| 341 | self.unindent() |
| 342 | |
| 343 | def bold(self, text): |
| 344 | return self.COLOR_BOLD + text + self.COLOR_RESET |
| 345 | |
| 346 | def bold_name(self, name): |
| 347 | return (self.COLOR_BOLD + |
| 348 | name.replace(self.email_postfix, '') + self.COLOR_RESET) |
| 349 | |
| 350 | def greyed(self, text): |
| 351 | return self.COLOR_GREY + text + self.COLOR_RESET |
| 352 | |
| 353 | def indent(self): |
| 354 | self.indentation += 1 |
| 355 | |
| 356 | def unindent(self): |
| 357 | self.indentation -= 1 |
| 358 | |
| 359 | def print_indent(self): |
| 360 | return ' ' * self.indentation |
| 361 | |
| 362 | def writeln(self, text=''): |
Raul Tambre | 80ee78e | 2019-05-06 22:41:05 +0000 | [diff] [blame] | 363 | print(self.print_indent() + text) |
ikarienator@chromium.org | faf3fdf | 2013-09-20 02:11:48 +0000 | [diff] [blame] | 364 | |
| 365 | def hr(self): |
| 366 | self.writeln('=====================') |
| 367 | |
| 368 | def print_info(self, owner): |
| 369 | self.hr() |
| 370 | self.writeln( |
| 371 | self.bold(str(len(self.unreviewed_files))) + ' file(s) left.') |
| 372 | self.print_owned_files_for(owner) |
| 373 | |
| 374 | def input_command(self, owner): |
| 375 | self.writeln('Add ' + self.bold_name(owner) + ' as your reviewer? ') |
| 376 | return raw_input( |
| 377 | '[yes/no/Defer/pick/files/owners/quit/restart]: ').lower() |