[dpdk-ci] [PATCH v2 04/10] tools: add functionality for detecting tree maintainers
alialnu at oss.nvidia.com
alialnu at oss.nvidia.com
Tue Sep 21 16:35:36 CEST 2021
From: Ali Alnubani <alialnu at nvidia.com>
Detecting a maintainer works by searching the
'General Project Administration' section for subsections
containing the provided tree, and then returning the maintainers
specified in that subsection.
Signed-off-by: Ali Alnubani <alialnu at nvidia.com>
---
tools/pw_maintainers_cli.py | 27 ++++++++++++++++++++++++++-
1 file changed, 26 insertions(+), 1 deletion(-)
diff --git a/tools/pw_maintainers_cli.py b/tools/pw_maintainers_cli.py
index d4c0056..9b77e77 100755
--- a/tools/pw_maintainers_cli.py
+++ b/tools/pw_maintainers_cli.py
@@ -33,6 +33,7 @@ to load the git configurations pw.{server,project,token}.
Example usage:
./pw_maintainers_cli.py --type series list_trees 2054
./pw_maintainers_cli.py --type patch list_trees 2054
+ ./pw_maintainers_cli.py --type patch list_maintainers 2054
Or if you want to use inside other scripts:
@@ -48,6 +49,7 @@ Or if you want to use inside other scripts:
files = Diff.find_filenames(_git_pw.api_get('patches', patch_id)['diff'])
tree_url = maintainers.get_tree(files)
tree_name = tree_url.split('/')[-1]
+ maintainers = maintainers.get_maintainers(tree_url)
"""
@@ -118,6 +120,7 @@ class Maintainers(object):
file_regex = r'F:\s(.*)'
tree_regex = r'T: (?P<url>git:\/\/dpdk\.org(?:\/next)*\/(?P<name>.*))'
+ maintainer_regex = r'M:\s(.*)'
section_regex = r'([^\n]*)\n-+.*?(?=([^\n]*\n-+)|\Z)'
subsection_regex = r'[^\n](?:(?!\n{{2}}).)*?^{}: {}$(?:(?!\n{{2}}).)*'
@@ -141,6 +144,26 @@ class Maintainers(object):
# Save already matched patterns.
self.matched = {}
+ def get_maintainers(self, tree):
+ """
+ Return a list of a tree's maintainers."""
+ maintainers = []
+ for section in self.sections:
+ if section.group(1) == 'General Project Administration':
+ # Find the block containing the tree.
+ regex = self.subsection_regex.format('T', re.escape(tree))
+ subsection_match = re.findall(
+ regex,
+ section.group(0),
+ re.DOTALL | re.MULTILINE)
+ if len(subsection_match):
+ subsection = subsection_match[-1]
+ # Look for maintainers
+ maintainers = re.findall(
+ self.maintainer_regex, subsection)
+ return maintainers
+ break
+
def get_tree(self, files):
"""
Return a git tree that matches a list of files."""
@@ -265,7 +288,7 @@ if __name__ == '__main__':
parser.add_argument(
'command',
choices=[
- 'list_trees'],
+ 'list_trees', 'list_maintainers'],
help='Command to perform')
parser.add_argument(
'id', type=int, help='patch/series id')
@@ -301,3 +324,5 @@ if __name__ == '__main__':
if command == 'list_trees':
print(tree.split('/')[-1])
+ elif command == 'list_maintainers':
+ print(*maintainers.get_maintainers(tree), sep='\n')
--
2.25.1
More information about the ci
mailing list