blob: a8a3bfd714cf2d7e92f5586ca531e4a9115a1221 [file] [log] [blame]
dpranke@chromium.org2a009622011-03-01 02:43:31 +00001# Copyright (c) 2010 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"""A database of OWNERS files."""
6
dpranke@chromium.orgfdecfb72011-03-16 23:27:23 +00007import collections
dpranke@chromium.org6dada4e2011-03-08 22:32:40 +00008import re
9
10
11# If this is present by itself on a line, this means that everyone can review.
12EVERYONE = '*'
13
14
15# Recognizes 'X@Y' email addresses. Very simplistic.
16BASIC_EMAIL_REGEXP = r'^[\w\-\+\%\.]+\@[\w\-\+\%\.]+$'
dpranke@chromium.org2a009622011-03-01 02:43:31 +000017
dpranke@chromium.org2a009622011-03-01 02:43:31 +000018
dpranke@chromium.org898a10e2011-03-04 21:54:43 +000019class SyntaxErrorInOwnersFile(Exception):
dpranke@chromium.org86bbf192011-03-09 21:37:06 +000020 def __init__(self, path, lineno, msg):
21 super(SyntaxErrorInOwnersFile, self).__init__((path, lineno, msg))
dpranke@chromium.org898a10e2011-03-04 21:54:43 +000022 self.path = path
dpranke@chromium.org86bbf192011-03-09 21:37:06 +000023 self.lineno = lineno
dpranke@chromium.org898a10e2011-03-04 21:54:43 +000024 self.msg = msg
25
26 def __str__(self):
dpranke@chromium.org86bbf192011-03-09 21:37:06 +000027 return "%s:%d syntax error: %s" % (self.path, self.lineno, self.msg)
dpranke@chromium.org898a10e2011-03-04 21:54:43 +000028
29
dpranke@chromium.org898a10e2011-03-04 21:54:43 +000030class Database(object):
31 """A database of OWNERS files for a repository.
32
33 This class allows you to find a suggested set of reviewers for a list
34 of changed files, and see if a list of changed files is covered by a
35 list of reviewers."""
36
37 def __init__(self, root, fopen, os_path):
38 """Args:
dpranke@chromium.org2a009622011-03-01 02:43:31 +000039 root: the path to the root of the Repository
dpranke@chromium.org2a009622011-03-01 02:43:31 +000040 open: function callback to open a text file for reading
dpranke@chromium.org6dada4e2011-03-08 22:32:40 +000041 os_path: module/object callback with fields for 'abspath', 'dirname',
42 'exists', and 'join'
dpranke@chromium.org2a009622011-03-01 02:43:31 +000043 """
44 self.root = root
45 self.fopen = fopen
46 self.os_path = os_path
47
dpranke@chromium.org627ea672011-03-11 23:29:03 +000048 # Pick a default email regexp to use; callers can override as desired.
dpranke@chromium.org6dada4e2011-03-08 22:32:40 +000049 self.email_regexp = re.compile(BASIC_EMAIL_REGEXP)
dpranke@chromium.org2a009622011-03-01 02:43:31 +000050
dpranke@chromium.org6dada4e2011-03-08 22:32:40 +000051 # Mapping of owners to the paths they own.
52 self.owned_by = {EVERYONE: set()}
53
54 # Mapping of paths to authorized owners.
dpranke@chromium.org2a009622011-03-01 02:43:31 +000055 self.owners_for = {}
56
dpranke@chromium.org6dada4e2011-03-08 22:32:40 +000057 # Set of paths that stop us from looking above them for owners.
58 # (This is implicitly true for the root directory).
59 self.stop_looking = set([''])
dpranke@chromium.org2a009622011-03-01 02:43:31 +000060
dpranke@chromium.org7eea2592011-03-09 21:35:46 +000061 def reviewers_for(self, files):
dpranke@chromium.orgfdecfb72011-03-16 23:27:23 +000062 """Returns a suggested set of reviewers that will cover the files.
dpranke@chromium.org2a009622011-03-01 02:43:31 +000063
dpranke@chromium.orgfdecfb72011-03-16 23:27:23 +000064 files is a sequence of paths relative to (and under) self.root."""
dpranke@chromium.org7eea2592011-03-09 21:35:46 +000065 self._check_paths(files)
66 self._load_data_needed_for(files)
67 return self._covering_set_of_owners_for(files)
dpranke@chromium.org2a009622011-03-01 02:43:31 +000068
dpranke@chromium.org7eea2592011-03-09 21:35:46 +000069 def files_are_covered_by(self, files, reviewers):
dpranke@chromium.org6dada4e2011-03-08 22:32:40 +000070 """Returns whether every file is owned by at least one reviewer."""
dpranke@chromium.org7eea2592011-03-09 21:35:46 +000071 return not self.files_not_covered_by(files, reviewers)
dpranke@chromium.org2a009622011-03-01 02:43:31 +000072
dpranke@chromium.org7eea2592011-03-09 21:35:46 +000073 def files_not_covered_by(self, files, reviewers):
dpranke@chromium.orgfdecfb72011-03-16 23:27:23 +000074 """Returns the set of files that are not owned by at least one reviewer.
75
76 Args:
77 files is a sequence of paths relative to (and under) self.root.
78 reviewers is a sequence of strings matching self.email_regexp."""
dpranke@chromium.org7eea2592011-03-09 21:35:46 +000079 self._check_paths(files)
80 self._check_reviewers(reviewers)
dpranke@chromium.org6dada4e2011-03-08 22:32:40 +000081 if not reviewers:
82 return files
83
dpranke@chromium.org7eea2592011-03-09 21:35:46 +000084 self._load_data_needed_for(files)
85 files_by_dir = self._files_by_dir(files)
86 covered_dirs = self._dirs_covered_by(reviewers)
dpranke@chromium.org6dada4e2011-03-08 22:32:40 +000087 uncovered_files = []
88 for d, files_in_d in files_by_dir.iteritems():
dpranke@chromium.org7eea2592011-03-09 21:35:46 +000089 if not self._is_dir_covered_by(d, covered_dirs):
dpranke@chromium.org6dada4e2011-03-08 22:32:40 +000090 uncovered_files.extend(files_in_d)
91 return set(uncovered_files)
92
dpranke@chromium.orgfdecfb72011-03-16 23:27:23 +000093 def _check_collection(self, obj):
94 assert (isinstance(obj, collections.Iterable) and
95 isinstance(obj, collections.Sized) and
96 not isinstance(obj, basestring))
97
dpranke@chromium.org7eea2592011-03-09 21:35:46 +000098 def _check_paths(self, files):
99 def _is_under(f, pfx):
dpranke@chromium.org6dada4e2011-03-08 22:32:40 +0000100 return self.os_path.abspath(self.os_path.join(pfx, f)).startswith(pfx)
dpranke@chromium.orgfdecfb72011-03-16 23:27:23 +0000101 self._check_collection(files)
dpranke@chromium.org7eea2592011-03-09 21:35:46 +0000102 assert all(_is_under(f, self.os_path.abspath(self.root)) for f in files)
dpranke@chromium.org6dada4e2011-03-08 22:32:40 +0000103
dpranke@chromium.org7eea2592011-03-09 21:35:46 +0000104 def _check_reviewers(self, reviewers):
dpranke@chromium.orgfdecfb72011-03-16 23:27:23 +0000105 self._check_collection(reviewers)
dpranke@chromium.org6dada4e2011-03-08 22:32:40 +0000106 assert all(self.email_regexp.match(r) for r in reviewers)
107
dpranke@chromium.org7eea2592011-03-09 21:35:46 +0000108 def _files_by_dir(self, files):
dpranke@chromium.org6dada4e2011-03-08 22:32:40 +0000109 dirs = {}
110 for f in files:
111 dirs.setdefault(self.os_path.dirname(f), []).append(f)
112 return dirs
113
dpranke@chromium.org7eea2592011-03-09 21:35:46 +0000114 def _dirs_covered_by(self, reviewers):
dpranke@chromium.org6dada4e2011-03-08 22:32:40 +0000115 dirs = self.owned_by[EVERYONE]
116 for r in reviewers:
117 dirs = dirs | self.owned_by.get(r, set())
118 return dirs
119
dpranke@chromium.org7eea2592011-03-09 21:35:46 +0000120 def _stop_looking(self, dirname):
dpranke@chromium.org6dada4e2011-03-08 22:32:40 +0000121 return dirname in self.stop_looking
122
dpranke@chromium.org7eea2592011-03-09 21:35:46 +0000123 def _is_dir_covered_by(self, dirname, covered_dirs):
124 while not dirname in covered_dirs and not self._stop_looking(dirname):
dpranke@chromium.org6dada4e2011-03-08 22:32:40 +0000125 dirname = self.os_path.dirname(dirname)
126 return dirname in covered_dirs
dpranke@chromium.org2a009622011-03-01 02:43:31 +0000127
dpranke@chromium.org7eea2592011-03-09 21:35:46 +0000128 def _load_data_needed_for(self, files):
dpranke@chromium.org2a009622011-03-01 02:43:31 +0000129 for f in files:
dpranke@chromium.org6dada4e2011-03-08 22:32:40 +0000130 dirpath = self.os_path.dirname(f)
131 while not dirpath in self.owners_for:
dpranke@chromium.org7eea2592011-03-09 21:35:46 +0000132 self._read_owners_in_dir(dirpath)
133 if self._stop_looking(dirpath):
dpranke@chromium.org6dada4e2011-03-08 22:32:40 +0000134 break
135 dirpath = self.os_path.dirname(dirpath)
dpranke@chromium.org2a009622011-03-01 02:43:31 +0000136
dpranke@chromium.org7eea2592011-03-09 21:35:46 +0000137 def _read_owners_in_dir(self, dirpath):
dpranke@chromium.org6dada4e2011-03-08 22:32:40 +0000138 owners_path = self.os_path.join(self.root, dirpath, 'OWNERS')
139 if not self.os_path.exists(owners_path):
140 return
dpranke@chromium.org2a009622011-03-01 02:43:31 +0000141
dpranke@chromium.org6dada4e2011-03-08 22:32:40 +0000142 lineno = 0
143 for line in self.fopen(owners_path):
144 lineno += 1
145 line = line.strip()
146 if line.startswith('#'):
147 continue
148 if line == 'set noparent':
149 self.stop_looking.add(dirpath)
150 continue
dpranke@chromium.org86bbf192011-03-09 21:37:06 +0000151 if line.startswith('set '):
152 raise SyntaxErrorInOwnersFile(owners_path, lineno,
153 'unknown option: "%s"' % line[4:].strip())
dpranke@chromium.org6dada4e2011-03-08 22:32:40 +0000154 if self.email_regexp.match(line) or line == EVERYONE:
155 self.owned_by.setdefault(line, set()).add(dirpath)
156 self.owners_for.setdefault(dirpath, set()).add(line)
157 continue
dpranke@chromium.org86bbf192011-03-09 21:37:06 +0000158 raise SyntaxErrorInOwnersFile(owners_path, lineno,
159 ('line is not a comment, a "set" directive, '
160 'or an email address: "%s"' % line))
dpranke@chromium.org2a009622011-03-01 02:43:31 +0000161
dpranke@chromium.org7eea2592011-03-09 21:35:46 +0000162 def _covering_set_of_owners_for(self, files):
dpranke@chromium.org2a009622011-03-01 02:43:31 +0000163 # TODO(dpranke): implement the greedy algorithm for covering sets, and
164 # consider returning multiple options in case there are several equally
165 # short combinations of owners.
166 every_owner = set()
167 for f in files:
dpranke@chromium.org6dada4e2011-03-08 22:32:40 +0000168 dirname = self.os_path.dirname(f)
169 while dirname in self.owners_for:
170 every_owner |= self.owners_for[dirname]
dpranke@chromium.org7eea2592011-03-09 21:35:46 +0000171 if self._stop_looking(dirname):
dpranke@chromium.org6dada4e2011-03-08 22:32:40 +0000172 break
173 dirname = self.os_path.dirname(dirname)
dpranke@chromium.org2a009622011-03-01 02:43:31 +0000174 return every_owner