Re: [Pce] PCE WG Last Call on draft-ietf-pce-iro-update-01

"Adrian Farrel" <adrian@olddog.co.uk> Tue, 05 May 2015 16:49 UTC

Return-Path: <adrian@olddog.co.uk>
X-Original-To: pce@ietfa.amsl.com
Delivered-To: pce@ietfa.amsl.com
Received: from localhost (ietfa.amsl.com [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 6E5E31A020D for <pce@ietfa.amsl.com>; Tue, 5 May 2015 09:49:24 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -100.701
X-Spam-Level:
X-Spam-Status: No, score=-100.701 tagged_above=-999 required=5 tests=[BAYES_40=-0.001, RCVD_IN_DNSWL_LOW=-0.7, USER_IN_WHITELIST=-100] autolearn=ham
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 X0bsnPVMmytj for <pce@ietfa.amsl.com>; Tue, 5 May 2015 09:49:20 -0700 (PDT)
Received: from asmtp2.iomartmail.com (asmtp2.iomartmail.com [62.128.201.249]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 4BD781A1B95 for <pce@ietf.org>; Tue, 5 May 2015 09:43:45 -0700 (PDT)
Received: from asmtp2.iomartmail.com (localhost.localdomain [127.0.0.1]) by asmtp2.iomartmail.com (8.13.8/8.13.8) with ESMTP id t45GhgmU010274 for <pce@ietf.org>; Tue, 5 May 2015 17:43:42 +0100
Received: from 950129200 (dsl-sp-81-140-15-32.in-addr.broadbandscope.com [81.140.15.32]) (authenticated bits=0) by asmtp2.iomartmail.com (8.13.8/8.13.8) with ESMTP id t45GhflO010259 (version=TLSv1/SSLv3 cipher=AES256-SHA bits=256 verify=NO) for <pce@ietf.org>; Tue, 5 May 2015 17:43:41 +0100
From: Adrian Farrel <adrian@olddog.co.uk>
To: pce@ietf.org
References: <21710_1430837092_5548D764_21710_282_1_5548D764.7080704@orange.com>
In-Reply-To: <21710_1430837092_5548D764_21710_282_1_5548D764.7080704@orange.com>
Date: Tue, 05 May 2015 17:43:39 +0100
Message-ID: <03a401d08752$a41b3960$ec51ac20$@olddog.co.uk>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
X-Mailer: Microsoft Outlook 14.0
Thread-Index: AQG2Fo0naf9YV2SZ1z9PrddI6CIALJ2ic4tA
Content-Language: en-gb
X-TM-AS-MML: disable
X-TM-AS-Product-Ver: IMSS-7.1.0.1576-7.5.0.1018-21524.001
X-TM-AS-Result: No--15.910-10.0-31-10
X-imss-scan-details: No--15.910-10.0-31-10
X-TMASE-MatchedRID: 6otD/cJAac1dpLkh5p97g+YAh37ZsBDC+q1Y+/eEArYY0A95tjAn+9Uc StYMaHX/zXEUzvqkO8aAPVAlFMe8bB2P280ZiGmRVF7yOiu4q2nWSrKtwxqWpd14Aqe8EzF8xGS h4nc4VFzdctq99gzG7yLQdatZJpCCBLJZuhcQ26cpa6LJktEjgMyZqgcnwGktZ5yuplze9pvUwu vLGfOD0i/catEG2e7ubOtUUj/X5Vnl3OZM/4d1DC/T1r68E/jWGY9Y+ATae1yD1OfYfymjI6EPL QgibzEiVR5nesAkYxBwouDsIRL+EPAFND5e9XpVvHKClHGjjr2STnFzEHOIwZsoi2XrUn/JyeMt MD9QOgDGlDvsLUDW2o6HM5rqDwqtlExlQIQeRG0=
Archived-At: <http://mailarchive.ietf.org/arch/msg/pce/txK0VnrngkFQLb9y-eWS4yfbCjk>
Subject: Re: [Pce] PCE WG Last Call on draft-ietf-pce-iro-update-01
X-BeenThere: pce@ietf.org
X-Mailman-Version: 2.1.15
Precedence: list
Reply-To: adrian@olddog.co.uk
List-Id: Path Computation Element <pce.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/pce>, <mailto:pce-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/pce/>
List-Post: <mailto:pce@ietf.org>
List-Help: <mailto:pce-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/pce>, <mailto:pce-request@ietf.org?subject=subscribe>
X-List-Received-Date: Tue, 05 May 2015 16:49:24 -0000

Hi,

No objections to this document from me, and thanks to the author for the
diligence with which he checked the impact of this change. We can't see from the
outside whether enough of the "real implementers" responded, but they had their
chance and this last call is their last chance :-)

---

I think it would be helpful if the "update" to RFC 5440 was more anchored into
that document.

So...

The update is to section 7.12, yes?
The text in the last paragraph of your section 2 is to be considered part of the
spec, yes?
The text is intended to replace the last line of 5440/7.12 that currently says
   The L bit of such sub-object has no meaning within an IRO.

I also think there is a bit of an over-use of "MUST" in...
   The content of an IRO object MUST be an ordered list of subobjects
   representing a series of abstract nodes.
Using "is" would be more appropriate. You could go "MUST be interpreted as", but
that also sounds excessive use of language.

---

In the Introduction you say
   During discussion of [I-D.ietf-pce-pcep-domain-sequence] it was
   proposed to have a new IRO type with ordered nature, as well as
   handling of Loose bit (L bit).

This is completely true and indisputable. But appearing like this it raises more
questions than it answers. Either delete the paragraph or add some resolution
such as "however, with the update to RFC 5440 described in this document, no new
IRO type is needed."

---

Section 2 has

   A survey of the existing and planned implementations was conducted in
   order to discover the current state of affairs amongst
   implementations.  [I-D.dhody-pce-iro-survey] describe the
   questionnaire, results and presents some conclusions and proposed
   action items.  More details in Appendix A.

Having read App A I don't think it adds any more details to what is in the Intro
and in this paragraph.
You have the reference to the survey i-D (which will stay in the archives for
ever), so I suggest to delete the appendix and the pointer to it.

---

It looks to me that RFC 3209 is a normative reference since I must look there to
find out how to interpret the L bit.

Cheers,
Adrian


> This message initiates a 2-week WG last call on
> draft-ietf-pce-iro-update-01. Please send your comments to the PCE
> mailing list by Tuesday May 19.
> 
> Thanks,
> 
> JP & Julien