[RTG-DIR] RtgDir review: draft-ietf-teas-p2mp-loose-path-reopt-07.txt
"Joel M. Halpern" <jmh@joelhalpern.com> Wed, 23 November 2016 21:09 UTC
Return-Path: <jmh@joelhalpern.com>
X-Original-To: rtg-dir@ietfa.amsl.com
Delivered-To: rtg-dir@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 7F4C81296D6; Wed, 23 Nov 2016 13:09:42 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.702
X-Spam-Level:
X-Spam-Status: No, score=-2.702 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, RCVD_IN_DNSWL_LOW=-0.7, SPF_HELO_PASS=-0.001, SPF_PASS=-0.001] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (1024-bit key) header.d=joelhalpern.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 l3onRa1Sq-NZ; Wed, 23 Nov 2016 13:09:41 -0800 (PST)
Received: from maila2.tigertech.net (maila2.tigertech.net [208.80.4.152]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 30EB612948A; Wed, 23 Nov 2016 13:09:38 -0800 (PST)
Received: from localhost (localhost [127.0.0.1]) by maila2.tigertech.net (Postfix) with ESMTP id F3572247571; Wed, 23 Nov 2016 13:09:37 -0800 (PST)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=joelhalpern.com; s=1.tigertech; t=1479935378; bh=YyC0ggY/gelRqTbllYtt443brOZs4+9zW40ZiQFh7JQ=; h=To:Cc:From:Subject:Date:From; b=rHHZSUNMjyT+P/he9Hay3MHw4zR293H/wmkou540IwlhiXtI+GiORh+jOjTtjW7PF UplqlRQyHfsmuDDpJF6GkcpymS09k4BQUt6g4FUguNUUwQuZ5szR0N/e1qe57YZPUM WzuuAeyFLdT44kj75c0K0YRpWbSkYTs3OMgRRbyo=
X-Virus-Scanned: Debian amavisd-new at maila2.tigertech.net
Received: from Joels-MacBook-Pro.local (209-255-163-147.ip.mcleodusa.net [209.255.163.147]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by maila2.tigertech.net (Postfix) with ESMTPSA id 5D2A624B481; Wed, 23 Nov 2016 13:09:37 -0800 (PST)
To: "rtg-ads@tools.ietf.org" <rtg-ads@tools.ietf.org>
From: "Joel M. Halpern" <jmh@joelhalpern.com>
Message-ID: <a703aa8f-3d9a-faa8-143b-470d09dd8c4b@joelhalpern.com>
Date: Wed, 23 Nov 2016 16:10:27 -0500
User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.9; rv:45.0) Gecko/20100101 Thunderbird/45.5.0
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"; format="flowed"
Content-Transfer-Encoding: 8bit
Archived-At: <https://mailarchive.ietf.org/arch/msg/rtg-dir/Xvu2tz1tzZQ8LvDKAx7qqFrU4Yg>
Cc: "rtg-dir@ietf.org" <rtg-dir@ietf.org>, draft-ietf-teas-p2mp-loose-path-reopt.all@ietf.org, "teas@ietf.org" <teas@ietf.org>
Subject: [RTG-DIR] RtgDir review: draft-ietf-teas-p2mp-loose-path-reopt-07.txt
X-BeenThere: rtg-dir@ietf.org
X-Mailman-Version: 2.1.17
Precedence: list
List-Id: Routing Area Directorate <rtg-dir.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/rtg-dir>, <mailto:rtg-dir-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/rtg-dir/>
List-Post: <mailto:rtg-dir@ietf.org>
List-Help: <mailto:rtg-dir-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/rtg-dir>, <mailto:rtg-dir-request@ietf.org?subject=subscribe>
X-List-Received-Date: Wed, 23 Nov 2016 21:09:42 -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 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-teas-p2mp-loose-path-reopt-07.txt Reviewer: Joel M. Halpern Review Date: 23-November-2016 IETF LC End Date: N/A Intended Status: Standards Track Summary: I have some moderate concerns about this document that I think should be resolved before publication is approved. Comments: Major: The use of SHOULD and MAY in section 4.1 seems to lead to a device which ostensibly supports this document, but does the wrong things. First, with regard to the SHOULDs, in the absence of any indication as to why it would not do this, it appears that the SHOULD is really "MUST if the device supports this document" which is what MUST in a document actually means. Section 4.2 first bullet says that a mid-point LSR "SHOULD" check for a preferable P2MP-TE LSP Tree. But if it doesn't, it is not supporting this document. As written, it could decide to ignore the message, even though it claims to support this RFC. Looking at the handling when a preferable P2MP-TE LSP tree is found, according to the document, the LSR MAY send the PathErr response. My assumption is that if it does not send the PathErr, it MUST propagate the request. If it does not do either one, the protocol does not function. It seems likely that if this is really intended to be optional (MAY), the document would be improved my giving implementors some hint as to when it is desirable or undesirable to send the message. Then in the third bullet, it is only a SHOULD to pass on the request. Thus, a device which supports this mechanism, but chooses not to pass on the request, is compliant to this document while preventing other devices from properly supporting the mechanism. Minor: The abstract is much too long. Much of the content of the abstract belongs in the introduction. Even teh second paragraph has too much detail for an abstract. Editorial: In the last paragraph of the introduction, it says that this document "proposes" solutions. Given we are now in the position of evaluating publication as a Proposed Standard, I would say that this document "defines" solutions.
- [RTG-DIR] RtgDir review: draft-ietf-teas-p2mp-loo… Joel M. Halpern
- Re: [RTG-DIR] RtgDir review: draft-ietf-teas-p2mp… Rakesh Gandhi (rgandhi)
- Re: [RTG-DIR] RtgDir review: draft-ietf-teas-p2mp… Rakesh Gandhi (rgandhi)
- Re: [RTG-DIR] RtgDir review: draft-ietf-teas-p2mp… Joel M. Halpern
- Re: [RTG-DIR] RtgDir review: draft-ietf-teas-p2mp… Rakesh Gandhi (rgandhi)