Re: [Idr] AD Review of draft-ietf-idr-tunnel-encaps-15

Susan Hares <shares@ndzh.com> Fri, 18 September 2020 11:54 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 5FE063A0544; Fri, 18 Sep 2020 04:54:37 -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 9xPSQYaDwj30; Fri, 18 Sep 2020 04:54:36 -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 9CEA73A0475; Fri, 18 Sep 2020 04:54:35 -0700 (PDT)
X-Default-Received-SPF: pass (skip=loggedin (res=PASS)) x-ip-name=174.25.170.45;
From: "Susan Hares" <shares@ndzh.com>
To: "'Alvaro Retana'" <aretana.ietf@gmail.com>, "'John Scudder'" <jgs@juniper.net>
Cc: <idr-chairs@ietf.org>, "'idr@ietf. org'" <idr@ietf.org>, <draft-ietf-idr-tunnel-encaps@ietf.org>
References: <CAMMESsw09LGWWhqyJ_0=jRimUN+_UuCjaXHCdqF9zkpaxSQgVQ@mail.gmail.com> <4A8D5D53-B3D2-4ACC-876C-F70D2B2EDC46@juniper.net> <CAMMESszqiNNoGpW_=39Jo5UNckFFrWU8=zTk5O2NGTFwnr7=QQ@mail.gmail.com> <612D7795-D53A-498E-BB5E-F231A32954F2@juniper.net> <CAMMESsxNgyz15kqzog4JeoDXe8Gf+ZFSrWZaOGt9tuUJP2msTQ@mail.gmail.com>
In-Reply-To: <CAMMESsxNgyz15kqzog4JeoDXe8Gf+ZFSrWZaOGt9tuUJP2msTQ@mail.gmail.com>
Date: Fri, 18 Sep 2020 07:54:29 -0400
Message-ID: <006701d68db2$7786b3e0$66941ba0$@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: AQJH7ekTdpypUgfVEnnZV4D7E+BXwwLfcsRDArs4M+YC6px+LwMmwv8GqC4orAA=
Content-Language: en-us
X-Antivirus: AVG (VPS 200918-0, 09/18/2020), Outbound message
X-Antivirus-Status: Not-Tested
X-Authenticated-User: skh@ndzh.com
Archived-At: <https://mailarchive.ietf.org/arch/msg/idr/gIVUe6M8v92ENfBwnwJ9cfJgHJQ>
Subject: Re: [Idr] AD Review of draft-ietf-idr-tunnel-encaps-15
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: Fri, 18 Sep 2020 11:54:38 -0000

John and Alvaro: 

Do you want me to do a complete new shepherd review at this point?  

Cheers, Sue 

-----Original Message-----
From: Alvaro Retana [mailto:aretana.ietf@gmail.com] 
Sent: Thursday, September 17, 2020 6:18 PM
To: John Scudder
Cc: idr-chairs@ietf.org; idr@ietf. org; draft-ietf-idr-tunnel-encaps@ietf.org
Subject: Re: AD Review of draft-ietf-idr-tunnel-encaps-15

On September 11, 2020 at 11:25:31 AM, John Scudder wrote:

John:

Hi!

Just a couple of comments...  Just one actionable change related to the reachability issue: you're right, we should move on.

Thanks!

Alvaro.



...
> > ...
> >> We added 5566 and 5640 to the list of obsoleted RFCs, along with an 
> >> explanation.
> >
> > [major] The IANA assignments made for those RFCs need to be declared 
> > obsolete in the IANA Considerations section.
>
> Is “obsolete” an IANA thing? I’ve added a subsection to the IANA 
> section as requested, but asked them to mark the code points as “deprecated”.


Yes, "obsolete" is an IANA thing. rfc8126: "Specific entries in a registry can be marked as "obsolete" (no longer in use) or "deprecated" (use is not recommended)."

You used both terms in §13.1 to refer to the code points.  You're right, deprecating is the right action, but IANA may be confused by the use of both.  This is a minor point, so we can wait for them to ask about it. :-)



...
> >>> 1054 4.2. Router's MAC Extended Community
> >
> > [] We've been talking about this section in a separate message...
>
> OK. To loop the WG in on the conclusion of that, it appears that 
> draft-ietf-bess-evpn-inter-subnet-forwarding only allows a single 
> Router’s MAC (authors have been asked to clarify) so assuming that’s 
> right, we must pick one or the other if it’s encoded twice in two ways 
> with different values. In -18 I’ve changed the text to say “use the 
> EC” instead of the former “use the sub-TLV”.

FYI...

I have a DISCUSS on draft-ietf-bess-evpn-inter-subnet-forwarding
precisely due to the lack of sync with this document.  I hope that the authors provide an answer and align with the changes here.




> >>> 1080 5. Semantics and Usage of the Tunnel Encapsulation attribute
...

> All of this is true. Basically, we can either hang progress of this 
> document up for a potentially long time waiting for something that may 
> never come: a detailed specification of exactly when a route is, and 
> isn’t, resolvable, or we can admit it’s an imperfect world and move 
> forward. I will observe that despite 4271’s lack of prescriptiveness, 
> somehow we have many interoperable implementations of BGP. I suspect 
> this is because when an implementor looks at route resolution, it’s a 
> little like Justice Potter Stewart looking at other material: “[I 
> can’t define it but] I know it when I see it.”
>
> I tried to make the paragraph clearer by adding a final clause. The 
> last sentence now reads, "The reachability condition is evaluated as 
> per [RFC4271], but the essence is that if the router could forward a 
> packet addressed to the IP address, the IP address is "reachable”.” We 
> could absolutely pick this short summary to shreds if we chose — see 
> above. It’s not meant to be prescriptive, only descriptive.


Ok.

Let's go with an "intermediate" approach, so that we don't try to say too much, but (hopefully) say enough:

NEW>
   *  The tunnel is specified in a TLV whose Tunnel Egress Endpoint sub-TLV
      identifies an IP address that is reachable.  The reachability condition
      is evaluated as per [RFC4271].  If the IP address is reachable via
      more than one forwarding table, local policy is used to determine
      which table to use.