Re: [spring] AD Review of draft-ietf-spring-segment-routing-mpls-10
Alvaro Retana <aretana.ietf@gmail.com> Thu, 02 November 2017 22:49 UTC
Return-Path: <aretana.ietf@gmail.com>
X-Original-To: spring@ietfa.amsl.com
Delivered-To: spring@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 805EF13F6CB; Thu, 2 Nov 2017 15:49:18 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.998
X-Spam-Level:
X-Spam-Status: No, score=-1.998 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, FREEMAIL_FROM=0.001, HTML_MESSAGE=0.001, RCVD_IN_DNSWL_NONE=-0.0001, SPF_PASS=-0.001, UNPARSEABLE_RELAY=0.001] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (2048-bit key) header.d=gmail.com
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 jBNs6yp_1Vez; Thu, 2 Nov 2017 15:49:15 -0700 (PDT)
Received: from mail-ot0-x22c.google.com (mail-ot0-x22c.google.com [IPv6:2607:f8b0:4003:c0f::22c]) (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 4468513F6AC; Thu, 2 Nov 2017 15:49:12 -0700 (PDT)
Received: by mail-ot0-x22c.google.com with SMTP id u41so959507otf.12; Thu, 02 Nov 2017 15:49:12 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=from:in-reply-to:references:mime-version:date:message-id:subject:to :cc; bh=+mvziGoh8+7SdEn9f9aiGGP0F2fs9yRR1Kf6N2NmV6c=; b=nboyhJIjPofVR/3WecUvEBODj9/YGwVG4hb8sBjK7NPHmXeR/y2sY1nveINQ0xPyuv cb899fKGLPoDCDEYILLR59UcFhTLbgw+nC9FjNPnJyCWEsXZgfKq0UW4ZPMNZCYuYqaR swqPbWW6FgyXDwg68cyRW9T/fjdt7Xc229DcGGf3V7HmIWmooBMgOj+lrYFm8mTCmsX/ mp8fNMtQtqbMq05IqzPollZp9//1wUA0+2EaYI5vqXscbA/eA304HByUSO/fbTNH0iyM UB5wCkrbrPzVjilrT0HzN2HbACV9q7OuxnTjIm7vlzXGNxCSQf3f1lobOi2s/IfLcmo4 K/5g==
X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:in-reply-to:references:mime-version:date :message-id:subject:to:cc; bh=+mvziGoh8+7SdEn9f9aiGGP0F2fs9yRR1Kf6N2NmV6c=; b=mEdIAw6ycGDY5JFRooSjZ0DD2YGRP/bW2O4rcWuSNfTwIfBnGFRS1/v5wsqWtEu0O+ L806Z5A9ZJZTdJvWalfe1UEkEHTB9aGJAY7cIv/mpUvgMdaEjyvLFUG5V6ODPE9e0L0L 5NlR8TIWtnJ5+NQO6jgjhzgvNfCef4EPbbl7fuZLSaDNlCTBaSHYvWR1Cx4WTo/bjn8L IYPSo8CgJSejnUfy7l+x19gBkWo1FrhGFjeiej9rvl2gWj3g0QqPMGvJzn3udfHTRX/S tI6YY72KVozhudX0z+1iCRFRYLnyBLK6+07lCIqQMl0L0G/fU+434gZsMv33Wnkt1QnU p4Mw==
X-Gm-Message-State: AJaThX5MEA/lGlEwJRB+Y2J957ETp2YWAhvUoPeryDEUWtbavu3YQ5Ex kqoFStNeFEVDQ8t59k2NjhMCo+UhKuSeNe32AA8=
X-Google-Smtp-Source: ABhQp+T+mMuzKoPf3QQ3SV5LhqOmj+H1E5jgYcb7NTZlKWmMJOJ3yI0dG+WI9YgT4BD+mf4roDliRbou8tqg+UXxqUI=
X-Received: by 10.157.91.107 with SMTP id e40mr3036936otj.62.1509662951529; Thu, 02 Nov 2017 15:49:11 -0700 (PDT)
Received: from 1058052472880 named unknown by gmailapi.google.com with HTTPREST; Thu, 2 Nov 2017 18:49:10 -0400
From: Alvaro Retana <aretana.ietf@gmail.com>
In-Reply-To: <50d87236662f49bb8a81308dff6abbf5@XCH-RTP-020.cisco.com>
References: <74FB72FC-AD69-4EA3-ACA2-739168EE0A44@cisco.com> <aa2ed0f2b0a845cc83b16adda7566b37@XCH-ALN-001.cisco.com> <50d87236662f49bb8a81308dff6abbf5@XCH-RTP-020.cisco.com>
X-Mailer: Airmail (457)
MIME-Version: 1.0
Date: Thu, 02 Nov 2017 18:49:10 -0400
Message-ID: <CAMMESswqFYQe_=zp8cPsuvvxMBFxUEe3t3DvkL=ZnZhA7RZO0w@mail.gmail.com>
To: "Ahmed Bashandy (bashandy)" <bashandy@cisco.com>
Cc: Martin Vigoureux <martin.vigoureux@nokia.com>, "spring-chairs@ietf.org" <spring-chairs@ietf.org>, "draft-ietf-spring-segment-routing-mpls@ietf.org" <draft-ietf-spring-segment-routing-mpls@ietf.org>, SPRING WG <spring@ietf.org>
Content-Type: multipart/alternative; boundary="94eb2c1c21bcf4e0f7055d07cb87"
Archived-At: <https://mailarchive.ietf.org/arch/msg/spring/yqe0g1fejkL0nrT6D5JRASGDb_A>
Subject: Re: [spring] AD Review of draft-ietf-spring-segment-routing-mpls-10
X-BeenThere: spring@ietf.org
X-Mailman-Version: 2.1.22
Precedence: list
List-Id: "Stacked Tunnels for Source Routing \(STATUS\)." <spring.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/spring>, <mailto:spring-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/spring/>
List-Post: <mailto:spring@ietf.org>
List-Help: <mailto:spring-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/spring>, <mailto:spring-request@ietf.org?subject=subscribe>
X-List-Received-Date: Thu, 02 Nov 2017 22:49:19 -0000
On October 30, 2017 at 2:12:18 PM, Ahmed Bashandy (bashandy) ( bashandy@cisco.com ) wrote: Ahmed: Hi! How are you? ... The main questions/concerns that I have related to this document is not just for the authors, but for the Shepherd and the Chairs too. Q1. Why is this document on the Standards Track? From the Introduction: “This drafts describes how Segment Routing operates on top of the MPLS data plane.” Describes, yes. On the other hand, the Shepherd’s write-up says that it “specifies the generic functions of the architecture” – I don’t see a specification, just a description. As such, I think this document should be Informational. #Ahmed: The new version of the draft specifies many things that are applicable to instantiation of SR over MPLS I’ll take your answer as confirming that the old version (-10) wasn’t really specifying anything. For this new version (-11), can you please be specific on what these “many things” are? I see some new Normative Language in the new 2.x sub-sections. I have some specific comments on that: Q1.A. Section 2.2. (SID Representation in the MPLS Forwarding Plane): The MCC MUST ensure that any label value corresponding to any SID it installs in the forwarding plane follows the following rules: o The label value MUST be unique within the router on which the MCC is running. i.e. the label MUST only be used to represent the SID. o The label value MUST NOT be identical to or within the range of any reserved label value or range [reserved-MPLS <https://tools.ietf.org/html/draft-ietf-spring-segment-routing-mpls-11#ref-reserved-MPLS>], respectively. These seem to be new requirements for the MCCs. Given that the protocol extensions (and LDP) are defined (in Section 2) as MCCs, how are they supposed to follow these rules, specially the first one? As far as I can tell, the IGP extensions (for example) can carry Label/SID information from an advertising node, so I don’t know how a local MCC (remote to that advertising node, which is locally "installing forwarding entries in the MPLS data plane”) can guarantee what the label is used for (“only used to represent the SID”). Maybe I’m missing something and this is already specified somewhere else…?? The second rule is just what the MPLS Architecture already specifies, no nothing new, right? BTW, the link in the reserved-MPLS reference doesn’t work — a better reference might be rfc3032 or rfc7274. Q1.B. Section 2.3. (Segment Routing Global Block and Local Block): The following rules apply to the list of MPLS ranges representing the SRGB o The list of label ranges MUST only be used to instantiate global SIDs into the MPLS forwarding plane o Every range in the list of ranges specifying the SRGB MUST NOT cover or overlap with a reserved label value or range [reserved- MPLS], respectively. . . . Just like SRGB, the SRLB need not be a single contiguous range of label, except the SRGB MUST only be used to instantiate global SIDs into the MPLS forwarding plane. The first rule (and the text below them) points to the global nature of the SR *global* Block. The architecture document already says that "In SR-MPLS, SRGB is a local property of a node and identifies the set of local labels reserved for global segments.” Nothing new specified here. Q1.C. Section 2.4. (Mapping a SID Index to an MPLS label) introduces an algorithm to calculate the label value. Note that the architecture document now includes an algorithm (in Section 3.1.2) as well — the algorithm in this document doesn’t look to be the same, but even if it is, it would be confusing to specify the same thing in two places. Q1.D. The rest of the sub-sections seem to rehash forwarding behaviors, which, because of the fact that the MPLS architecture is not changing, seem to add nothing interesting or important to this document. Having said all that, I still don’t see a clear justification for this document to be in the Standards Track. Q2. Section 2. (MPLS Instantiation of Segment Routing) is the only one with any real content…but there are only a couple of things in it that are not in the Architecture document: the introduction of the SRLB, and some words about the index – both of which should be really explained in the Architecture document, and not here. I wonder what the value of publishing this document really is. What long-term archival value does it provide? #Ahmed: The long term plan is to move details of MPLS-specific specifications to this document and keep the architecture document more general The “long term plan” ?? What do you mean? The architecture document is already in IETF Last Call — are you saying that both documents still require more changes and are not ready for publication? I really hope that is not what you mean. In any case, you did not answer my question about the value of the document as is. No worries, the intent is not to stop publication, but to understand whether there is something I’m missing and the relationship to the proposed Status. Q3. I also have to wonder about the IPR declared for this document. If most of the information here is already defined, described or specified in draft-ietf-spring-segment-routing, should the IPR declaration apply to that document as well (or maybe instead of this one)? It is not the IETF’s role (including the WG) to discuss whether a piece of IPR is valid or not – I just want to make sure the disclosures apply to the right document. #Ahmed: We will make sure that all IPR is declared That was not my question! And declaring IPR at this point in time is very, very late (rfc6701). I didn’t find a discussion on the list about any of these points. I am concerned about all the text that has been added (which in my opinion doesn’t contribute much to the document), resulting in about a 70% increase the document length! I will wait for your answer to my points above before returning the document to the WG for a thorough review. I have some other specific comments below. Thanks! Alvaro. ... P3. Section 2. (MPLS Instantiation of Segment Routing) says that “a controller-driven network…MAY use the control plane to discover the available set of local SIDs”. The “MAY” implies that there is a choice (i.e. it is optional) and that other discovery mechanisms exist. What are those other choices? Note that earlier in this section you already wrote that IGPs are used for flooding the information. s/MAY/may #Ahmed: This document gives an example of a use of the SRLB. Listing all possible uses of the SRLB is certainly not within the scope of this document. I’m not sure what your answer has to do with my comment…but giving examples is a good reason to *not* use Normative language. In any case, that piece of text was not changed from: In a controller-driven network, some controllers or applications MAY use the control plane to discover the available set of local SIDs on a particular router. In such cases, the SRLB is advertised in the control plane (e.g., using [I-D.ietf-isis-segment-routing-extensions <https://tools.ietf.org/html/draft-ietf-spring-segment-routing-mpls-10#ref-I-D.ietf-isis-segment-routing-extensions>]). …to… In a controller-driven network, some controllers or applications MAY use the control plane to discover the available set of local SIDs on a particular router [I.D. filsfils-spring-segment- routing-policy]. The “MAY” is still there (which was the main point of the comment above), and you added a new reference to apparently illustrate the point that the control plane can be used "to discover the available set of local SIDs on a particular router”. Isn’t that already specified in the architecture document? Why do we need this new reference? ... P5. References. Please take a look at rfc8174 and update the “Requirements Language” and associated references. #Ahmed: I agree. I modified the paragraph about requirements language to conform to rfc8174 You forgot to include the rfc8174 reference. Nits N1. I think that the references to *-segment-routing-extensions are superfluous. BTW, the fourth paragraph of the Introduction uses a reference to *-segment-routing-extensions to point at ISIS/OSPF (the protocols!). #Ahmed: I agree. Now we have clear references to each IGP extensions separately The point was that I think all references to the extensions are superfluous — not a request to add them.
- [spring] AD Review of draft-ietf-spring-segment-r… Alvaro Retana (aretana)
- Re: [spring] AD Review of draft-ietf-spring-segme… Martin Vigoureux
- Re: [spring] AD Review of draft-ietf-spring-segme… Alvaro Retana (aretana)
- Re: [spring] AD Review of draft-ietf-spring-segme… Ahmed Bashandy (bashandy)
- Re: [spring] AD Review of draft-ietf-spring-segme… Ahmed Bashandy (bashandy)
- Re: [spring] AD Review of draft-ietf-spring-segme… Alvaro Retana
- Re: [spring] AD Review of draft-ietf-spring-segme… Alvaro Retana
- Re: [spring] AD Review of draft-ietf-spring-segme… Ahmed Bashandy (bashandy)