From: Ali Alnubani <alialnu@oss.nvidia.com>
To: <ci@dpdk.org>
Cc: <thomas@monjalon.net>, <jerinj@marvell.com>,
<ferruh.yigit@intel.com>, <david.marchand@redhat.com>
Subject: [dpdk-ci] [PATCH 5/9] tools: add functionality for setting pw delegates
Date: Mon, 6 Sep 2021 18:45:33 +0300 [thread overview]
Message-ID: <20210906154537.1299-6-alialnu@nvidia.com> (raw)
In-Reply-To: <20210906154537.1299-1-alialnu@nvidia.com>
A new command was added to set patch delegates in Patchwork
based on the emails found in DPDK's MAINTAINERS file.
Example usage:
$ export MAINTAINERS_FILE_PATH=/path/to/dpdk/MAINTAINERS
$ ./maintainers.py --command set_pw_delegate --type series SERIES_ID
Signed-off-by: Ali Alnubani <alialnu@nvidia.com>
---
tools/maintainers.py | 46 +++++++++++++++++++++++++++++++++++++++++---
1 file changed, 43 insertions(+), 3 deletions(-)
diff --git a/tools/maintainers.py b/tools/maintainers.py
index c9b0502..a1ba5fc 100755
--- a/tools/maintainers.py
+++ b/tools/maintainers.py
@@ -14,6 +14,7 @@ from requests.exceptions import HTTPError
from git_pw import config
from git_pw import api
from git_pw import utils
+from git_pw import patch as git_pw_patch
"""
Description:
@@ -84,6 +85,29 @@ class GitPW(object):
else:
raise
+ def set_delegate(self, patch_list, delegate):
+ """Set the delegate for a patch.
+ Only tries to set a delegate for patches that don't have one set already.
+
+ Reference:
+ https://github.com/getpatchwork/git-pw/blob/76b79097dc0a57c89b45dd53d9cacb7ff7b31bb2/git_pw/patch.py#L167
+ """
+ users = api.index('users', [('q', delegate)])
+ if len(users) != 1:
+ # Zero or multiple users found
+ print('Cannot choose a Patchwork user to delegate to from '
+ 'user list ({}). Skipping..'.format(users))
+ return
+ for patch in patch_list:
+ if patch['delegate']:
+ print('Patch {} is already delegated to {}. '
+ 'Skipping..'.format(patch['id'], patch['delegate']['email']))
+ continue
+ print("Delegating patch {} to {}.".format(
+ patch['id'], users[0]['email']))
+ _ = api.update(
+ 'patches', patch['id'], [('delegate', users[0]['id'])])
+
class Diff(object):
@@ -267,7 +291,8 @@ if __name__ == '__main__':
'--command',
choices=(
'list_trees',
- 'list_maintainers'),
+ 'list_maintainers',
+ 'set_pw_delegate'),
required=True, help='Command to perform')
options_parser.add_argument(
'--type',
@@ -325,5 +350,20 @@ if __name__ == '__main__':
if command == 'list_trees':
print(tree.split('/')[-1])
- elif command == 'list_maintainers':
- print(*maintainers.get_maintainers(tree), sep='\n')
+ if command in ['list_maintainers', 'set_pw_delegate']:
+ maintainer_list = maintainers.get_maintainers(tree)
+ if command == 'list_maintainers':
+ print(*maintainer_list, sep='\n')
+ elif command == 'set_pw_delegate':
+ if len(maintainer_list) > 0:
+ # Get the email of the first maintainer in the list.
+ try:
+ delegate = re.match(
+ r".*\<(?P<email>.*)\>",
+ maintainer_list[0]).group('email')
+ except AttributeError:
+ print("Unexpected format: '{}'".format(maintainer_list[0]))
+ sys.exit(1)
+ _git_pw.set_delegate(patch_list, delegate)
+ else:
+ print('No maintainers found. Not setting a delegate.')
--
2.25.1
next prev parent reply other threads:[~2021-09-06 15:46 UTC|newest]
Thread overview: 57+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-09-06 15:45 [dpdk-ci] [PATCH 0/9] Automatic patchwork delegation Ali Alnubani
2021-09-06 15:45 ` [dpdk-ci] [PATCH 1/9] tools: rename guess_git_tree script Ali Alnubani
2021-09-07 11:03 ` Juraj Linkeš
2021-09-08 16:54 ` Ali Alnubani
2021-09-06 15:45 ` [dpdk-ci] [PATCH 2/9] tools: match by tree url instead of tree name Ali Alnubani
2021-09-06 15:45 ` [dpdk-ci] [PATCH 3/9] tools: update script usage Ali Alnubani
2021-09-06 15:45 ` [dpdk-ci] [PATCH 4/9] tools: add functionality for detecting tree maintainers Ali Alnubani
2021-09-06 15:45 ` Ali Alnubani [this message]
2021-09-06 15:45 ` [dpdk-ci] [PATCH 6/9] add git-pw to requirements file Ali Alnubani
2021-09-06 15:45 ` [dpdk-ci] [PATCH 7/9] tools: filter new patchwork IDs by date Ali Alnubani
2021-09-06 15:58 ` Ali Alnubani
2021-09-06 15:45 ` [dpdk-ci] [PATCH 8/9] tools: add support for fetching new series IDs Ali Alnubani
2021-09-06 15:45 ` [dpdk-ci] [PATCH 9/9] tools: filter new patchwork IDs by project name Ali Alnubani
2021-09-21 14:35 ` [dpdk-ci] [PATCH v2 00/10] Automatic patchwork delegation alialnu
2021-09-21 14:35 ` [dpdk-ci] [PATCH v2 01/10] tools: rename guess_git_tree script alialnu
2021-09-21 14:35 ` [dpdk-ci] [PATCH v2 02/10] tools: match by tree url instead of tree name alialnu
2021-09-30 8:00 ` Thomas Monjalon
2021-10-18 7:48 ` Ali Alnubani
2021-09-21 14:35 ` [dpdk-ci] [PATCH v2 03/10] tools: update script usage alialnu
2021-09-30 8:09 ` Thomas Monjalon
2021-09-21 14:35 ` [dpdk-ci] [PATCH v2 04/10] tools: add functionality for detecting tree maintainers alialnu
2021-09-30 8:29 ` Thomas Monjalon
2021-09-21 14:35 ` [dpdk-ci] [PATCH v2 05/10] tools: add functionality for setting pw delegates alialnu
2021-09-30 9:15 ` Thomas Monjalon
2021-10-18 7:48 ` Ali Alnubani
2021-10-26 14:08 ` Thomas Monjalon
2021-11-04 16:48 ` Ali Alnubani
2021-11-04 18:16 ` Thomas Monjalon
2021-11-08 7:45 ` Ali Alnubani
2021-09-21 14:35 ` [dpdk-ci] [PATCH v2 06/10] add git-pw to requirements file alialnu
2021-09-21 14:35 ` [dpdk-ci] [PATCH v2 07/10] tools: filter new patchwork IDs by date alialnu
2021-10-11 20:08 ` Ali Alnubani
2021-09-21 14:35 ` [dpdk-ci] [PATCH v2 08/10] tools: add support for fetching new series IDs alialnu
2021-09-30 10:25 ` Thomas Monjalon
2021-09-21 14:35 ` [dpdk-ci] [PATCH v2 09/10] tools: filter new patchwork IDs by project name alialnu
2021-09-30 10:28 ` Thomas Monjalon
2021-09-21 14:35 ` [dpdk-ci] [PATCH v2 10/10] tools: skip the IDs we already fetched alialnu
2021-09-30 10:32 ` Thomas Monjalon
2021-10-11 19:30 ` Ali Alnubani
2021-10-12 6:44 ` Thomas Monjalon
2021-10-18 8:04 ` Ali Alnubani
2021-10-26 14:07 ` Thomas Monjalon
2021-11-04 16:53 ` Ali Alnubani
2021-11-04 18:08 ` Thomas Monjalon
2021-11-08 7:44 ` Ali Alnubani
2021-11-08 6:28 ` [dpdk-ci] [PATCH v4 00/10] Automatic patchwork delegation Ali Alnubani
2021-11-08 6:28 ` [dpdk-ci] [PATCH v4 01/10] tools: rename guess_git_tree script Ali Alnubani
2021-11-08 6:28 ` [dpdk-ci] [PATCH v4 02/10] tools: match by tree URL instead of tree name Ali Alnubani
2021-11-08 6:28 ` [dpdk-ci] [PATCH v4 03/10] tools: update script usage Ali Alnubani
2021-11-08 6:28 ` [dpdk-ci] [PATCH v4 04/10] tools: add functionality for detecting tree maintainers Ali Alnubani
2021-11-08 6:28 ` [dpdk-ci] [PATCH v4 05/10] tools: add functionality for setting pw delegates Ali Alnubani
2021-11-08 6:28 ` [dpdk-ci] [PATCH v4 06/10] add git-pw to requirements file Ali Alnubani
2021-11-08 6:28 ` [dpdk-ci] [PATCH v4 07/10] tools: filter new Patchwork IDs by date Ali Alnubani
2021-11-08 6:28 ` [dpdk-ci] [PATCH v4 08/10] tools: support fetching series Ali Alnubani
2021-11-08 6:28 ` [dpdk-ci] [PATCH v4 09/10] tools: filter new patchwork IDs by project name Ali Alnubani
2021-11-08 6:28 ` [dpdk-ci] [PATCH v4 10/10] tools: skip the IDs we already fetched Ali Alnubani
2022-01-05 1:05 ` [PATCH v4 00/10] Automatic patchwork delegation Thomas Monjalon
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20210906154537.1299-6-alialnu@nvidia.com \
--to=alialnu@oss.nvidia.com \
--cc=ci@dpdk.org \
--cc=david.marchand@redhat.com \
--cc=ferruh.yigit@intel.com \
--cc=jerinj@marvell.com \
--cc=thomas@monjalon.net \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).