[Gen-art] Gen-ART Last Call review of draft-ietf-bess-evpn-bum-procedure-updates-09
Paul Kyzivat <pkyzivat@alum.mit.edu> Sat, 04 September 2021 16:49 UTC
Return-Path: <pkyzivat@alum.mit.edu>
X-Original-To: gen-art@ietfa.amsl.com
Delivered-To: gen-art@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 69EFE3A1340; Sat, 4 Sep 2021 09:49:05 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.001
X-Spam-Level:
X-Spam-Status: No, score=-2.001 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, RCVD_IN_MSPIKE_H2=-0.001, SPF_PASS=-0.001, URIBL_BLOCKED=0.001] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (1024-bit key) header.d=alum.mit.edu
Received: from mail.ietf.org ([4.31.198.44]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id 8pGhVTffjsAs; Sat, 4 Sep 2021 09:49:00 -0700 (PDT)
Received: from NAM11-BN8-obe.outbound.protection.outlook.com (mail-bn8nam11on2068.outbound.protection.outlook.com [40.107.236.68]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 94E693A1CAA; Sat, 4 Sep 2021 09:48:57 -0700 (PDT)
ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=SFkUXJqigTgPcmHuv+YRSPC+2ScrhaYKWuK3WLmRj0ukLGZtNV5vTS5PLvQf1yEY6LVmmJnhlPsFBMeT76HLLJWi6Fd0Ut+XCwmMixeg3deLeAWGGmCrPEB9xx6O2+J0Ee9CA9SF09TiEk+uxqspw5c13aWuH7fPNT7jLit2NH3M4kLo1sQnkqVorpM9mg2lDPjnv8YBefEGoYmGHDE1+xD655zvFdq1QUD6VupMMNG1XJOCYsEnsB91W3KRPL0ierVKOK2uDDehy/qh4g27xnVn0L2nGFirXde9UUJ7+yjPhaEjRpjUCWN/VPWjpl8XXwHbIPYkXVqz2BdeTdW2Iw==
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=i+NMVXvguM1USLsaYnUd4amfF+6KueEfXnXpAfMLLsA=; b=Y6iCzzTKqOosG5XEOyNcR1Aq/3M6J0lphnzOfS7nLTR92Aik5QyQy6+76T3kr4FGSoQziL+rmIpB+KhprpTx9b04XuOHrxVfgGgRAr4DQ/gVvyosZ9ka3mkK/kRR68GdsXWcwA2IPXOh6Ihu0XB275U6mqLHqPPvndf3v1qEXgfz5Hv+8qhmsv6eYfxCIQZRQDjLKna/7B4tlK8g3Q5CiSB9/VjJwEA1l2/Te81fZ+j6N/oB5A+SKo5xAB74Lch5BaWzitqTYF+5rWQoCZEY/g3+u/wyY4Gaxzr5gq2xjKDN4gnyGd8spls2SeJetC3+49Ahy/Ii83bOPNjeZbGdvA==
ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass (sender ip is 18.7.68.33) smtp.rcpttodomain=ietf.org smtp.mailfrom=alum.mit.edu; dmarc=bestguesspass action=none header.from=alum.mit.edu; dkim=none (message not signed); arc=none
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=alum.mit.edu; s=selector2; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=i+NMVXvguM1USLsaYnUd4amfF+6KueEfXnXpAfMLLsA=; b=cL4TrvlDAQI12R8JyhJsSx7syilxutJuBZ4uokDQTggbNIgTVznpqekj95f2B7zO9QcmpKwmPAwZD7KblD+7n52WBb8mg6+UA6ZxLI34XZh3bagzV2caHhSNWrE9GEVsb5Q3C2ELWeCU/BPN9tVhELlhI5bUFjFp/dji5ZisU7g=
Received: from SN2PR01CA0016.prod.exchangelabs.com (2603:10b6:804:2::26) by DM6PR12MB2793.namprd12.prod.outlook.com (2603:10b6:5:4f::27) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4478.17; Sat, 4 Sep 2021 16:48:55 +0000
Received: from SN1NAM02FT0004.eop-nam02.prod.protection.outlook.com (2603:10b6:804:2:cafe::dd) by SN2PR01CA0016.outlook.office365.com (2603:10b6:804:2::26) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4478.19 via Frontend Transport; Sat, 4 Sep 2021 16:48:55 +0000
X-MS-Exchange-Authentication-Results: spf=pass (sender IP is 18.7.68.33) smtp.mailfrom=alum.mit.edu; ietf.org; dkim=none (message not signed) header.d=none;ietf.org; dmarc=bestguesspass action=none header.from=alum.mit.edu;
Received-SPF: Pass (protection.outlook.com: domain of alum.mit.edu designates 18.7.68.33 as permitted sender) receiver=protection.outlook.com; client-ip=18.7.68.33; helo=outgoing-alum.mit.edu;
Received: from outgoing-alum.mit.edu (18.7.68.33) by SN1NAM02FT0004.mail.protection.outlook.com (10.97.4.226) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4478.19 via Frontend Transport; Sat, 4 Sep 2021 16:48:55 +0000
Received: from MacBook-Air.localdomain (c-24-62-227-142.hsd1.ma.comcast.net [24.62.227.142]) (authenticated bits=0) (User authenticated as pkyzivat@ALUM.MIT.EDU) by outgoing-alum.mit.edu (8.14.7/8.12.4) with ESMTP id 184Gmq8g008857 (version=TLSv1/SSLv3 cipher=AES128-GCM-SHA256 bits=128 verify=NOT); Sat, 4 Sep 2021 12:48:53 -0400
From: Paul Kyzivat <pkyzivat@alum.mit.edu>
To: draft-ietf-bess-evpn-bum-procedure-updates.all@ietf.org
Cc: General Area Review Team <gen-art@ietf.org>
Message-ID: <f51bcb31-369a-ea02-7f18-b05346ac12ac@alum.mit.edu>
Date: Sat, 04 Sep 2021 12:48:52 -0400
User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:78.0) Gecko/20100101 Thunderbird/78.13.0
MIME-Version: 1.0
Content-Type: text/plain; charset="windows-1252"; format="flowed"
Content-Language: en-US
Content-Transfer-Encoding: 7bit
X-EOPAttributedMessage: 0
X-MS-PublicTrafficType: Email
X-MS-Office365-Filtering-Correlation-Id: eb77a00d-7b82-4b6b-53c9-08d96fc3e231
X-MS-TrafficTypeDiagnostic: DM6PR12MB2793:
X-Microsoft-Antispam-PRVS: <DM6PR12MB2793596935CFC1E7C826BA49F9D09@DM6PR12MB2793.namprd12.prod.outlook.com>
X-MS-Oob-TLC-OOBClassifiers: OLM:10000;
X-MS-Exchange-SenderADCheck: 1
X-MS-Exchange-AntiSpam-Relay: 0
X-Microsoft-Antispam: BCL:0;
X-Microsoft-Antispam-Message-Info: k6PMf5yAvJILEO3O2jVuJ42G5+belspwWC1WZ/dvXT54LrzPiFUJmv0jnor8NYmLoBmyEYZnMXGkJx4XciKju39/eb9Kqsd6Hy1hHyLQmZMVKwr1jH5JBS0NMlhhUrIffvNSDNXCDV/ooz8W8oY2KlBT3mdSaMYtkM0F837cRYNu1Lm0U3PJtRsmMmJKT2uj4VQi381Mb8jRYzglFyzPBmr71AqI6u9xTvCfE3HgxmM1Pv/gaZOQTfb7VTLRXjdjZ9cGKlup/W4muwv+9hM7Yh7EGIZHvQKkODY9NnNeZnI3WGxzrHYDnDlv2ViIqk+yV3sVk0ImJIiFell8DA6zx8rfld9gPrfndmKL5ZP1Aa9+wLH4HX24cFt3un2Y3nG+iOtR8EXqOF0Rumrxi/Y8N68oCC848AqBYdkRE3xOHOGVfINg5md8ihBP+0sS8zfKUH9+GJqDTjJJbTTORp0sjgsoQAfZmyTmq9pkEQ+UsUkXTB+RUU4JgDRcnjgwKWMy5CWX3nz1R95v0ySH8cPBk9REPTgFcYyf2WiXOu7jnjwGZldr/oSDjG95uhW2tIekT+vhSHNFXGnKgihnLU6PPp3Kn+jj4iNnIEQfXvxWw9l2Gwh2orKuX8sbxCqAEfkcrZ2/O82UOSaEcesTvwweK/eKKY9uGkOw/aL0G9I0f6a6D1y1NExab+tHMx4qXFxDP+9hj2Yve5PCuf/2+MpP8qd9pYi1pqo2kKFNIyggzZCczUVLKv8bdPslOEkTbXGdicDVbHs903j9cZL/xVoVyk3NsvL8RRxgdThcdPCHSO1c76d6XvzINKOJdOWB27zv
X-Forefront-Antispam-Report: CIP:18.7.68.33; CTRY:US; LANG:en; SCL:1; SRV:; IPV:CAL; SFV:NSPM; H:outgoing-alum.mit.edu; PTR:outgoing-alum.mit.edu; CAT:NONE; SFS:(36840700001)(46966006)(356005)(15650500001)(70206006)(70586007)(31696002)(31686004)(2906002)(7596003)(86362001)(4326008)(5660300002)(82310400003)(6916009)(8936002)(8676002)(75432002)(186003)(336012)(83380400001)(2616005)(36906005)(450100002)(36860700001)(508600001)(47076005)(26005)(956004)(316002)(43740500002); DIR:OUT; SFP:1101;
X-OriginatorOrg: alum.mit.edu
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 04 Sep 2021 16:48:55.1885 (UTC)
X-MS-Exchange-CrossTenant-Network-Message-Id: eb77a00d-7b82-4b6b-53c9-08d96fc3e231
X-MS-Exchange-CrossTenant-Id: 3326b102-c043-408b-a990-b89e477d582f
X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=3326b102-c043-408b-a990-b89e477d582f; Ip=[18.7.68.33]; Helo=[outgoing-alum.mit.edu]
X-MS-Exchange-CrossTenant-AuthSource: SN1NAM02FT0004.eop-nam02.prod.protection.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Anonymous
X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM6PR12MB2793
Archived-At: <https://mailarchive.ietf.org/arch/msg/gen-art/LjtdVBTeUMV-In2jVGhQb0D4dXc>
Subject: [Gen-art] Gen-ART Last Call review of draft-ietf-bess-evpn-bum-procedure-updates-09
X-BeenThere: gen-art@ietf.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: "GEN-ART: General Area Review Team" <gen-art.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/gen-art>, <mailto:gen-art-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/gen-art/>
List-Post: <mailto:gen-art@ietf.org>
List-Help: <mailto:gen-art-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/gen-art>, <mailto:gen-art-request@ietf.org?subject=subscribe>
X-List-Received-Date: Sat, 04 Sep 2021 16:49:06 -0000
I am the assigned Gen-ART reviewer for this draft. The General Area Review Team (Gen-ART) reviews all IETF documents being processed by the IESG for the IETF Chair. Please treat these comments just like any other last call comments. For more information, please see the FAQ at <https://trac.ietf.org/trac/gen/wiki/GenArtfaq>. Document: draft-ietf-bess-evpn-bum-procedure-updates-09 Reviewer: Paul Kyzivat Review Date: 2021-09-04 IETF LC End Date: 2021-09-07 IESG Telechat date: ? Summary: This draft has serious issues, described in the review, and needs to be rethought. General: My review of this document is limited because I have no knowledge of the subject domain. Nevertheless I think I was able to grasp the gist of what this document intends to achieve and identify a concern. However it is possible that I have misunderstood and so my comments may be off base. While I have no reason to doubt the mechanisms specified, I think the manner in which they are specified is likely to lead to confusion and misunderstanding by developers. IIUC, RFC7117 improved the handling of BUM traffic for VPLS, but did not address BUM traffic for EVPN. Then RFC7432 specified how to handle BUM traffic for EVPN while referencing RFC7117 for some of its procedures, even though RFC7117 had no provision for support of EVPN. It appears to me that someone who had an implementation of RFC7117 for VPLS might have had to modify it to support RFC7432, yet RFC7432 did not indicate that it updated RFC7117. The developer would have had to infer what changes were required. But at least the changes seem to be small and unlikely to be misunderstood. The current document specifies in its heading and abstract that it updates RFC7432, while not mentioning RFC7117. Yet section 2 says: ... For brevity, only changes/additions to relevant [RFC7117] and [RFC7524] procedures are specified, instead of repeating the entire procedures. From these it is ambiguous whether RFC7117 is or is not being updated. It then goes on to state: Note that these are to be applied to EVPN only, and not updates to [RFC7117] or [RFC7524]. This further clouds things. How could it be known how future updates to those documents might interact with the changes in this document? In the remainder of the document I find no explicit text that appears to normatively updates RFC7432. I find this mystifying. However there are numerous places that normatively change RFC7117. Several of these are of the form: The following bullet in Section N.N.N.N of [RFC7117]: ... is changed to the following when applied to EVPN: ... even though RFC7117 didn't contemplate supporting EVPN at all. This seems to assume that an entirely different implementation of RFC7117 will be made for EVPN and these modifications made to that, while not impacting implementations of RFC7117 being used for other types of VPNs. Is this a reasonable assumption? Overall it seems that it will be very difficult for a developer to read this document and determine exactly what must be implemented, or after the fact to determine whether an implementation conforms to this document. IMO a different style of specification is called for. Probably an RFC7117bis and perhaps a RFC7432bis.
- [Gen-art] Gen-ART Last Call review of draft-ietf-… Paul Kyzivat
- Re: [Gen-art] Gen-ART Last Call review of draft-i… Jeffrey (Zhaohui) Zhang
- Re: [Gen-art] Gen-ART Last Call review of draft-i… Paul Kyzivat
- Re: [Gen-art] Gen-ART Last Call review of draft-i… Jeffrey (Zhaohui) Zhang
- Re: [Gen-art] Gen-ART Last Call review of draft-i… Paul Kyzivat