Re: [Pals] RtgDir review: draft-ietf-pals-p2mp-pw-02.txt

"Patrice Brissette (pbrisset)" <pbrisset@cisco.com> Thu, 11 May 2017 01:08 UTC

Return-Path: <pbrisset@cisco.com>
X-Original-To: pals@ietfa.amsl.com
Delivered-To: pals@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id D7C611294FD; Wed, 10 May 2017 18:08:10 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -14.522
X-Spam-Level:
X-Spam-Status: No, score=-14.522 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, RP_MATCHES_RCVD=-0.001, SPF_HELO_PASS=-0.001, SPF_PASS=-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 UsqEsTAmlm8V; Wed, 10 May 2017 18:08:08 -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 6B157128768; Wed, 10 May 2017 18:08:08 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=cisco.com; i=@cisco.com; l=39840; q=dns/txt; s=iport; t=1494464888; x=1495674488; h=from:to:cc:subject:date:message-id:references: in-reply-to:mime-version; bh=SNw2ZD+ZrJrN2Zk6fvznNK0IP+8a9WWBG/USyEw8GtI=; b=Z7yof2k0t5g+cfZZDvkr+T0uulqF9Ww6gdx47pWLbdcSAFO1nN3b6D8C Lp76mOQP36ZySU7jdNPoHI0jsqkIf5ID0Ytk6CJeQoputeQwZOn/sRtlA HbJD5T2MN+A1+0pmzSrsalnFH+W91IKwOgHgpwtFQnXB9VS9nOSvDVSOb c=;
X-IronPort-Anti-Spam-Filtered: true
X-IronPort-Anti-Spam-Result: A0BWAQBauBNZ/5ldJa1dGQEBAQEBAQEBAQEBBwEBAQEBgm5nYoEMAQaDYooYkTchlXSCDyyFeAIahGg/GAECAQEBAQEBAWsdC4UWBiMKTBACAQYCFCQBCQICAjAlAgQOiiUOlCCdYIImK4pLAQEBAQEBAQEBAQEBAQEBAQEBAQEBHYZfgV0BKwuCZYUDgnIvgjEFiUuUPwGHG4t/ggRVgQyDWooshjSODgEPEDiBCnAVWAGEYxyBY3eIC4ENAQEB
X-IronPort-AV: E=Sophos;i="5.38,322,1491264000"; d="scan'208,217";a="241973865"
Received: from rcdn-core-2.cisco.com ([173.37.93.153]) by rcdn-iport-7.cisco.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 11 May 2017 01:08:07 +0000
Received: from XCH-RTP-007.cisco.com (xch-rtp-007.cisco.com [64.101.220.147]) by rcdn-core-2.cisco.com (8.14.5/8.14.5) with ESMTP id v4B186Gp008971 (version=TLSv1/SSLv3 cipher=AES256-SHA bits=256 verify=FAIL); Thu, 11 May 2017 01:08:07 GMT
Received: from xch-rtp-009.cisco.com (64.101.220.149) by XCH-RTP-007.cisco.com (64.101.220.147) with Microsoft SMTP Server (TLS) id 15.0.1210.3; Wed, 10 May 2017 21:08:06 -0400
Received: from xch-rtp-009.cisco.com ([64.101.220.149]) by XCH-RTP-009.cisco.com ([64.101.220.149]) with mapi id 15.00.1210.000; Wed, 10 May 2017 21:08:06 -0400
From: "Patrice Brissette (pbrisset)" <pbrisset@cisco.com>
To: "draft-ietf-pals-p2mp-pw@ietf.org" <draft-ietf-pals-p2mp-pw@ietf.org>
CC: "pals-chairs@ietf.org" <pals-chairs@ietf.org>, "pals@ietf.org" <pals@ietf.org>, Jonathan Hardwick <Jonathan.Hardwick@metaswitch.com>, Min Ye <amy.yemin@huawei.com>, "db3546@att.com" <db3546@att.com>
Thread-Topic: RtgDir review: draft-ietf-pals-p2mp-pw-02.txt
Thread-Index: AQHSye+FRsEKCBWmy0+OMqPB6EOaH6HuUjeA
Date: Thu, 11 May 2017 01:08:05 +0000
Message-ID: <D0BAABDE-58B6-4343-AFA8-674DF294BA29@cisco.com>
References: <34EAD12F-59EB-4CFE-B237-ABE5C18D1C56@cisco.com>
In-Reply-To: <34EAD12F-59EB-4CFE-B237-ABE5C18D1C56@cisco.com>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
user-agent: Microsoft-MacOutlook/f.21.0.170409
x-ms-exchange-messagesentrepresentingtype: 1
x-ms-exchange-transport-fromentityheader: Hosted
x-originating-ip: [10.82.178.18]
Content-Type: multipart/alternative; boundary="_000_D0BAABDE58B64343AFA8674DF294BA29ciscocom_"
MIME-Version: 1.0
Archived-At: <https://mailarchive.ietf.org/arch/msg/pals/3X_uKLntUwFVciOMoj9LvVz4uRk>
Subject: Re: [Pals] RtgDir review: draft-ietf-pals-p2mp-pw-02.txt
X-BeenThere: pals@ietf.org
X-Mailman-Version: 2.1.22
Precedence: list
List-Id: "Pseudowire And LDP-enabled Services dicussion list." <pals.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/pals>, <mailto:pals-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/pals/>
List-Post: <mailto:pals@ietf.org>
List-Help: <mailto:pals-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/pals>, <mailto:pals-request@ietf.org?subject=subscribe>
X-List-Received-Date: Thu, 11 May 2017 01:08:11 -0000

Hello,
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<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: draft-ietf-pals-p2mp-pw-02.txt
Reviewer: Patrice Brissette
Review Date: May 10, 2017
IETF LC End Date: May 12, 2017
Intended Status: Standard Track
Summary:

·         I have some minor concerns about this document that I think should be resolved before publication.

Comments:
·         Please supply an overview of the draft quality and readability.
·         Include anything else that you think will be helpful toward understanding your review.
Major Issues:
·         "No major issues found."
Minor Issues:

·         Technically, I think the draft is completed. However, it doesn’t flow very well. Information is all over. I suggest the authors to review the layout/flow of the document.


Here are my “detailed” comments:

Abstract — What is the plus value on that draft? No clear

Many Long sentences in the text. very hard to understand and follow. Syntax to be improved.

Introduction
Typo : “A reference model or a P2MP PW is depicted in Figure 1 below”

“In this document, we specify a method of signaling P2MP
   PW using LDP.” —> suggest to move it from intro to abstract

Also, make sure the 3rd person is used. Try to a void “we” usage

May I suggest to have a requirement section. Requirements are all over the document.

“   In case of mLDP, a Leaf-PE can decide to join the P2MP LSP at any
   time; whereas in the case of RSVP-TE, the P2MP LSP is set up by the
   R-PE, generally at the initial service provisioning time. It should
   be noted that local policy can override any decision to join, add or
   prune existing or new L-PE(s) from the tree. In any case, the PW
   setup can ignore these differences, and simply assume that the P2MP
   PSN LSP is available when needed
“
Quite complex to follow. Missing to “why” / explanation.

“The LDP liberal label retention mode is used“
Another requirement… is that a MAY, SHOULD, MUST?

“In this case, a PW status message with status
   code of 0x00000008 (Local PSN-facing PW (ingress) Receive Fault) MUST
   also be sent to the R-PE“
How? The L-PE fails to join the P2MP PSN LSP.

Section 2.2
“   Note that since the LDP label mapping message is only sent by the R-
   PE to all the L-PEs, it is not possible to negotiate any interface
   parameters.“
Why is that note there? Is that already been mentioned previously. Fig.4 must  be moved to proper in the text OR create 2 subsection in 2.2

“As such, PW status negotiation procedure
   described in [RFC4447bis] is not applicable to P2MP PW. A node MUST
   NOT claim to be  P2MP PW capable by sending a LDP P2MP PW Capability
   TLV  if it is not also capable of handling PW status“

Should a node send LDP P2MP PW Capability TLV or not? Not well explain

There is some reference to LSR in the text where the major part use the wording “node”.

Nits:
N/A

Regards,
Patrice Brissette