[RTG-DIR] RtgDir review: draft-ietf-pce-association-group-08

Stig Venaas <stig@venaas.com> Sat, 06 April 2019 00:55 UTC

Return-Path: <stig@venaas.com>
X-Original-To: rtg-dir@ietfa.amsl.com
Delivered-To: rtg-dir@ietfa.amsl.com
Received: from localhost (localhost []) by ietfa.amsl.com (Postfix) with ESMTP id 0B22B12028F for <rtg-dir@ietfa.amsl.com>; Fri, 5 Apr 2019 17:55:32 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.901
X-Spam-Status: No, score=-1.901 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIMWL_WL_MED=-0.001, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, RCVD_IN_DNSWL_NONE=-0.0001] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (2048-bit key) header.d=venaas-com.20150623.gappssmtp.com
Received: from mail.ietf.org ([]) by localhost (ietfa.amsl.com []) (amavisd-new, port 10024) with ESMTP id jLV6B6DfLeQl for <rtg-dir@ietfa.amsl.com>; Fri, 5 Apr 2019 17:55:29 -0700 (PDT)
Received: from mail-ed1-x532.google.com (mail-ed1-x532.google.com [IPv6:2a00:1450:4864:20::532]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 98C4F120278 for <rtg-dir@ietf.org>; Fri, 5 Apr 2019 17:55:29 -0700 (PDT)
Received: by mail-ed1-x532.google.com with SMTP id d11so6967323edp.11 for <rtg-dir@ietf.org>; Fri, 05 Apr 2019 17:55:29 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=venaas-com.20150623.gappssmtp.com; s=20150623; h=mime-version:from:date:message-id:subject:to:cc; bh=tFSm0FUMEYwyLlg1gh91Y4H9f2ThuPX7jazd6FamHpM=; b=tP7e+u/2av9VNmsbNmHM/9Xcc8LczB5HBaC6A/ipYwMmeRM0JUW5zj1VuP5wX1pVdT 6GRr/OZe/S8v0wWtxo+Q6N5c0UBorsOuitOZqyDL3DyIZK6ZXBRZt/6UH3iDbD2+RZVW MimnHg9pGRFCuGwaeUfKWUT3mE6PowQRlmMtf30fgil7/3TNR0aXQbFiNpyHn2yM6mxe auqDtyJNljbZN1k7HYEmzWN+h3uDsuLo1WoPJE4uHihnS31sv/Hq01IGp64w5uA3bzvr 8G4BCPUX+nceMhOCQETG6Eozcy/76VXZOLT+qHjyKVLDD3jPOSoXcp4UN3PqlIR5M7ip 2YWw==
X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:from:date:message-id:subject:to:cc; bh=tFSm0FUMEYwyLlg1gh91Y4H9f2ThuPX7jazd6FamHpM=; b=Q8T8jCDW7rI3RPteu5XC3V0eF1gQvXVu2fgyA3PH6oOkG6YQDekzlSRFQJNsgqmnbe yeZSuDzKF4qjI/uz5MopvNqt8Fwx3Ty9lju+tBxItrmFszinHapyOMKf56CTSU49TpBv 4LhA+TMYTZU6v9JtwYwdOd74zScDrg29j7Ec5UTuOhNnqKg+TnD9hVcOt1DYZozy8AH7 haL3pRZSDr3oyqj4Vk0fYsZ/PMMjX1yvifcZAJGSonsEyIxRlQL9Og1roheLsvPzgp5S Uiz+p5tbPWnIsz2Wk66ELR9FnbVMQk/yt/xXKR1LivNUDNswYy/VMkcwcYJ9rw8peEgh C96w==
X-Gm-Message-State: APjAAAVuT9cvaLOhwraYabsS2gx/66DOi1WJcsMzmXoN5p2EW59Yln0F U6BuiKnm+DQLJOnauJcEP8+FbEqECU8tkco4Vb9OcQ==
X-Google-Smtp-Source: APXvYqywKQEqzKDdCoA40Ksiq8GBxmf14JndhgsxM0Dd1+qrYYBvmI/d9Vtg7QlkON9s+Lrg5vjh7dqI7AWagQIAs2g=
X-Received: by 2002:a17:906:6c0d:: with SMTP id j13mr8989688ejr.249.1554512128089; Fri, 05 Apr 2019 17:55:28 -0700 (PDT)
MIME-Version: 1.0
From: Stig Venaas <stig@venaas.com>
Date: Fri, 05 Apr 2019 17:55:17 -0700
Message-ID: <CAHANBt+wJupX8fyZ=w6qhOf6R7qVu0_Khi8fo7BdhfVJSsmo5Q@mail.gmail.com>
To: "<rtg-ads@ietf.org>" <rtg-ads@ietf.org>
Cc: rtg-dir@ietf.org, draft-ietf-pce-association-group.all@ietf.org, pce@ietf.org
Content-Type: text/plain; charset="UTF-8"
Archived-At: <https://mailarchive.ietf.org/arch/msg/rtg-dir/1lugjtqa75DoxsBpfQNsro-BYpY>
Subject: [RTG-DIR] RtgDir review: draft-ietf-pce-association-group-08
X-BeenThere: rtg-dir@ietf.org
X-Mailman-Version: 2.1.29
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: Sat, 06 Apr 2019 00:55:32 -0000


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

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-pce-association-group-08.txt
Reviewer: Stig Venaas
Review Date: April 5th 2019
IETF LC End Date: April 5th 2019
Intended Status: Standards Track

This document is basically ready for publication, but has nits that
should be considered prior to publication.

The document is in good shape and fairly easy to read. There are some
grammar errors and some language that could be improved somewhat. I'm
sure the RFC editor will help with that, but I've added some comments

Major Issues:
No major issues found.

Minor Issues:
No minor issues found.


In Introduction first paragraph
Typo: Generalzied

In section 3.2:
   can either be dependent or independent.  The SVEC object identify the
Should be identifies                                        ^^^^^^^^

   The motivation behind the association group defined in this document
   and the SVEC object are quite different, though some use case may
   overlap.  The PCEP extensions that defines new association type
   should clarify the relationship between SVEC object and association
   type, if any.

A few issues here. Perhaps it should be
   The motivation behind the association group defined in this document
   and the SVEC object are quite different, though some use cases may
   overlap.  PCEP extensions that define a new association type should
   clarify the relationship between the SVEC object and the association
   type, if any.

In section 3.3:
   For the operator-configured association, the association parameters
   such as the association identifier, association type, as well as the
   association source IP address is manually configured by the operator.
The last line should perhaps be
   association source IP address, are manually configured by the

Right below:
This line
   the association identifier is allocated dynamically by the PCEP

should perhaps be
   the association identifier, are allocated dynamically by the PCEP

Also, right below:
   operator-configured association are known to the PCEP peer before

Should be "is known".

In this paragraph:
  The associations are properties of the LSP and thus could be stored
   in the LSP state database.  The dynamic association exist as long as
   the LSP state.  In case of PCEP session termination, the LSP state
   clean-up MUST also take care of associations.

the sentence "The dynamic association exist as long as the LSP state."
should perhaps be "The dynamic association exists as long as the LSP
state exists"?

In 3.4:
   A range of association identifier for each Association type
identifiers                     ^^^^^

In 6.1:
   association type are defined in separate documents.
In 6.3.1:
   <state-report-list> ::= <state-report>[<state-report-list>]
Should there be a space here?                        ^^^^^

In 6.3.2:
   <request-list>::= <request>[<request-list>]
Space here?                ^^^^^

In 6.3.3
Space here?                  ^^^^^

In 6.4:
   attached to LSP state and the association exist till there is an
exists                                      ^^^^^^^

   in [RFC5440].  If a PCEP speaker understand the ASSOCIATION object
understands                         ^^^^^^^^^^^

   a new associations, it MUST return a PCErr message with Error-Type 26

In 7.2:
   31 (Early   Extended Association Id     [This.I-D]
I think it should say ID           ^^^^
If you agree, then this should also be fixed in the IANA registry. It
currently says Id.

In 7.4:
   There are no association type specified in this document, future
types                      ^^^^^^