[Idr] Martin Vigoureux's No Objection on draft-ietf-idr-tunnel-encaps-20: (with COMMENT)
Martin Vigoureux via Datatracker <noreply@ietf.org> Sun, 29 November 2020 22:03 UTC
Return-Path: <noreply@ietf.org>
X-Original-To: idr@ietf.org
Delivered-To: idr@ietfa.amsl.com
Received: from ietfa.amsl.com (localhost [IPv6:::1]) by ietfa.amsl.com (Postfix) with ESMTP id 2C2973A04BC; Sun, 29 Nov 2020 14:03:19 -0800 (PST)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
From: Martin Vigoureux via Datatracker <noreply@ietf.org>
To: The IESG <iesg@ietf.org>
Cc: draft-ietf-idr-tunnel-encaps@ietf.org, idr-chairs@ietf.org, idr@ietf.org, John Scudder <jgs@juniper.net>, aretana.ietf@gmail.com, Susan Hares <shares@ndzh.com>, shares@ndzh.com
X-Test-IDTracker: no
X-IETF-IDTracker: 7.23.0
Auto-Submitted: auto-generated
Precedence: bulk
Reply-To: Martin Vigoureux <martin.vigoureux@nokia.com>
Message-ID: <160668739914.29163.16132907073207059118@ietfa.amsl.com>
Date: Sun, 29 Nov 2020 14:03:19 -0800
Archived-At: <https://mailarchive.ietf.org/arch/msg/idr/yFsz559yAW0OPbdrOwvoQZJygy0>
Subject: [Idr] Martin Vigoureux's No Objection on draft-ietf-idr-tunnel-encaps-20: (with COMMENT)
X-BeenThere: idr@ietf.org
X-Mailman-Version: 2.1.29
List-Id: Inter-Domain Routing <idr.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/idr>, <mailto:idr-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/idr/>
List-Post: <mailto:idr@ietf.org>
List-Help: <mailto:idr-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/idr>, <mailto:idr-request@ietf.org?subject=subscribe>
X-List-Received-Date: Sun, 29 Nov 2020 22:03:19 -0000
Martin Vigoureux has entered the following ballot position for draft-ietf-idr-tunnel-encaps-20: No Objection When responding, please keep the subject line intact and reply to all email addresses included in the To and CC lines. (Feel free to cut this introductory paragraph, however.) Please refer to https://www.ietf.org/iesg/statement/discuss-criteria.html for more information about IESG DISCUSS and COMMENT positions. The document, along with other ballot positions, can be found here: https://datatracker.ietf.org/doc/draft-ietf-idr-tunnel-encaps/ ---------------------------------------------------------------------- COMMENT: ---------------------------------------------------------------------- Hi thank you for the work you put in this document. I only have minor comments/questions (though a couple of ones might be a bit more important than the rest). Because RFC 8365 depends on RFC 5640, it is similarly obsoleted. This is further discussed in Appendix A. But neither the abstract nor the metadata mention this, and in fact Appendix A is not clear about whether RFC 8365 is effectively obsoleted. Not critical but IANA registration of the Tunnel Encapsulation Attribute is with a upper-case 'A', while through out the document lower-case 'a' is used. A Tunnel Encapsulation TLV, also known as Tunnel TLV but in the document you most often omit using Tunnel, thus reducing it to TLV. The Reserved subfield SHOULD be originated as zero. It MUST be disregarded on receipt, and it MUST be propagated unchanged. In the rest of the document, for unused bits or reserved fields you require MUST be zero. Any reason for different levels of requirements? The Address Family subfield contains a value from IANA's "Address Family Numbers" registry. This document assumes that the Address Family is either IPv4 or IPv6; use of other address families is outside the scope of this document. If the Address Family subfield has any value other than IPv4 or IPv6, the Tunnel Egress Endpoint sub-TLV is considered "unrecognized" Not critical but I have the impression that these two paragraphs slightly contradict each other. First one allows for other AFIs to be used in the future in the Address Family subfield, while the second kind of rules that out. Maybe simply add at the beginning of the second paragraph: "In the context of this specification,". Note that in order to send an IP packet or an MPLS packet through a VXLAN tunnel, the packet must first be encapsulated in an Ethernet header, which becomes the "inner Ethernet header" described in [RFC7348]. The VXLAN Encapsulation sub-TLV may contain information (e.g.,the MAC address) that is used to form this Ethernet header. This seems redundant with the last paragraph of the second bullet above (in the document) sub-TLVs must be used MUST? 0 1 2 3 4 5 6 7 +-+-+-+-+-+-+-+-+ | 0 or 1 | +-+-+-+-+-+-+-+-+ Should the values be 1 or 2 instead? At least the text says the only two allowed values are 1 and 2. If a Label-Index is present in the Prefix-SID sub-TLV, then when a packet is sent through the tunnel identified by the TLV, if that tunnel is from a labeled address family do you mean from "a BGP UPDATE of a labeled address family" ? document specifying such joint use should provide details SHOULD? +--------------+--------------------------------+-----------------+ | 0 | V (Virtual Network Identifier) | (this document) | +--------------+-----------------+-----------------+ | 0 | V (VN-ID) | (this document) | Purely cosmetic, but may be choose VN-ID or Virtual Network Identifier as the same name for the two allocations. Nits "Value Field" in the Fig1 and Fig2 legends seems superfluous. s/In this case, the length field/In this case, the Length field/ s/address subfield/Address subfield/ (6 occurrences) s/depicted in the Address Field/depicted in the Address subfield/ (3 occurrences, including Section 3.1.1 title) s/Tunnel Type itself/tunnel type itself/? s/value field/Value field/ (17 occurrences + 6 occurrences with line break between the two words) s/Reserved (two fields)/Reserved (two octets)/ (twice) s/The flags field of the VXLAN header/The flags field of the NVGRE header/ (in the NVGRE section) s/Inner Destination MAC address/Inner Destination MAC Address/ s/of the cookie/of the Cookie/ s/procedures of Section 9/procedures of Section 9)/ s/VNI (Virtual Network Identifier)/VNI (VXLAN Network Identifier)/
- [Idr] Martin Vigoureux's No Objection on draft-ie… Martin Vigoureux via Datatracker
- Re: [Idr] Martin Vigoureux's No Objection on draf… John Scudder