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 D839DA0C4C; Thu, 2 Sep 2021 18:33:30 +0200 (CEST) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 88D134003E; Thu, 2 Sep 2021 18:33:30 +0200 (CEST) Received: from mga05.intel.com (mga05.intel.com [192.55.52.43]) by mails.dpdk.org (Postfix) with ESMTP id BDE214003C for ; Thu, 2 Sep 2021 18:33:28 +0200 (CEST) X-IronPort-AV: E=McAfee;i="6200,9189,10095"; a="304743338" X-IronPort-AV: E=Sophos;i="5.85,262,1624345200"; d="scan'208";a="304743338" Received: from fmsmga001.fm.intel.com ([10.253.24.23]) by fmsmga105.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 02 Sep 2021 09:33:27 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.85,262,1624345200"; d="scan'208";a="602456185" Received: from orsmsx606.amr.corp.intel.com ([10.22.229.19]) by fmsmga001.fm.intel.com with ESMTP; 02 Sep 2021 09:33:27 -0700 Received: from orsmsx610.amr.corp.intel.com (10.22.229.23) by ORSMSX606.amr.corp.intel.com (10.22.229.19) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2242.12; Thu, 2 Sep 2021 09:33:27 -0700 Received: from orsedg603.ED.cps.intel.com (10.7.248.4) by orsmsx610.amr.corp.intel.com (10.22.229.23) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2242.12 via Frontend Transport; Thu, 2 Sep 2021 09:33:27 -0700 Received: from NAM11-CO1-obe.outbound.protection.outlook.com (104.47.56.174) by edgegateway.intel.com (134.134.137.100) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.1.2242.10; Thu, 2 Sep 2021 09:33:26 -0700 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=lrbzxFy6ZUr42jjIrk8rovfN4tHJ4DIgvXcxdFCchQxZb4TYVMHyN68Uu2fCgNO6nQuX/ROj8DJ/VVOrDRHosxuNBvsKsHgYPlT/6JpoP5ttxjMI+fSpsQZkRF2MGRC/IesX9g3aSZ7CODP/zf0joP0f1d3u9mUO7Us2Da3393ylFq+RAEoZGdRS+jNwK59UkGLnM8b6N6NpaC6cnQO28yXI+ZlDrtwFrFDC8MP30tK+9CNfx8hJSobNInn4F6/iilUBCgYLOpZP9hWjELwSUxx/rE0lgW9jte6ACShmr3HTwIHf4yatnvpEyfesw29NBX7ulEM55ZPNB0POUTTVPg== 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; bh=DaF/woYE7YN6PEbLIC5eshDSX9Xt/ORjqR8rpWDO9lE=; b=JcziOdKpEgLOLtNxKwV05davCiLdGdIAmnHR0w4V9nS4VZELgmUYK0d5pZlGshlbmks1AWdJ4qW7PyhCWHQLqJ2IsQJKsAiyizfI+BnaDOAx4rIY0NCo2FFjq100pgYatEJtfFJ0KJzV8Y3Et7bt+Jn1p6l+OvCvnCD8NGPyphxoZWI4fFFV3e1po44iLVrY0biIGKBbvz98nOhYb052uG7MhyK2xjTEjlY0/mvULsoMiWMBdeDNmESxR6xu5ORhQUe3bU4cgzJA9AE+bH/2j9AjQcVhPbcac1KFWSSUb+K81v1zBXEcwpNkK+Ix8Kh0bmk8uHxwlDiVmvVG67s24A== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=intel.com; dmarc=pass action=none header.from=intel.com; dkim=pass header.d=intel.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=intel.onmicrosoft.com; s=selector2-intel-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=DaF/woYE7YN6PEbLIC5eshDSX9Xt/ORjqR8rpWDO9lE=; b=FsRVZvvFY6Slmx338ov0VGKEleGe5GbsAq3eETxFomYFLnITeHitQVlceFgf87M8XINPANkpfVOq2m2XvEiWtPaANzPimqcVfpgz5tom2VkSoHMG7Lo6TMx9WkNOU1HUds1vOYmfxZbTbxoc6j7C5FRBNMUnPdoWtnr2pcko9AU= Authentication-Results: intel.com; dkim=none (message not signed) header.d=none;intel.com; dmarc=none action=none header.from=intel.com; Received: from PH0PR11MB5000.namprd11.prod.outlook.com (2603:10b6:510:41::19) by PH0PR11MB5173.namprd11.prod.outlook.com (2603:10b6:510:39::12) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4478.19; Thu, 2 Sep 2021 16:33:25 +0000 Received: from PH0PR11MB5000.namprd11.prod.outlook.com ([fe80::747b:3a08:d1ec:31fc]) by PH0PR11MB5000.namprd11.prod.outlook.com ([fe80::747b:3a08:d1ec:31fc%5]) with mapi id 15.20.4478.020; Thu, 2 Sep 2021 16:33:25 +0000 To: Thomas Monjalon , CC: , Asaf Penso , John McNamara , Ajit Khaparde , "Bruce Richardson" References: <1610304247-11455-1-git-send-email-asafp@nvidia.com> <20210826101102.709502-1-thomas@monjalon.net> From: Ferruh Yigit X-User: ferruhy Message-ID: Date: Thu, 2 Sep 2021 17:33:20 +0100 In-Reply-To: <20210826101102.709502-1-thomas@monjalon.net> Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 8bit X-ClientProxiedBy: DB6PR0202CA0036.eurprd02.prod.outlook.com (2603:10a6:4:a5::22) To PH0PR11MB5000.namprd11.prod.outlook.com (2603:10b6:510:41::19) MIME-Version: 1.0 Received: from [192.168.0.206] (37.228.236.146) by DB6PR0202CA0036.eurprd02.prod.outlook.com (2603:10a6:4:a5::22) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4478.17 via Frontend Transport; Thu, 2 Sep 2021 16:33:23 +0000 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 1dc61236-3957-4339-1dd7-08d96e2f630b X-MS-TrafficTypeDiagnostic: PH0PR11MB5173: X-LD-Processed: 46c98d88-e344-4ed4-8496-4ed7712e255d,ExtAddr X-MS-Exchange-Transport-Forked: True X-Microsoft-Antispam-PRVS: 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: XCOrEoDeF44lt/TOLceuf6aU5nbiwvh+TIna8MytydRgCmrUbl2lXwT0gLI2kdMeOHp7G1bDOtVefpmfeM6YdqbxOGqQH0XmE5LG8kiAv6EYNmVf61gBRhldTchN8j+jMr1ZqCzCuR6qRN1f/HqTrfE6xu2eMv7Kt6IWOVgwY2box0KjozJP4kYSR/DLL1Tf5T5bXqS9pmQ1VUrKY88AHICtyH6jpanzsMhrCokDYqSvsOVcu61zCq0WawdFEDLsGtzYYziZdnWRzMBPeQvHIWTeZBHmcwde5CvH0oK8vJxJt1G9v67hf623lTO17ZLdu0O4VOby17A6+EMVBn0nYljaVONhdX/z77P9rNLlseM3Xpgij5XkSI8SCkYLkS4E1HMsBlVNbHZW8ImEevXinOn8953nJonw6mwjxTgxViuAdPQqxE0r09EiEVQmrqfpK1P8dci/ep/uwXv/njCHQBrP/BT3En5LmqktjsPDsO5JLOxQwP6NjLfOZ1+crtPZYczi8Ikm6ow02zfXX8wSTUkJ1QXG9whmKzt9tyEtP0vg6jSTW7F+mgocXuKOQkVlblhUXdCzDMACIVwOmmG8GenDAgnuId5ekK1M22sFwed+WezVpUDQA6iD7BYu+F+EkaDIR8r7v5hY/S4mqSFiQLe+C840tN+F8QMY9H81bXDy+7hFwvwer1k4VBpYKy3YtepPmwoIFDutPiSnhYgoZXQJj12qWk6V5kutzesFq11lq1C4jUglGHQ1FfA3Z90j4revUhM5v//XZUQfGRMEcPG2+ev1uwFhDVAmnVLajoo= X-Forefront-Antispam-Report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:PH0PR11MB5000.namprd11.prod.outlook.com; PTR:; CAT:NONE; SFS:(4636009)(136003)(346002)(366004)(396003)(39860400002)(376002)(316002)(4326008)(31686004)(956004)(8936002)(26005)(2616005)(107886003)(16576012)(83380400001)(54906003)(5660300002)(36756003)(186003)(8676002)(6486002)(38100700002)(53546011)(66476007)(66556008)(2906002)(31696002)(86362001)(478600001)(66946007)(44832011)(45980500001); DIR:OUT; SFP:1102; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?K0hnRnJBSk9FNTRIUkRpWkthZXBSbm8rM1o0QXBhOWVleEZxMFVwTFFYQ21X?= =?utf-8?B?aVc4NTVzRDlKN0VUNzUrWG56REUyb056YTN2TE41N3N4M3pCbGp4U0oyNHg5?= =?utf-8?B?QVN3Y0tDQm5aem81TXFmcFBVZXJOS2lZdGxqYm8zWTRpVFpXb3JvM1M0bElu?= =?utf-8?B?bWlUV1EyZE5aVmx3bmUrTTZsL2Q4dUw2d0NURGJwRHUrcmZhQUFqV0NhT0hW?= =?utf-8?B?WEhsN1Z0elgzdW1ma1JwTWlIL0NXUy8rNjRVeFJRYnJ0RGhrYlpJY1h5cHFT?= =?utf-8?B?OERUV3pLMndMaUJtSG5yZklvRWJOcS9uQ1lPRFpJRUhHSys4L3E0MG8wSjFv?= =?utf-8?B?Y0hhb0d2dHRSVno3a0dLTUx2c2YweFpiaVhSRERibDhOcVUzTk1LNE0wQWg2?= =?utf-8?B?VzA3NUVSd3JNcmoxd3Q0c2hvbXZCam5iK3NDMDNhUTJ4YjRoSUJxWXh3M0c4?= =?utf-8?B?SHRFMWxTS2FGMTI3RDV4RWltNXh1QWpTR0Q2T0o1MVJDQit5U1oxWEdkb3cy?= =?utf-8?B?OHVkcmN6QzZpWVczWHhWZmZidXVETC9sQk9jKytUL0N4eStOa29NdGZTczZ5?= =?utf-8?B?L2hyb2QzNWFSbkt3S3FTUk95TmwxVndqRzVIVnRVUkI3YWs4ek9hYTV5LzAv?= =?utf-8?B?VU9XN25yT0N3eFpTNFc1dUJVSTBTcS9nQmp0d3VSekRyWEJWMDhaRktQUnVQ?= =?utf-8?B?cWNCa3ZkMzE2ZFdjZFFRWnVSM2N0djl5cy83cUJqNEozdnpRckF4NVpKdUp3?= =?utf-8?B?UGszdnNJMnViV21vd0NMWmFERXlWVFdNWTBSWWRPV1o2am02RlB3Y01wRXhF?= =?utf-8?B?dUYrR1V0eS9UN1RMN0pRZ0tlenlpL1V1b1FiNVVIN0FPaHZRZTlaOWNHSlZG?= =?utf-8?B?TVdHOWlGNS8rOEw0UkZjMFVob3NYdkNtTjh1bzlUbitleGMwTzNYWkhyRXR4?= =?utf-8?B?RlE3cjFIQUcvTGtsWlRmUTdPVXpxbk1EZVdjUUNIeFV3KzBDdlIwNWFVcFQ3?= =?utf-8?B?a3pJZGRseG0vTlJhL1pveXdCVDRlUkI1OGx3T3BrYWZEMFdUais0OU1meWlE?= =?utf-8?B?SlZjaXJpN29rNEE3clNWVjR6a1Rnb2NwaHpFejUxL29DVE84VitTSC95RGRj?= =?utf-8?B?TTZvUSs4eG9ZVDdEUzU2YTRXUHQ3SitwL2Q0Y1hWOXdpSmt2SmRtejByeDZp?= =?utf-8?B?TzdMd0ExbXk1d21SdGdpMXltZFNzV3RSZXN0b3NHWVY3VUF4aHBLTlE4Si8z?= =?utf-8?B?eVMrNTNEWUppeXFMcXh3SXUrQWZqZGVDNjB4bTczTUNVWC93SDFiTUF3Nmsx?= =?utf-8?B?QThvcUpqU0owRHg3eWpKZjd3cStKOTJmSHNDRHE2ZTJJR00wWTNVVU15RUgx?= =?utf-8?B?SGYwTlRpUzhHYUdpcXR5ekROTnRxbFZVZ3Axa2tkMmdRRWxIeFo3NHZCNUwy?= =?utf-8?B?bGF5cWFqeS9qTGhSSlZxcTFWMXQvb3IzN2I5L25yelgyU3JFUjVvMzlTeEpF?= =?utf-8?B?cW9RVjJMTWs5UitINnByNnVZWjFjU1FzTWtnbUtONEJMY2YrUTlmbUx6THYw?= =?utf-8?B?YTlwL25nRUNEUHBIb0x3ZlI2ZnBwR2tkSkhnMHI3MEh2SC93Mk1CUXE0RE12?= =?utf-8?B?U1BKenBQV1VpUUpnV0VmTHdndUNScDBQbFBubzNzeFJhRjlxbU05djZyNSs2?= =?utf-8?B?SGl5VWd5akZhKzBlSkhkRXB3U2tDK2QwN1JES1U5MVlEbjZRUmwzcWx6c2l5?= =?utf-8?Q?DZ9Zr+K+r2FUk35s9lPaewzGbHg/iZZbyc3fAFm?= X-MS-Exchange-CrossTenant-Network-Message-Id: 1dc61236-3957-4339-1dd7-08d96e2f630b X-MS-Exchange-CrossTenant-AuthSource: PH0PR11MB5000.namprd11.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 02 Sep 2021 16:33:25.5091 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 46c98d88-e344-4ed4-8496-4ed7712e255d X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: IVhXbDDE/2nAKIopMQexHqV5lLOu9VY9XU4N3z5ve3o/liPnZW+b3J+3behq8usnUW2E4OfG0zRQvucP/3ZhYw== X-MS-Exchange-Transport-CrossTenantHeadersStamped: PH0PR11MB5173 X-OriginatorOrg: intel.com Subject: Re: [dpdk-dev] [PATCH v8] doc: add release milestones definition X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" On 8/26/2021 11:11 AM, Thomas Monjalon wrote: > From: Asaf Penso > > Adding more information about the release milestones. > This includes the scope of change, expectations, etc. > > Signed-off-by: Asaf Penso > Signed-off-by: Thomas Monjalon > Acked-by: John McNamara > Acked-by: Ajit Khaparde > Acked-by: Bruce Richardson > --- > v2: fix styling format and add content in the commit message > v3: change punctuation and avoid plural form when unneeded > v4: avoid abbreviations, "Priority" in -rc, and reword as John suggests > v5: note that release candidates may vary > v6: merge RFC and proposal deadline, add roadmap link and reduce duplication > v7: make expectations clearer and stricter > v8: add tests, more fixes, maintainers approval and new API rules > --- > doc/guides/contributing/patches.rst | 85 +++++++++++++++++++++++++++-- > 1 file changed, 80 insertions(+), 5 deletions(-) > > diff --git a/doc/guides/contributing/patches.rst b/doc/guides/contributing/patches.rst > index b9cc6e67ae..ef784d0f99 100644 > --- a/doc/guides/contributing/patches.rst > +++ b/doc/guides/contributing/patches.rst > @@ -164,6 +164,10 @@ Make your planned changes in the cloned ``dpdk`` repo. Here are some guidelines > the :doc:`ABI policy ` and :ref:`ABI versioning ` > guides. New external functions should also be added in alphabetical order. > > +* Any new API function should be used in ``/app`` test directory. > + > +* When introducing a new device API, at least one driver should implement it. > + ack > * Important changes will require an addition to the release notes in ``doc/guides/rel_notes/``. > See the :ref:`Release Notes section of the Documentation Guidelines ` for details. > > @@ -177,6 +181,8 @@ Make your planned changes in the cloned ``dpdk`` repo. Here are some guidelines > * Add documentation, if relevant, in the form of Doxygen comments or a User Guide in RST format. > See the :ref:`Documentation Guidelines `. > > +* Code and related documentation must be updated atomically in the same patch. > + ack > Once the changes have been made you should commit them to your local repo. > > For small changes, that do not require specific explanations, it is better to keep things together in the > @@ -185,11 +191,6 @@ Larger changes that require different explanations should be separated into logi > A good way of thinking about whether a patch should be split is to consider whether the change could be > applied without dependencies as a backport. > > -It is better to keep the related documentation changes in the same patch > -file as the code, rather than one big documentation patch at the end of a > -patchset. This makes it easier for future maintenance and development of the > -code. > - > As a guide to how patches should be structured run ``git log`` on similar files. > > > @@ -663,3 +664,77 @@ patch accepted. The general cycle for patch review and acceptance is: > than rework of the original. > * Trivial patches may be merged sooner than described above at the tree committer's > discretion. > + > + > +Milestones definition > +--------------------- > + > +Each DPDK release has milestones that help everyone to converge to the release date. > +The following is a list of these milestones together with > +concrete definitions and expectations for a typical release cycle. > +An average cycle lasts 3 months and have 4 release candidates in the last month. > +Test reports are expected to be received after each release candidate. > +The number and expectations of release candidates might vary slightly. > +The schedule is updated in the `roadmap `_. > + > +.. note:: > + Sooner is always better. Deadlines are not ideal dates. > + > + Integration is never guaranteed but everyone can help. > + > +Roadmap > +~~~~~~~ > + > +* Announce new features in libraries, drivers, applications, and examples. > +* To be published before the first day of the release cycle. > + > +Proposal Deadline > +~~~~~~~~~~~~~~~~~ > + > +* Must send an RFC (Request For Comments) or a complete patch of new features. > +* Early RFC gives time for design review before complete implementation. > +* Should include at least the API changes in libraries and applications. > +* Library code should be quite complete at the deadline. > +* Nice to have: driver implementation, example code, and documentation. > + > +rc1 > +~~~ > + > +* Priority: libraries. No library feature should be accepted after -rc1. > +* API changes or additions must be implemented in libraries. > +* The API must include Doxygen documentation > + and be part of the relevant .rst files (library-specific and release notes). > +* API should be used in a test application (``/app``). > +* At least one PMD should implement the API. > + It may be a draft sent in a separate series. > +* The above should be sent to the mailing list at least 2 weeks before -rc1 > + to give time for review and maintainers approval. > +* If no review after 10 days, a reminder should be sent. > +* Nice to have: example code (``/examples``) > + > +rc2 > +~~~ > + > +* Priority: drivers. No driver feature should be accepted after -rc2. > +* A driver change must include documentation > + in the relevant .rst files (driver-specific and release notes). > +* The above should be sent to the mailing list at least 2 weeks before -rc2. Is 'the above' driver changes? It is good the have all driver changes two weeks before -rc2 but taking into account that overall time between -rc1 & -rc2 is 3/4 weeks, two weeks before -rc2 may be hard to do practically. We may go with this and try and evaluate later or reduce the requirement to one week before -rc2, what do you think? > +* Any issue found in -rc1 should be fixed. > + > +rc3 > +~~~ > + > +* Priority: applications. No application feature should be accepted after -rc3. > +* New functionality that does not depend on libraries update > + can be integrated as part of -rc3. > +* The application change must include documentation in the relevant .rst files > + (application-specific and release notes if significant). > +* Libraries and drivers cleanup are allowed. > +* Small driver reworks. > +* Critical and minor bug fixes. As mentioned before, my concern is this may create false impression that bugs are fixed only in this phase. What about remove this line completely and update below -rc4 one as 'Critical bug fixes only.'? I think that makes intention more clear. > + > +rc4 > +~~~ > + > +* Documentation updates. > +* Critical bug fixes. >