Re: [Lsr] [RTG-DIR] RtgDir Review: draft-ietf-isis-segment-routing-msd-15

Julien Meuric <julien.meuric@orange.com> Fri, 05 October 2018 16:18 UTC

Return-Path: <julien.meuric@orange.com>
X-Original-To: lsr@ietfa.amsl.com
Delivered-To: lsr@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id AE30A130E47; Fri, 5 Oct 2018 09:18:46 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: 3.826
X-Spam-Level: ***
X-Spam-Status: No, score=3.826 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, GB_SUMOF=5, HTML_MESSAGE=0.001, MIME_HTML_ONLY=0.723, SPF_FAIL=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 91U14lb3dzJg; Fri, 5 Oct 2018 09:18:43 -0700 (PDT)
Received: from p-mail-ext.rd.orange.com (p-mail-ext.rd.orange.com [161.106.1.9]) by ietfa.amsl.com (Postfix) with ESMTP id B1066130E34; Fri, 5 Oct 2018 09:18:42 -0700 (PDT)
Received: from p-mail-ext.rd.orange.com (localhost [127.0.0.1]) by localhost (Postfix) with SMTP id 4AD2A561575; Fri, 5 Oct 2018 18:18:19 +0200 (CEST)
Received: from p-mail-int.rd.francetelecom.fr (p-mail-int.rd.francetelecom.fr [10.192.117.12]) by p-mail-ext.rd.orange.com (Postfix) with ESMTP id 37E5B5613CE; Fri, 5 Oct 2018 18:18:19 +0200 (CEST)
Received: from p-mail-int.rd.francetelecom.fr (localhost.localdomain [127.0.0.1]) by localhost (Postfix) with SMTP id 4AA281812CC; Fri, 5 Oct 2018 18:18:21 +0200 (CEST)
Received: from FTRDCH01.rd.francetelecom.fr (ftrdch01.rd.francetelecom.fr [10.194.32.11]) by p-mail-int.rd.francetelecom.fr (Postfix) with ESMTP id 2D8D6181272; Fri, 5 Oct 2018 18:18:21 +0200 (CEST)
Received: from [10.193.71.187] (10.193.71.187) by FTRDCH01.rd.francetelecom.fr (10.194.32.11) with Microsoft SMTP Server id 14.3.408.0; Fri, 5 Oct 2018 18:18:20 +0200
To: "Les Ginsberg (ginsberg)" <ginsberg@cisco.com>
CC: "bruno.decraene@orange.com" <bruno.decraene@orange.com>, "Acee Lindem (acee)" <acee@cisco.com>, Benjamin Kaduk <kaduk@mit.edu>, Routing Directorate <rtg-dir@ietf.org>, "draft-ietf-isis-segment-routing-msd@ietf.org" <draft-ietf-isis-segment-routing-msd@ietf.org>, "rtg-ads@ietf.org" <rtg-ads@ietf.org>, Alvaro Retana <aretana.ietf@gmail.com>, "lsr@ietf.org" <lsr@ietf.org>, Jeff Tantsura <jefftant.ietf@gmail.com>
References: <F4955ED6-1FC9-4BD9-A3AE-CEAE3C474586@cisco.com> <20010_1538575635_5BB4CD13_20010_244_10_71d2bce7-864e-4292-a6fa-94313fb56b0e@OPEXCLILMA2.corporate.adroot.infra.ftgroup> <107C953C-9823-4EA5-8901-D2B6CECCA253@cisco.com> <3fcd73d1337744c1a2dde556b2e2d5ea@XCH-ALN-001.cisco.com> <13052_1538639771_5BB5C79B_13052_444_2_8a894fbb-2988-49e9-8c97-d0596b738e2d@OPEXCLILM34.corporate.adroot.infra.ftgroup> <fc397e121c144f50ba42c92aee072d87@XCH-ALN-001.cisco.com>
From: Julien Meuric <julien.meuric@orange.com>
Organization: Orange
Message-ID: <5da32c65-0144-d965-92b7-cffb3a132c58@orange.com>
Date: Fri, 05 Oct 2018 18:18:20 +0200
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.9.1
MIME-Version: 1.0
In-Reply-To: <fc397e121c144f50ba42c92aee072d87@XCH-ALN-001.cisco.com>
Content-Type: text/html; charset="utf-8"
Content-Language: en-US
Content-Transfer-Encoding: 8bit
Archived-At: <https://mailarchive.ietf.org/arch/msg/lsr/euL4DNHaKaz2g5OlZIUHZWWDJuM>
Subject: Re: [Lsr] [RTG-DIR] RtgDir Review: draft-ietf-isis-segment-routing-msd-15
X-BeenThere: lsr@ietf.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: Link State Routing Working Group <lsr.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/lsr>, <mailto:lsr-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/lsr/>
List-Post: <mailto:lsr@ietf.org>
List-Help: <mailto:lsr-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/lsr>, <mailto:lsr-request@ietf.org?subject=subscribe>
X-List-Received-Date: Fri, 05 Oct 2018 16:18:47 -0000

Hi Les,

Indeed, this version resolves my initial comments.

Thank you.

Julien


Oct. 05, 2018 - ginsberg@cisco.com:

Bruno/Julien/Benjamin –

 

V18 of the draft has been published. I believe this addresses all outstanding comments.

 

Thanx very much for your input.

 

    Les

 

 

From: bruno.decraene@orange.com <bruno.decraene@orange.com>
Sent: Thursday, October 04, 2018 12:56 AM

 

Les,

 

Thanks for the proposed text.

It’s crystal clear. It works for me.

 

--Bruno

 

From: Les Ginsberg (ginsberg) [mailto:ginsberg@cisco.com]
Sent: Wednesday, October 03, 2018 11:27 PM

 

(Hard to follow Acee’s post – especially for entertainment value)

 

Bruno –

 

I think that we do want some less awkward text. So I am proposing to add the following into the Introduction:

 

“Label Imposition is the act of modifying and/or adding labels to the outgoing label stack associated with a packet.

This includes:

 

o  replacing the label at the top of the label stack with a new label

o  pushing one or more new  labels onto the label stack.

 

The number of labels imposed is then the sum of the labels which are replaced and the labels which are pushed.

See [RFC3031] for further details.”

 

The BMI definition then becomes:

 

“Base MPLS Imposition MSD (BMI-MSD) signals the total number of MPLS

   labels which can be imposed, including all service/transport/special

   labels.”

 

Does this work??

 

    Les

 

 

From: Acee Lindem (acee)
Sent: Wednesday, October 03, 2018 2:05 PM

 

Hi Bruno, 

 

On Oct 3, 2018, at 10:07 AM, bruno.decraene@orange.com wrote:

 

Hi Acee,

 

 

From: Acee Lindem (acee) [mailto:acee@cisco.com] 

 

Hey Bruno, Jeff, Les,

 

Have we agreed on the precise definition of “label imposition”?

Thanks for asking.

Not so far.

We don’t necessarily need to agree on a precision definition of “label imposition”. In my latest email (a few hours ago), I proposed to reuse the phrasing from RFC 3031, which does not use that term. If we are fine with using RFC 3031 terms, that would be fine for me.

 

Since the MSD type has always been defined in terms of “Imposition” in both the OSPF and IS-IS MSD drafts, I think it would be better to clarify any ambiguities the text Les quotes below.

 

Of course, we don’t want to get too bogged down in semantics as has happened in the past: https://www.youtube.com/watch?v=-P8IYKxpqG0" rel="nofollow">https://www.youtube.com/watch?v=-P8IYKxpqG0

 

Thanks,

Acee

 

 

 

Thanks,

--Bruno

 

 

Thanks,

Acee

 

From: Lsr <lsr-bounces@ietf.org> on behalf of Bruno Decraene <bruno.decraene@orange.com>
Date: Wednesday, October 3, 2018 at 4:37 AM

 

 

From: Jeff Tantsura [mailto:jefftant.ietf@gmail.com] 
Sent: Tuesday, October 02, 2018 8:28 PM

 

Gents,

 

I’m 100% with Les here, going into platform/asic specifics within this document would inevitably create ambiguity. 

Absolutely.

And nobody is asking for this.

 

Cheers

--Bruno

 

 


Cheers, 

Jeff

On Oct 2, 2018, 11:20 AM -0700, Les Ginsberg (ginsberg) <ginsberg@cisco.com>, wrote:

Bruno –

 

Trimming the thread…

 

[Les2:] Label imposition is meant to cover both the SWAP operation and the PUSH operation. In the example you provided above where a label stack of “12” is replaced by a label stack of “14,15” the number of labels “imposed” is 2.

[Bruno2] In that case, I definitely think that the discussion was useful and that this point needs to be clarified in the document.

Whether you choose to call that (1 POP, 2 PUSH) or (1 SWAP, 1 PUSH)  or simply a SWAP isn’t relevant here (though it might matter to folks like the RFC 3031 authors).

 

With that ibn mind, here is proposed text:

 

“Base MPLS Imposition MSD (BMI-MSD) signals the total number of MPLS

   labels which can be imposed, including all service/transport/special

   labels.  Imposition includes swap and/or push operations.

 

If the advertising router performs label imposition in the context of

   the ingress interface, it is not possible to meaningfully advertise

   per link values.  In such a case only the Node MSD SHOULD be

   advertised.”

 

[Bruno2] Given that the term “imposition” does not seem to be defined within the IETF, I would still favor a formal definition not using it. e.g. “BMI-MSD advertises the ability to increase the depth of the label stack by BMI-MSD labels”.

Alternatively, I’d propose the following rewording which seems clearer to me:

OLD: Imposition includes swap and/or push operations.

NEW: A swap operation counts as an imposition of one label; just like one push operation.

 

[Les3:] This gets into implementation specific issues that I would really like to avoid.

For example, some implementations perform one and only one  “operation”. Conceptually that may involve a swap and a push – but from the internal implementation POV it is simply one operation. And this may be true regardless of how many labels are involved. Other implementations might perform this in several discrete steps. The language we use here should not imply anything about how many labels are associated with a specific operation.

 

The term “increase” isn’t accurate because in the case of a swap there is no increase, yet the label which is replaced is counted.

 

 

The term “imposition” is generic – and as Alvaro has pointed out is used in RFC 4221. And the language proposed above does define the relationship between “swap and push” and “imposition”.

 

I appreciate your desire for clarity – and I am still open to new language – but at this point I still think what I proposed is  the most accurate.

 

   Les

 

 

 

Thanks,

--Bruno