Re: [mpls] [spring] RtgDir review: https://www.ietf.org/id/draft-ietf-mpls-spring-lsp-ping-06.txt

"Nagendra Kumar Nainar (naikumar)" <naikumar@cisco.com> Thu, 21 September 2017 23:57 UTC

Return-Path: <naikumar@cisco.com>
X-Original-To: mpls@ietfa.amsl.com
Delivered-To: mpls@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 207AC13336A; Thu, 21 Sep 2017 16:57:31 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -14.509
X-Spam-Level:
X-Spam-Status: No, score=-14.509 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, HTML_MESSAGE=0.001, RCVD_IN_DNSWL_HI=-5, RCVD_IN_MSPIKE_H3=-0.01, RCVD_IN_MSPIKE_WL=-0.01, SPF_PASS=-0.001, T_FILL_THIS_FORM_SHORT=0.01, URIBL_BLOCKED=0.001, USER_IN_DEF_DKIM_WL=-7.5] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (1024-bit key) header.d=cisco.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 VG-fTUsBI3_7; Thu, 21 Sep 2017 16:57:21 -0700 (PDT)
Received: from rcdn-iport-7.cisco.com (rcdn-iport-7.cisco.com [173.37.86.78]) (using TLSv1.2 with cipher DHE-RSA-SEED-SHA (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 38042133352; Thu, 21 Sep 2017 16:57:09 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=cisco.com; i=@cisco.com; l=57198; q=dns/txt; s=iport; t=1506038229; x=1507247829; h=from:to:cc:subject:date:message-id:references: in-reply-to:mime-version; bh=S3LH8AEK60LOIWWbg0re2pA9NuBauPmiMGjy9o4gWaA=; b=AsJIRuCz9ScwyVaz2CCT07ldiCVTne17uUUcpuZaY+dgh+4+LrIXKNCP 5j7p7JX0z+2AH1LX7yg+IHLHr3LeYDwCtTf8GNTnsI4Mma176a/+bABv7 JPdn+yAjPCPkQJS59C1gNxWaOu01kHdHP2BTz7RJoXs6t+4uSadwenCmI 8=;
X-IronPort-Anti-Spam-Filtered: true
X-IronPort-Anti-Spam-Result: A0CpAAD2UMRZ/4cNJK1bGQEBAQEBAQEBAQEBBwEBAQEBgm8+LWRuJweDb4ogj3uBUiJ5lTEOggQKI4UYAhqDej8YAQIBAQEBAQEBayiFGAEBAQEDGglWDgICAQgRAwEBASEBBgMCAgIUHBQJCAIEDgUbiTRkEKdPgicnilsBAQEBAQEBAQEBAQEBAQEBAQEBAQEdBYMmggKDNSsLgWVYNYRIAQsGAgEGHxAJFgKCWy+CMQWKBwGJF4URF4hMAodbgRWLZoITW4UQg36BK4VXigWLDAIRGQGBOAEPEDhBQQt3FR88AYUGHBkYgTZ2AYkdAiQHgQWBEAEBAQ
X-IronPort-AV: E=Sophos;i="5.42,426,1500940800"; d="scan'208,217";a="296601657"
Received: from alln-core-2.cisco.com ([173.36.13.135]) by rcdn-iport-7.cisco.com with ESMTP/TLS/DHE-RSA-AES256-SHA; 21 Sep 2017 23:57:07 +0000
Received: from xch-rcd-011.cisco.com (xch-rcd-011.cisco.com [173.37.102.21]) by alln-core-2.cisco.com (8.14.5/8.14.5) with ESMTP id v8LNv744001632 (version=TLSv1/SSLv3 cipher=AES256-SHA bits=256 verify=FAIL); Thu, 21 Sep 2017 23:57:07 GMT
Received: from xch-rcd-015.cisco.com (173.37.102.25) by XCH-RCD-011.cisco.com (173.37.102.21) with Microsoft SMTP Server (TLS) id 15.0.1263.5; Thu, 21 Sep 2017 18:57:01 -0500
Received: from xch-rcd-015.cisco.com ([173.37.102.25]) by XCH-RCD-015.cisco.com ([173.37.102.25]) with mapi id 15.00.1263.000; Thu, 21 Sep 2017 18:57:01 -0500
From: "Nagendra Kumar Nainar (naikumar)" <naikumar@cisco.com>
To: Alexander Vainshtein <Alexander.Vainshtein@ecitele.com>
CC: "rtg-dir@ietf.org" <rtg-dir@ietf.org>, "draft-ietf-mpls-lsp-ping.all@ietf.org" <draft-ietf-mpls-lsp-ping.all@ietf.org>, "mpls@ietf.org" <mpls@ietf.org>, "rtg-ads@ietf.org" <rtg-ads@ietf.org>, "spring@ietf.org" <spring@ietf.org>, "Jonathan Hardwick (Jonathan.Hardwick@metaswitch.com)" <Jonathan.Hardwick@metaswitch.com>
Thread-Topic: [spring] RtgDir review: https://www.ietf.org/id/draft-ietf-mpls-spring-lsp-ping-06.txt
Thread-Index: AQHTMmGTKrYHjrFRHE+7WhCrFfdLIaK/p7EAgABu9wA=
Date: Thu, 21 Sep 2017 23:57:01 +0000
Message-ID: <D47670B9-4B4E-4862-A03A-0488725D0572@cisco.com>
References: <DBE838B2-3146-4EDD-8A27-6B06A7C0C3ED@cisco.com> <AM4PR03MB1713C3FB1B1764E15AFD557F9D660@AM4PR03MB1713.eurprd03.prod.outlook.com>
In-Reply-To: <AM4PR03MB1713C3FB1B1764E15AFD557F9D660@AM4PR03MB1713.eurprd03.prod.outlook.com>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
user-agent: Microsoft-MacOutlook/f.25.0.170815
x-ms-exchange-messagesentrepresentingtype: 1
x-ms-exchange-transport-fromentityheader: Hosted
x-originating-ip: [10.118.20.13]
Content-Type: multipart/alternative; boundary="_000_D47670B94B4E4862A03A0488725D0572ciscocom_"
MIME-Version: 1.0
Archived-At: <https://mailarchive.ietf.org/arch/msg/mpls/xmcJx3Mv-C5KXJyEVBNzihTJLNs>
Subject: Re: [mpls] [spring] RtgDir review: https://www.ietf.org/id/draft-ietf-mpls-spring-lsp-ping-06.txt
X-BeenThere: mpls@ietf.org
X-Mailman-Version: 2.1.22
Precedence: list
List-Id: Multi-Protocol Label Switching WG <mpls.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/mpls>, <mailto:mpls-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/mpls/>
List-Post: <mailto:mpls@ietf.org>
List-Help: <mailto:mpls-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/mpls>, <mailto:mpls-request@ietf.org?subject=subscribe>
X-List-Received-Date: Thu, 21 Sep 2017 23:57:31 -0000

Hi Sasha,

Thanks ☺

I will wait for your confirmation.

Thanks,
Nagendra

From: spring <spring-bounces@ietf.org> on behalf of Alexander Vainshtein <Alexander.Vainshtein@ecitele.com>
Date: Thursday, September 21, 2017 at 2:26 PM
To: Nagendra Kumar Nainar <naikumar@cisco.com>
Cc: "rtg-dir@ietf.org" <rtg-dir@ietf.org>, "draft-ietf-mpls-lsp-ping.all@ietf.org" <draft-ietf-mpls-lsp-ping.all@ietf.org>, "mpls@ietf.org" <mpls@ietf.org>, "rtg-ads@ietf.org" <rtg-ads@ietf.org>, "spring@ietf.org" <spring@ietf.org>, "Jonathan Hardwick (Jonathan.Hardwick@metaswitch.com)" <Jonathan.Hardwick@metaswitch.com>
Subject: Re: [spring] RtgDir review: https://www.ietf.org/id/draft-ietf-mpls-spring-lsp-ping-06.txt

Nagendra,
I will read the -08 version and send my formal comments in a few days.
Based on your inline comments, I hopefully will be able to confirm that all my concerns have been successfully resolved.

I also hope that 1-year extension of the IANA early allocations will suffice for the document to be published. It is definitely needed.

Regards,
Sasha

Office: +972-39266302
Cell:      +972-549266302
Email:   Alexander.Vainshtein@ecitele.com

From: Nagendra Kumar Nainar (naikumar) [mailto:naikumar@cisco.com]
Sent: Thursday, September 21, 2017 1:41 AM
To: Alexander Vainshtein <Alexander.Vainshtein@ecitele.com>; rtg-ads@ietf.org
Cc: Jonathan Hardwick (Jonathan.Hardwick@metaswitch.com) <Jonathan.Hardwick@metaswitch.com>; rtg-dir@ietf.org; draft-ietf-mpls-lsp-ping.all@ietf.org; spring@ietf.org; mpls@ietf.org
Subject: Re: [spring] RtgDir review: https://www.ietf.org/id/draft-ietf-mpls-spring-lsp-ping-06.txt

Hi Sasha,

Thanks for the comments. Please see below:

Hi,
I have been selected as the Routing Directorate reviewer for this draft. The Routing Directorate seeks to review all routing or routing-related drafts as they pass through IETF last call and IESG review, and sometimes on special request. The purpose of the review is to provide assistance to the Routing ADs. For more information about the Routing Directorate, please see ​http://trac.tools.ietf.org/area/rtg/trac/wiki/RtgDir

Although these comments are primarily for the use of the Routing ADs, it would be helpful if you could consider them along with any other IETF Last Call comments that you receive, and strive to resolve them through discussion or by updating the draft.

Document: https://www.ietf.org/id/draft-ietf-mpls-spring-lsp-ping-06.txt

Reviewer: Alexander (“Sasha”) Vainshtein

Review date: 20.09.2017

IETF LC End Date: Not known

Intended status: Standards Track

Summary:

I have significant concerns about this document and recommend that the Routing ADs discuss these issues further with the authors.

Comments:

I have noticed that there is an earlier review of the same version of  the draft<https://www.ietf.org/mail-archive/web/rtg-dir/current/msg03542.html>, and I fully agree with the previous reviewer regarding the overall impression and specific issues raised. I have tried to avoid reporting the same issues again and focus only on new issues.

This review have been done on a very tight schedule for me, and therefore I could have missed some issues and definitely many nits. The tight schedule has also limited my ability to discuss the draft with the authors in sufficient detail – but some interactions did took place. I would like to thank the authors for their cooperation.

Major Issues:


  1.  Section 7.4 “Segment ID Check” of the draft claims to update “the procedure defined in Step 6 of Section 4.4  of [RFC8029]”. However, I have failed to integrate the logic defined by the pseudocode in this section with the logic defined by the referenced pseudocode.
     *   I have suspected that the newly introduced logic should be part of the Egress FEC Validation logic defined in Section 4.4.1 of RFC 8029.  This suspicion has been acknowledged by the authors, and they plan to fix the wrong reference in the next revision of the draft.
<Nagendra> We have this addressed in the new revision. We have updated the section as 4.4.1.


     *   From my POV the best way to avoid possible misinterpretation by the implementers would be inclusion of the updated version of the pseudocode (with explicit marking of the new logic) directly in the document. I have suggested this to the authors, but I did not get their response so far.
<Nagendra> We addressed this one as well. We included the modified/updated version of Step 4 (from RFC 8029) and continued the procedure as Step 4a.


  1.  I agree with the previous reviewer regarding the mismatch between the (presumed) claim and the actual scope covered by the draft. To the best of my understanding, the authors have already agreed to fix this in the next revision of the draft.

<Nagendra> We changed the title to reflect the scope appropriately.

Minor Issues:


  1.  I wonder why the authors place such an emphasis on Service labels (which, AFAIK, have not been defined in any level of detail anywhere so far) – these labels are mentioned in Sections 4.2, 5 (that mentions them as “Service segments”)  and 8, while so many types of Segment IDs that are already well defined both in the SRPING WG documents and in the extensions of the relevant routing protocols are left out of scope of the draft.  The authors have acknowledged this and plan to fix this issue (probably by removing the Service labels completely).
<Nagendra> As we discussed offline, this section was included as part of the merge of the two individual drafts. To avoid any confusion and stay within the scope of this draft, we removed any discussion about service label/segment.


  1.  The description of possible data plane malfunctions in section 4.1 seems to assume that all the nodes in the referenced topology (shown in Figure 1) us the same SRGB. If this assumption (which is not explicitly mentioned) is not correct, the LSP Ping Echo request packets would not be “delivered to their expected destinations but not via the expected path” (as the text in this section claims). I suggest making any such assumptions explicit in the text

<Nagendra> Does the below updated text clarifies?:

OLD:
                   Assume in above topology, R1 sends traffic with segment stack as
   {9124, 5008} so that the path taken will be R1-R2-R4-R5-R7-R8.  If
   the Adjacency Segment ID 9124 is misprogrammed in R2 to send the
   packet to R1 or R3, it will still be delivered to R8 but is not via
   the expected path.

NEW:
                   Assume in above topology, R1 sends traffic with segment stack as
   {9124, 5008} so that the path taken will be R1-R2-R4-R5-R7-R8.  If
   the Adjacency Segment ID 9124 is misprogrammed in R2 to forward the
   packet to R1 or R3, the packet may still be delivered to R8 (if the nodes are
 configured with same SRGB)but is not via the expected path.



  1.   I am not sure if “OSPF” and ISIS” (without any reference to Segment Routing) are the proper names for the label distribution protocols in the proposed new IANA registry in Section 10.2. From my POV “OSPF/ISIS SR Extensions” would be more appropriate. The authors disagree with this proposal claiming that OSPF and ISIS are not used for label distribution in any other way. (RFC 8029 that encodes the label distribution protocol in the Downstream  Detailed Mapping TLV did not request a dedicated registry for this purpose, and I think that the authors have been right to request such a registry).
<Nagendra> As we discussed, we are not aware of the use of OSPF/ISIS for label distribution other than SR. I am not sure if there is a need to differentiate BGP as signaling protocol for RFC3107 vs ePE. We may need to guage it and update protocol BGP if required. If the WG and/or AD is favor of renaming it as “OSPF/ISIS SR Extensions”, we can definitely update it accordingly.



  1.  I doubt the draft needs address any FRR issues even in future (as mentioned in the last statement in Section 5), since, to the best of my understanding, any form of FRR:
     *   Is operated locally by the node that is upstream to the failure without passing any information to the initiator of LSP-Ping Request packets
     *   In any case is just a transient state in the (presumably short) interval between the failure being detected by the upstream node and re-convergence of IGP

<Nagendra> Any discussion related to FRR has been removed already.

I also think that references to “future versions” are not quite appropriate in the document that is going to the IETF LC. I recommend removing any such references from the document,

<Nagendra> Service segment is the one which has some reference to future versions or document. With the updated revision, we cleared all references to any future versions.


  1.  Section 9 “Backward Compatibility with non-Segment Routing devices” defines the behavior for the two slightly different use cases:
     *   LSP Ping Echo Request packets are initiated by an SR-capable node (and therefore their target FEC stack contains SR-related FECs), while the responder is legacy device that is not SR-capable. In this case the proposed solution (respond with the FEC-return-code “Replying router has no mapping for the FEC at stack-depth”) looks as the only possibility since the legacy device cannot parse SR FECs in any case
     *   LSP Ping Echo Request packets are initiated by a legacy device while the responder is SR-capable. The draft defines the same behavior in this case – but, IMHO and FWIW, the responder could provide slightly better diagnostics since it can parse the “old” target FEC and recognize the equivalent Node ID. An additional return code would be required to implement this behavior.

<Nagendra> In this case, I believe the legacy node (Initiator) will not be able to parse the new return code received. I am not sure if it will be helpful.

This issue has not been discussed with the authors due to lack of time.
Nits:


  1.  The note to the RFC Editor in Section 10 mentions early IANA allocation for some Sub-TLV types defined in the document – but, as of today, these early allocations have already expired (they have been in force until 15.09.17).  I have to admit that I do not fully understand the implications of this fact – e.g., whether the authors may continue to refer to the specific values of parameters (e.g., Sub-TLV 34, 35 etc.) for which early IANA allocation has expired, or should use some other form of reference (e.g., TBDx).


<Nagendra> Loa and Amanda helped us to renew this allocation for another year. So I think the allocated value is valid for another year.


  1.  The draft mentions a TBD7 value to be assigned by IANA, but there are no TBD1, …, TBD6. The authors have acknowledged this and plan to fix it. However, I do not know how this can be affected by expiration of early IANA allocations (see above)
<Nagendra> We changed this one to TBD1 ☺


  1.  I have failed to understand the intention of the authors regarding already mentioned Service labels from the following statement in section 8: “How a node treats Service label is outside the scope of this document and will be included in this or a different document later”. Of course, if the Service labels are removed from the draft, this sentence would hopefully disappear with its internal controversy.

 <Nagendra>  As mentioned earlier, this section and any related discussion about service segment has been removed ☺.

Once again, thanks a lot for the thorough review and comments.

Thanks,
Nagendra

Regards,
Sasha

Office: +972-3926302
Cell:      +972-549266302
Email:   Alexander.Vainshtein@ecitele.com<mailto:Alexander.Vainshtein@ecitele.com>


___________________________________________________________________________

This e-mail message is intended for the recipient only and contains information which is
CONFIDENTIAL and which may be proprietary to ECI Telecom. If you have received this
transmission in error, please inform us by e-mail, phone or fax, and then delete the original
and all copies thereof.
___________________________________________________________________________




___________________________________________________________________________

This e-mail message is intended for the recipient only and contains information which is
CONFIDENTIAL and which may be proprietary to ECI Telecom. If you have received this
transmission in error, please inform us by e-mail, phone or fax, and then delete the original
and all copies thereof.
___________________________________________________________________________