From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id 47B9DA0C43 for ; Mon, 18 Oct 2021 09:48:10 +0200 (CEST) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 38EB540141; Mon, 18 Oct 2021 09:48:10 +0200 (CEST) Received: from AZHDRRW-EX02.NVIDIA.COM (azhdrrw-ex02.nvidia.com [20.64.145.131]) by mails.dpdk.org (Postfix) with ESMTP id 2C33D40041 for ; Mon, 18 Oct 2021 09:48:09 +0200 (CEST) Received: from NAM02-SN1-obe.outbound.protection.outlook.com (104.47.57.43) by mxs.oss.nvidia.com (10.13.234.37) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.2.858.15; Mon, 18 Oct 2021 00:48:08 -0700 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=fdc28l9scBvZQQz0woRLhh6avGfqw5qEZX2XBcWchcnsaCUVWBbkjVxSNohyaxOVyzr5Nybl+FTuZlXIujl1Ce0SEYy2LlZbph7y3Y8wAaQVznYdVdqmdEaSxRcVmdCuYwanOyptOlH4vD42Ny2GbA8j8Zbez+YHSDyU9qydJeesXjMvpjLbWt5HdX5L8aQcBOnXEOYTUWnur5s5shCKLaghEoLOWsqvNUu7bnYANihVTPMJfOVyqAcjwkKawckdUjUyz073fDb+8yVQfscnVGwnN1kA2eFa7StFYTurKRwFwfDtN1oBD9nd6RGpkZ6iY3NMfUkHOvGrKjj3gq+3oQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=0BVsT3IkejRvORh6mV2C/uhdW1e17DxXD5QIAxbJvHY=; b=ezlgj68P5tPjAUTUwsn3+uczQrNF4EMEb3IIk+WoydE1lHWI0xZvg9DAwyAQta3dyqxD/v4LRmEuGBgC9en+oS15FR/SGILstCYUaX+qJynZQqTkPKaQWaBEsC6WHMaFmWt40wrdH1WD5ntv3vgaTXDfE2o4wWtWOSpbusBGiunHCEiTrytJLmnhTq92MQzSu7bBXHry2jeDwkGEnzQYDYofsfcGkLBqA7qHYeUsXK2V84kXLEQhcwng1Nu05RLmQdxpNTb4QInUFcKqKAPSq4MbMPkQMhEMx/9xAGY1s0cDYcu/Y/h9qau3gNYwss41w8OLnYe6b5Vh5tQEQLuIrw== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=nvidia.com; dmarc=pass action=none header.from=nvidia.com; dkim=pass header.d=nvidia.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=Nvidia.com; s=selector2; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=0BVsT3IkejRvORh6mV2C/uhdW1e17DxXD5QIAxbJvHY=; b=V1ZwJFr9ktbhCuDpTtav2XjL+B5dIF3sa4jLmEssPpHIfLEPdh0ZcGvc+NMsMDb7fiRefnEcQT6aPJdGKd00cg0rQXTOHoyOv6FonhPm2YAvLTtr8QCMcCRHX15daai7VZlWGD1kVdSmdFo+/gaFeCacH3WxCc4ojqRINwt4bPDktR5JJEIVwJfK6vUVHAIA+T7dljwLeeis1uD5jX3A5gMi4rcZImraxzpyZHBzkz4t+GkfKnXsI5/Zfk+vBdyOoqkY++uXPDfd8oj/uQ/ckyTfTGz2TDLAgfYwA/jkzgkI77PDLC5Gr5TKI1N1em1Tpu6tAIUsxgG73JepVjf2pA== Received: from DM4PR12MB5167.namprd12.prod.outlook.com (2603:10b6:5:396::10) by DM4PR12MB5087.namprd12.prod.outlook.com (2603:10b6:5:38a::17) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4608.16; Mon, 18 Oct 2021 07:48:05 +0000 Received: from DM4PR12MB5167.namprd12.prod.outlook.com ([fe80::3c73:8e07:c9ba:f6db]) by DM4PR12MB5167.namprd12.prod.outlook.com ([fe80::3c73:8e07:c9ba:f6db%3]) with mapi id 15.20.4608.018; Mon, 18 Oct 2021 07:48:05 +0000 From: Ali Alnubani To: NBU-Contact-Thomas Monjalon CC: "ci@dpdk.org" , "jerinj@marvell.com" , "ferruh.yigit@intel.com" , "david.marchand@redhat.com" , "juraj.linkes@pantheon.tech" Thread-Topic: [PATCH v2 05/10] tools: add functionality for setting pw delegates Thread-Index: AQHXtdu5qAm+7mlJpkCgH71Ih8ojTavYfQEg Date: Mon, 18 Oct 2021 07:48:05 +0000 Message-ID: References: <20210906154537.1299-1-alialnu@nvidia.com> <20210921143542.4412-1-alialnu@nvidia.com> <20210921143542.4412-6-alialnu@nvidia.com> <24744594.yUaU8azXQc@thomas> In-Reply-To: <24744594.yUaU8azXQc@thomas> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: authentication-results: monjalon.net; dkim=none (message not signed) header.d=none;monjalon.net; dmarc=none action=none header.from=nvidia.com; x-ms-publictraffictype: Email x-ms-office365-filtering-correlation-id: 38445e45-0b76-4683-dc47-08d9920b9eb9 x-ms-traffictypediagnostic: DM4PR12MB5087: x-ld-processed: 43083d15-7273-40c1-b7db-39efd9ccc17a,ExtAddr x-microsoft-antispam-prvs: x-ms-exchange-transport-forked: True x-ms-oob-tlc-oobclassifiers: OLM:8882; x-ms-exchange-senderadcheck: 1 x-ms-exchange-antispam-relay: 0 x-microsoft-antispam: BCL:0; x-microsoft-antispam-message-info: KUHdzU01M13rip3ydLU1tT1EYeb7ffe9tiuRXyHXqDUc1b9h7BXB700bbO++qnweTETKT5VWRU5iCSLR8Ut6/62uMO2VsFqZ1MvgF8kdpWZ8MZmQbvt2ZoqgFxeiAXzz33oKjG5EsRUagEUvHf8i8/CU00/Xmqzo4dPGuR2eFbgZ0+CA6I/nbYIHCvOpXKxcSNYwIBkwfzo9uO4EhobyNcULyrxs4hZydpj8JGwRRLDmp1PQ1mT1BaHcrJte4On7gg3K5kkJDIODTIzwHYQUuOe5sHE/SuBYEZGH99C1E7c0lJ/lacBD6mKagKgMof1OMmu6nij3ut1oMFn0luNxNSaJlH1kUOsv0p3KWS+DO/5wi8S2rzEagQIoQ/aEkCpuehJogqJe6603+HhrFbNoYADRhjUbSXtEySIUdiZ4uNnK6GxMDShWQ2iq0+qeFEcKTXiYf5jIPuL1b0OIRO3mq+3dnLKWLRicnsDuDw6fZGPbrimCRjmQOkvt40vIqDJfYrBkATSof4vcwR5+9aL+g3qO/DeycWfIELoDYvHDjXbjAh8QHuxlDwaDLk14uyMNWCb/T53BDUxniCS0q82C4P971dcp9dyTXV1/IML/ohamS+1HN/1pmt1jhqG48ewSP3FPeSPIA/3bD/fQb21oes1Ax07JczUvekKZcuRQwXwbkYYQP2TWp14SQH+U+stiUOxb1EPU8Q1GcuXdEydFmCmRzFNlpcxH2apTOYvalQ4Z4Ke8LWRlMpA5tiL8Z+hoH0geBln97gV9l2qa6T9+UY/oRKd0ncOD1Lxrg0pgV3E= x-forefront-antispam-report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:DM4PR12MB5167.namprd12.prod.outlook.com; PTR:; CAT:NONE; SFS:(4636009)(366004)(122000001)(55016002)(316002)(38100700002)(9686003)(83380400001)(6862004)(8936002)(76116006)(38070700005)(107886003)(66446008)(64756008)(66556008)(66476007)(66946007)(8676002)(4326008)(52536014)(5660300002)(966005)(508600001)(26005)(33656002)(86362001)(54906003)(6506007)(186003)(2906002)(7696005)(53546011)(71200400001); DIR:OUT; SFP:1101; x-ms-exchange-antispam-messagedata-chunkcount: 1 x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?J3McHEKDaOeD1jLLwJsTzf1xNYjkBjzc8C+1+mAlcLpGSxHGymdGOk8UbP5/?= =?us-ascii?Q?Bo+VbcmEH3bd5bbxDnjILeiKJex+ZSdhlWnGbW6iu/k4vJ52/g7F8QS+TRli?= =?us-ascii?Q?VP6EakxU5tq95L0Nylql7YTHoMudo8NyRXknrP6gTFjl8UjOkl+3kc3hr4v7?= =?us-ascii?Q?xcJ3psWYxwcMxdQP6DGiCOVOJSsxbDzaj+li7UUdTvJqcTiRggC7kuVyxa6q?= =?us-ascii?Q?MzsR74IEsTfPhETkppmkELiOD1kGpIjzlTQmFN2Vl57yteMLHssz+kIsQvY/?= =?us-ascii?Q?lgeZpVU98X5b71CmmJz0D1ODP306QQVWtRJhEgFseZJVl/+sjFZZpMjj0cot?= =?us-ascii?Q?zZk+QsIIrh6XbssSTm3yeYRBjUXa70t0lNAha1QEoCwnl1fglDn4JZeQ5GoO?= =?us-ascii?Q?YlAvBwoeSMq0x8B/HOGxG/qt6vSyxsgr0zHXhcLOaT5s/QkwhEopeVJKZgUB?= =?us-ascii?Q?CYkIedgz9NVhs3WCiGmSY/4EyLKUtBG5tBcSwcQgonOP6bO4or8uYkm1dUS1?= =?us-ascii?Q?ykbpMjbXQvt3LI1QwtpzapuE/13NFpQlD/zKkg6IjoZccagBFDwVsn0ifW6V?= =?us-ascii?Q?z8h69zuIIBiTHJ4KRBKV0GnT1ra0NV0gST0EYFyeb5gNWe13iXOmUemBJvkB?= =?us-ascii?Q?+Fc+imma/8W1/rz0biR6uVMusFZWl6WDw5zA74cG0efIvptSau5tBSUf5W/H?= =?us-ascii?Q?7gQXSYyVC6fBD1y5CkqI3LtWxp76cIqXnefhKtEJuLCf1jfFI02+fJjz4F+h?= =?us-ascii?Q?fOSF0HrEu68LL0ZxuFrCG9zd0j3WCCvY9r7k5bAdfuOnLewWR4sSevvSNUSZ?= =?us-ascii?Q?F3jDltlEHqg1kbgtgBfx7HG8nPH4MCEqvqdBt1YLfO5OSzrJy0feT7zajctn?= =?us-ascii?Q?wB0Yl/5BALomRwzc6FitwopBwvnDigYV8h6cLdb1Bo2/qYOQSWjuvXVmeGwR?= =?us-ascii?Q?BC6zG153zFkQxU8HOL3SnXCjUaV3AJtgwclZ8wHOnqGSyLWTmS9tV59VK2ng?= =?us-ascii?Q?NapSu6cDef8L9rmiQs7yD9Itq8pgzQl29l3D9KiAGbC0kmQP/EzxGau6flaD?= =?us-ascii?Q?lwR/yh6PYuKXfyOGueD0O/m1OnahJ7l8dxq+RzD1VP6TXdQ8DBn3VmDjlz19?= =?us-ascii?Q?rBuQwDbsjwb/iQEW23ul0/2vEvFGk5oxjwAFBLlWUkW2TysXEPssszf616Kr?= =?us-ascii?Q?PaNk5QkRRPGQUMZbmOWzeHPUJy0zdT6PLQfpf+WBufXqwjn2c4gYld4JPJax?= =?us-ascii?Q?Q/8YQzBhzBD4gh5Tc1kBH8rdpQYi7qRfFWPotiDcEuhaK8q39XA3Ahw9JwsC?= =?us-ascii?Q?G2nh4/G8oTdEgohrN9T97W45?= Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-AuthSource: DM4PR12MB5167.namprd12.prod.outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: 38445e45-0b76-4683-dc47-08d9920b9eb9 X-MS-Exchange-CrossTenant-originalarrivaltime: 18 Oct 2021 07:48:05.3737 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: 43083d15-7273-40c1-b7db-39efd9ccc17a X-MS-Exchange-CrossTenant-mailboxtype: HOSTED X-MS-Exchange-CrossTenant-userprincipalname: WeFrRAlJPbQ18ZGZIb81SysTFveuXCutUWwkDeL0wBmWzbzvomk4nD1U2a4fJuRWw5TFiRnZkAePL+LqGpqMfw== X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM4PR12MB5087 Subject: Re: [dpdk-ci] [PATCH v2 05/10] tools: add functionality for setting pw delegates X-BeenThere: ci@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK CI discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: ci-bounces@dpdk.org Sender: "ci" > -----Original Message----- > From: Thomas Monjalon > Sent: Thursday, September 30, 2021 12:15 PM > To: Ali Alnubani > Cc: ci@dpdk.org; jerinj@marvell.com; ferruh.yigit@intel.com; > david.marchand@redhat.com; juraj.linkes@pantheon.tech > Subject: Re: [PATCH v2 05/10] tools: add functionality for setting pw > delegates >=20 > 21/09/2021 16:35, alialnu@nvidia.com: > > From: Ali Alnubani > > > > 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=3D/path/to/dpdk/MAINTAINERS > > $ ./pw_maintainers_cli.py --type series set_pw_delegate SERIES_ID > > > > Signed-off-by: Ali Alnubani > > --- > > + 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. >=20 > I'm not sure we should skip patches which are already delegated. > If we use the command to explicitly delegate the patch, > we should do it, right? >=20 > The skip logic may be implemented at a higher level in the CI. I added an arg (--force_set_delegate) to force overriding delegates in v3. >=20 > > + Reference: > > + https://github.com/getpatchwork/git- > pw/blob/76b79097dc0a57c89b45dd53d9cacb7ff7b31bb2/git_pw/patch.py#L1 > 67 > > + """ > > + users =3D api.index('users', [('q', delegate)]) > > + if len(users) !=3D 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'])) > > + _ =3D api.update( > > + 'patches', patch['id'], [('delegate', users[0]['id= '])]) >=20 >=20