[Pce] Progressing draft-ietf-pce-vendor-constraints after WG last call
"Adrian Farrel" <adrian@olddog.co.uk> Thu, 17 October 2013 22:01 UTC
Return-Path: <adrian@olddog.co.uk>
X-Original-To: pce@ietfa.amsl.com
Delivered-To: pce@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 6BE7011E8260 for <pce@ietfa.amsl.com>; Thu, 17 Oct 2013 15:01:11 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.56
X-Spam-Level:
X-Spam-Status: No, score=-2.56 tagged_above=-999 required=5 tests=[AWL=0.039, BAYES_00=-2.599]
Received: from mail.ietf.org ([12.22.58.30]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id AhmV8YYw5nOA for <pce@ietfa.amsl.com>; Thu, 17 Oct 2013 15:01:05 -0700 (PDT)
Received: from asmtp4.iomartmail.com (asmtp4.iomartmail.com [62.128.201.175]) by ietfa.amsl.com (Postfix) with ESMTP id 3B58F11E8202 for <pce@ietf.org>; Thu, 17 Oct 2013 15:01:05 -0700 (PDT)
Received: from asmtp4.iomartmail.com (localhost.localdomain [127.0.0.1]) by asmtp4.iomartmail.com (8.13.8/8.13.8) with ESMTP id r9HM0pZH007920 for <pce@ietf.org>; Thu, 17 Oct 2013 23:00:52 +0100
Received: from 950129200 (dsl-sp-81-140-15-32.in-addr.broadbandscope.com [81.140.15.32]) (authenticated bits=0) by asmtp4.iomartmail.com (8.13.8/8.13.8) with ESMTP id r9HM0oni007904 (version=TLSv1/SSLv3 cipher=AES128-SHA bits=128 verify=NO) for <pce@ietf.org>; Thu, 17 Oct 2013 23:00:51 +0100
From: Adrian Farrel <adrian@olddog.co.uk>
To: pce@ietf.org
Date: Thu, 17 Oct 2013 23:00:49 +0100
Message-ID: <04b701cecb84$57411e10$05c35a30$@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: Ac7LhFIEyR5QBkEcQ1CPTzzGKi/3SA==
Content-Language: en-gb
Subject: [Pce] Progressing draft-ietf-pce-vendor-constraints after WG last call
X-BeenThere: pce@ietf.org
X-Mailman-Version: 2.1.12
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: Thu, 17 Oct 2013 22:01:11 -0000
Hi WG, Well, we (or probably me) were a bit slow in responding to WG last call comments. Thanks for the input. We are preparing a new revision. Here are some notes on what is changing. Thanks, Adrian --- Nits from Julien and others picked up and included. --- There were some discussions about whether the RBNF should be "strict" and I think this ties in with a number of existing RFCs and a long-running debate in the working group. Now that there is an I-D that attempts to pull together all PCEP RBNF into a "definitive" document, I don't think it would be right to try to embed that work in this document. Furthermore, asking this I-D to fix a concern about the relative object orderings in RFC 5440, RFC 5541, and RFC 6006 would be unreasonable. Similarly, i don't think it falls to this I-d to define the placement of the BNC or UNREACH-DESTINATION objects in a PCReq. --- We have clarified the distinction in error processing between not supporting the Vendor Information object, and not recognising the enterprise number. the text in Section 2 now reads... A PCE that receives a PCReq message containing a Vendor Information object MUST act according to the P flag in the object header. That is, if the P flag is set, the object will be treated as mandatory and the request will either be processed using the contents of the object, or the request will be rejected as defined in [RFC5440] (see also Section 2.1). If the P flag is clear then, as defined in [RFC5440], the object may be used by the PCE or may be ignored. The PCC sets the P flag according to how it wishes the request to be processed. The PCE determines how to interpret the information in the Vendor Information object by examining the Enterprise Number it contains. An implementation that supports the Vendor Information object, but receives one carrying an Enterprise number that it does not support MUST act according to the P flag in the object. That is, if the P flag is set, the PCE MUST reject the PCReq as defined in [RFC5440] by sending an Error message with Error-Type="Not supported Object" along with the corresponding Vendor Information object. ...while 2.1 is unaltered. --- Ramon suggested > * <request> ::= I would suggest moving the vendor-info-list after > endpoints (for both p2p and p2mp) My personal preference would > be after metric list and objective function. endpoint-rro-pair-list at > least includes one mandatory ENDPOINTS object, making the > mandatory RP and ENDPOINTS objects appear first. This looks good, but there is a need to distinguish between vendor info that belongs to the <request> and vendor info that belongs to the <end-point-rro-pair-list>. So it must appear before the first <end-point-rro-pair-list> if it applies to the whole <request>. Hence, no change. [I consider it a bug in RFC 6006 that the same doesn't apply to <BANDWIDTH> and really it would be nice to put <end-point-rro-pair-list> last in <request>.] --- There was some push to also show RBNF for the PCRep message. Much though this goes against the grain for me (I really think that you should all have to implement RFC3209 a few times before complaining about PCEP ;-) and I can't actually think of why you would put Vendor info in a PCRep (but early on someone in the WG insisted) we have added the following: Thus, the PCRep is encoded as follows: <PCRep Message> ::= <Common Header> <response> <response> ::= <RP> [<vendor-info-list>] [<end-point-path-pair-list>] [<NO-PATH>] [<attribute-list>] where: <end-point-path-pair-list> ::= [<END-POINTS>] <path> [<vendor-info-list>] [<end-point-path-pair-list>] <path> ::= (<ERO>|<SERO>) [<path>] <attribute-list> ::= [<OF>] [<LSPA>] [<BANDWIDTH>] [<metric-list>] [<IRO>] ---
- [Pce] Progressing draft-ietf-pce-vendor-constrain… Adrian Farrel