Re: [Tsv-art] Tsvart last call review of draft-ietf-idr-te-pm-bgp-15

"Susan Hares" <shares@ndzh.com> Fri, 14 December 2018 22:36 UTC

Return-Path: <shares@ndzh.com>
X-Original-To: tsv-art@ietfa.amsl.com
Delivered-To: tsv-art@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 1EAE613133D; Fri, 14 Dec 2018 14:36:35 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: 0.947
X-Spam-Level:
X-Spam-Status: No, score=0.947 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DOS_OUTLOOK_TO_MX=2.845, HTML_MESSAGE=0.001, RCVD_IN_DNSWL_NONE=-0.0001, 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 2qKVvbzZH131; Fri, 14 Dec 2018 14:36:33 -0800 (PST)
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 2DD33130EB5; Fri, 14 Dec 2018 14:36:32 -0800 (PST)
X-Default-Received-SPF: pass (skip=loggedin (res=PASS)) x-ip-name=166.177.57.147;
From: Susan Hares <shares@ndzh.com>
To: 'Alvaro Retana' <aretana.ietf@gmail.com>, "'Les Ginsberg (ginsberg)'" <ginsberg@cisco.com>, 'Yoshifumi Nishida' <nishida@sfc.wide.ad.jp>
Cc: draft-ietf-idr-te-pm-bgp.all@ietf.org, tsv-art@ietf.org, idr@ietf.org, nishida@wide.ad.jp, ietf@ietf.org
References: <154469190410.2732.7123292408392294701@ietfa.amsl.com> <d07090284c9f46f48c5d479297b4a865@XCH-ALN-001.cisco.com> <CAO249yd5v3pDuOgLqmVsp2yr+zfXWJyvSx6b67kM21Tusza7DQ@mail.gmail.com> <9eff1b233b484c9cb6dae8b64dff6d89@XCH-ALN-001.cisco.com> <006a01d493bc$31912ec0$94b38c40$@ndzh.com> <CAMMESswwkKCQohdw65RYhDy1nWUr3CYW3FDAZhkevKyHrDO7AA@mail.gmail.com>
In-Reply-To: <CAMMESswwkKCQohdw65RYhDy1nWUr3CYW3FDAZhkevKyHrDO7AA@mail.gmail.com>
Date: Fri, 14 Dec 2018 17:36:28 -0500
Message-ID: <005801d493fd$753cb9f0$5fb62dd0$@ndzh.com>
MIME-Version: 1.0
Content-Type: multipart/alternative; boundary="----=_NextPart_000_0059_01D493D3.8C68FBE0"
X-Mailer: Microsoft Outlook 14.0
Thread-Index: AQDZ28V8NsfnFsTpD4hdo6VWIHblPAMBQV0tAVeIPmcBkr5qoQJinBErAl5TQgKnHtxB0A==
Content-Language: en-us
X-Antivirus: AVG (VPS 181214-6, 12/14/2018), Outbound message
X-Antivirus-Status: Not-Tested
X-Authenticated-User: skh@ndzh.com
Archived-At: <https://mailarchive.ietf.org/arch/msg/tsv-art/LaB50d1Ols5c0HVvhJbr_SF9TEE>
Subject: Re: [Tsv-art] Tsvart last call review of draft-ietf-idr-te-pm-bgp-15
X-BeenThere: tsv-art@ietf.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: Transport Area Review Team <tsv-art.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/tsv-art>, <mailto:tsv-art-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/tsv-art/>
List-Post: <mailto:tsv-art@ietf.org>
List-Help: <mailto:tsv-art-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/tsv-art>, <mailto:tsv-art-request@ietf.org?subject=subscribe>
X-List-Received-Date: Fri, 14 Dec 2018 22:36:35 -0000

Alvaro: 

 

Yes – this is the format I was indicating – 

In some BGP-LS document I’ve seen tables that map the BGP-LS TLVs to the corresponding ISIS/OSPF TLVs — with the understanding that the format is different.  One such example is in §2.4/2.5 of draft-ietf-idr-bgp-ls-segment-routing-ext [1].  Is that the alignment you’re asking about?  If so, then I think that such a table would be useful.

 

This makes the implementations easier to create.  More importantly helps those who deploy the protocol additions to understand how to debug issues.   Having started out my Internet career as a OPS person, I like to see things that make life easier for the OPS/NM people.  We can comments in each draft or put pointers in a single draft (living or RFC).  … just wondering what would help.. 

 

I agree with your ADs comment on the text – that said 

 

“The semantic of the TLV is described in [RFC7810] and [RFC7471]” (in -14) to "The semantics of the value field in the TLV are described in [I-D.ietf-lsr-isis-rfc7810bis] and [RFC7471]” (in -15)…and the text about "TLV formats follow the rules defined in [RFC7752]” was added, as Les pointed out.  

 

Your comment makes the text correct. Thanks for catching that point. I thought Les already agreed to that change – so I did “ack” it or expand on it.   

 

 

Sue 

 

 

 

From: Alvaro Retana [mailto:aretana.ietf@gmail.com] 
Sent: Friday, December 14, 2018 12:55 PM
To: Susan Hares; Les Ginsberg (ginsberg); Yoshifumi Nishida
Cc: draft-ietf-idr-te-pm-bgp.all@ietf.org; tsv-art@ietf.org; idr@ietf.org; nishida@wide.ad.jp; ietf@ietf.org
Subject: RE: Tsvart last call review of draft-ietf-idr-te-pm-bgp-15

 

On December 14, 2018 at 9:49:28 AM, Susan Hares (shares@ndzh.com) wrote:

 

Sue:

 

Hi!

 

I think we might be talking about two different things…or I got lost, which is completely possible. :-)

 

I think Yoshi is asking about the format of the TLVs, which happens to be the same as the OSPF ones (rfc7471), but slightly different than ISIS (rfc7810bis): the type and length fields are different.  I had already pointed this out in my AD review, so the text changed from "The semantic of the TLV is described in [RFC7810] and [RFC7471]” (in -14) to "The semantics of the value field in the TLV are described in [I-D.ietf-lsr-isis-rfc7810bis] and [RFC7471]” (in -15)…and the text about "TLV formats follow the rules defined in [RFC7752]” was added, as Les pointed out.  I think this is enough.

 

When you say "align with IGP TLVs”, are you talking about the format of the TLV, or are you referring to something else?

 

In some BGP-LS document I’ve seen tables that map the BGP-LS TLVs to the corresponding ISIS/OSPF TLVs — with the understanding that the format is different.  One such example is in §2.4/2.5 of draft-ietf-idr-bgp-ls-segment-routing-ext [1].  Is that the alignment you’re asking about?  If so, then I think that such a table would be useful.

 

Thanks!

 

Alvaro.

 

 

[1] https://tools.ietf.org/html/draft-ietf-idr-bgp-ls-segment-routing-ext-11#section-2.4 

 

 

 

You both raised two sides of the issue regarding the BGP-LS TLVs when these TLVs align with IGP TLVs. 

 

Side 1: It is helpful to note that the TLVs are the same

Side 2: It is confusing to have this information in individual drafts.    

 

This is not a single draft issue but a concern regarding several drafts in the BGP-LS series from IDR.

 

Yoshi -  Would it help the clarity of the BGP-LS series to have an explanatory draft that indicates the places to find the alignment?  Or should we suggest a note to IANA registration to provide clarity.  

 

Thank you for raising both sides of this issue.   Your IDR chairs and AD have been discussing how to make this easier to understand for implementers and those who deploy the BGP-LS code.