Re: [RTG-DIR] Rtgdir last call review of draft-ietf-pce-rfc6006bis-02

Ben Niven-Jenkins <ben@niven-jenkins.co.uk> Sun, 02 July 2017 17:20 UTC

Return-Path: <ben@niven-jenkins.co.uk>
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 32FF1120227; Sun, 2 Jul 2017 10:20:30 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -4.699
X-Spam-Level:
X-Spam-Status: No, score=-4.699 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, HTML_MESSAGE=0.001, RCVD_IN_DNSWL_NONE=-0.0001, RCVD_IN_MSPIKE_H2=-2.8] autolearn=ham autolearn_force=no
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 uhO46zhEFAkw; Sun, 2 Jul 2017 10:20:28 -0700 (PDT)
Received: from mailex.mailcore.me (mailex.mailcore.me [94.136.40.146]) by ietfa.amsl.com (Postfix) with ESMTP id C7FE0129A97; Sun, 2 Jul 2017 10:20:27 -0700 (PDT)
Received: from [176.24.45.127] (helo=[192.168.0.6]) by smtp04.mailcore.me with esmtpa (Exim 4.89) (envelope-from <ben@niven-jenkins.co.uk>) id 1dRiXp-0009QF-NU; Sun, 02 Jul 2017 18:20:26 +0100
Content-Type: multipart/alternative; boundary="Apple-Mail=_9619BE4F-C095-4565-9B85-02689330DE39"
Mime-Version: 1.0 (Mac OS X Mail 8.2 \(2098\))
From: Ben Niven-Jenkins <ben@niven-jenkins.co.uk>
In-Reply-To: <ce5e5a75-e0a8-7baa-84ce-2fe9d60aab23@gmail.com>
Date: Sun, 02 Jul 2017 18:20:23 +0100
Cc: rtg-dir@ietf.org, draft-ietf-pce-rfc6006bis.all@ietf.org, pce@ietf.org, rtg-ads@ietf.org, "dhruv.dhody@huawei.com" <dhruv.dhody@huawei.com>
Message-Id: <C27B0040-5248-44FC-A7AE-ACB326175D5C@niven-jenkins.co.uk>
References: <149899773681.17431.10285629192748593381@ietfa.amsl.com> <ce5e5a75-e0a8-7baa-84ce-2fe9d60aab23@gmail.com>
To: Dhruv Dhody <dhruv.ietf@gmail.com>
X-Mailer: Apple Mail (2.2098)
X-Mailcore-Auth: 9600544
X-Mailcore-Domain: 172912
X-KLMS-Rule-ID: 1
X-KLMS-Message-Action: clean
X-KLMS-AntiSpam-Status: not scanned, license restriction
X-KLMS-AntiPhishing: not scanned, license restriction
X-KLMS-AntiVirus: Kaspersky Security 8.0 for Linux Mail Server, version 8.0.1.721, bases: 2017/07/02 11:49:00 #9933356
X-KLMS-AntiVirus-Status: Clean, skipped
Archived-At: <https://mailarchive.ietf.org/arch/msg/rtg-dir/tstDJK9ZFrxc2CzNTAmY69cF30Y>
Subject: Re: [RTG-DIR] Rtgdir last call review of draft-ietf-pce-rfc6006bis-02
X-BeenThere: rtg-dir@ietf.org
X-Mailman-Version: 2.1.22
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: Sun, 02 Jul 2017 17:20:30 -0000

Hi Dhruv,

> On 2 Jul 2017, at 17:08, Dhruv Dhody <dhruv.ietf@gmail.com> wrote:
> 
> Hi Ben,
> 
> Thanks for your review. Please see inline.
> 
> On Sunday 02 July 2017 05:45 PM, Ben Niven-Jenkins wrote:
>> 3) Section 5 says “PCEP implementations SHOULD consider the additional security
>> provided by Transport Layer Security (TLS) [I-D.ietf-pce-pceps].”
>> 
>> Use of SHOULD says to me you expect the majority of implementations to
>> implement I-D.ietf-pce-pceps. So should the reference to I-D.ietf-pce-pceps be
>> normative?
> 
> [Dhruv]:Hmm, you may be right.
> It just that other documents have put this down as Informative usually. PCEPS is also in publication process, so normative reference will most likely not block progress. I am not sure if we should deviate in this document. Thoughts?

I don’t have a strong opinion, I point it out as something that struck me as a possible oversight. If the authors & ADs are happy for it to be an informative reference, that’s fine with me.

>> 4) Section 6.5 - PCEP Objects. Should you specify the meaning of Object-Types
>> 0, 1 & 2 for the END-POINTS object, like you do for the other objects in this
>> section?
> 
> [Dhruv]: END-POINTS is an existing object defined in RFC5440. This document defines new object-types for the END-POINTS object. Thus I don't think there is a reason to mention 0,1 & 2.

Fair enough.

Ben