Re: [Idr] Genart last call review of draft-ietf-idr-tunnel-encaps-19

Susan Hares <shares@ndzh.com> Tue, 06 October 2020 22:59 UTC

Return-Path: <shares@ndzh.com>
X-Original-To: idr@ietfa.amsl.com
Delivered-To: idr@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id E1BC83A1530; Tue, 6 Oct 2020 15:59:32 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: 0.948
X-Spam-Level:
X-Spam-Status: No, score=0.948 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DOS_OUTLOOK_TO_MX=2.845, SPF_HELO_NONE=0.001, SPF_NONE=0.001, URIBL_BLOCKED=0.001] autolearn=no autolearn_force=no
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 pbr4r2sGPYo2; Tue, 6 Oct 2020 15:59:31 -0700 (PDT)
Received: from hickoryhill-consulting.com (50-245-122-97-static.hfc.comcastbusiness.net [50.245.122.97]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id D069B3A152F; Tue, 6 Oct 2020 15:59:30 -0700 (PDT)
X-Default-Received-SPF: pass (skip=forwardok (res=PASS)) x-ip-name=174.25.164.118;
From: "Susan Hares" <shares@ndzh.com>
To: "'Gyan Mishra'" <hayabusagsm@gmail.com>, <gen-art@ietf.org>
Cc: <draft-ietf-idr-tunnel-encaps.all@ietf.org>, <last-call@ietf.org>, <idr@ietf.org>
References: <160170536718.20424.6017677549974569740@ietfa.amsl.com>
In-Reply-To: <160170536718.20424.6017677549974569740@ietfa.amsl.com>
Date: Tue, 6 Oct 2020 18:59:27 -0400
Message-ID: <008601d69c34$5801e540$0805afc0$@ndzh.com>
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: quoted-printable
X-Mailer: Microsoft Outlook 14.0
Thread-Index: AQG+acMSNBgPiJKCgYkY4Ytc8D88rKm7b6rw
Content-Language: en-us
X-Antivirus: AVG (VPS 201006-2, 10/06/2020), Outbound message
X-Antivirus-Status: Not-Tested
X-Authenticated-User: skh@ndzh.com
Archived-At: <https://mailarchive.ietf.org/arch/msg/idr/szkjOZdsFHhztONlNG5L1BaaZDk>
Subject: Re: [Idr] Genart last call review of draft-ietf-idr-tunnel-encaps-19
X-BeenThere: idr@ietf.org
X-Mailman-Version: 2.1.29
Precedence: list
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: Tue, 06 Oct 2020 22:59:33 -0000

Gyan: 

Could you unpack the following statements a bit: 
 
Is tunnel type RFC 7510 MPLS over UDP supported.  Also RFC 8663 SR-MPLS over IP for SRv6 and SR-MPLS interoperability where MPLS over GRE RFC 4023 is used or RFC 7510 MPLS over UDP and outer encapsulation is IPv6 for SRv6 forwarding plane to tunnel SR-MPLS.

Thank you, 
Sue Hares 
(new shepherd for draft-ietf-idr-tunnel-encaps)

-----Original Message-----
From: Gyan Mishra via Datatracker [mailto:noreply@ietf.org] 
Sent: Saturday, October 3, 2020 2:09 AM
To: gen-art@ietf.org
Cc: draft-ietf-idr-tunnel-encaps.all@ietf.org; last-call@ietf.org; idr@ietf.org
Subject: Genart last call review of draft-ietf-idr-tunnel-encaps-19

Reviewer: Gyan Mishra
Review result: Ready

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-idr-tunnel-encaps-??
Reviewer: Gyan Mishra
Review Date: 2020-10-02
IETF LC End Date: 2020-10-01
IESG Telechat date: Not scheduled for a telechat

Summary:
   This draft updates RFC 5512 is which was never used in production.
    This document deprecates the original RFC 5512
   Encapsulation SAFI used for GRE encapsulation type only and defined extended
   community attributes  (which has never been used in production) , and
   specifies semantics for the attribute when it is carried in UPDATEs of
   certain other SAFIs.  This document adds support for additional Tunnel
   Types, and allows a remote tunnel endpoint address to be specified for each
   tunnel.  This document also provides support for specifying fields of any
   inner or outer encapsulations that may be used by a particular tunnel.

This document is well written and clearly defines all the tunnel encapsulation attributes that exist today. This draft modification of the usage of the tunnel encapsulation attribute is backwards compatible so does not impact the existing deployments of the tunnel encapsulation types such as vxlan,vxlan GPE, NVGRE, MPLS over GRE, L2TPV3.

Major issues:
None

Minor issues:
Is tunnel type RFC 7510 MPLS over UDP supported.  Also RFC 8663 SR-MPLS over IP for SRv6 and SR-MPLS interoperability where MPLS over GRE RFC 4023 is used or RFC 7510 MPLS over UDP and outer encapsulation is IPv6 for SRv6 forwarding plane to tunnel SR-MPLS.

Nits/editorial comments:
As this update is backwards compatible to tunnel encapsulation attributes deployed today is there any code upgrade for vendor implementation required now for router vendors to support the new encapsulation attributes TLV and Sub TLVs and extended community attributes defined with this draft.