[Gen-art] Review of draft-ietf-ccamp-flexible-grid-ospf-ext-08

Pete Resnick <presnick@qti.qualcomm.com> Fri, 10 February 2017 23:24 UTC

Return-Path: <presnick@qti.qualcomm.com>
X-Original-To: gen-art@ietf.org
Delivered-To: gen-art@ietfa.amsl.com
Received: from ietfa.amsl.com (localhost [IPv6:::1]) by ietfa.amsl.com (Postfix) with ESMTP id 2962712A060; Fri, 10 Feb 2017 15:24:06 -0800 (PST)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
From: Pete Resnick <presnick@qti.qualcomm.com>
To: gen-art@ietf.org
X-Test-IDTracker: no
X-IETF-IDTracker: 6.43.0
Auto-Submitted: auto-generated
Precedence: bulk
Message-ID: <148676904616.29176.18172899169844310297.idtracker@ietfa.amsl.com>
Date: Fri, 10 Feb 2017 15:24:06 -0800
Archived-At: <https://mailarchive.ietf.org/arch/msg/gen-art/nVPjLHJ_rT_eEsR86awLRdv02OA>
Cc: draft-ietf-ccamp-flexible-grid-ospf-ext.all@ietf.org, ccamp@ietf.org
Subject: [Gen-art] Review of draft-ietf-ccamp-flexible-grid-ospf-ext-08
X-BeenThere: gen-art@ietf.org
X-Mailman-Version: 2.1.17
List-Id: "GEN-ART: General Area Review Team" <gen-art.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/gen-art>, <mailto:gen-art-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/gen-art/>
List-Post: <mailto:gen-art@ietf.org>
List-Help: <mailto:gen-art-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/gen-art>, <mailto:gen-art-request@ietf.org?subject=subscribe>
X-List-Received-Date: Fri, 10 Feb 2017 23:24:06 -0000

Reviewer: Pete Resnick
Review result: Ready with Nits

I am the assigned Gen-ART reviewer for this draft. The General Area
Review Team (Gen-ART) reviews all IETF documents being processed
by the IESG for the IETF Chair. Please wait for direction from your
document shepherd or AD before posting a new version of the draft.

For more information, please see the FAQ at

<https://trac.ietf.org/trac/gen/wiki/GenArtfaq>.

Document: draft-ietf-ccamp-flexible-grid-ospf-ext-08
Reviewer: Pete Resnick
Review Date: 2017-02-10
IETF LC End Date: 2017-01-31
IESG Telechat date: 2017-02-16

Summary: Ready with Nits

A couple of nits that I mentioned in my earlier review that you might
want to address, but none of them are essential. (You may have decided
that I was wrong; that's OK too.) I didn't bother Cc'ing the IETF list
on this, since they're both very minor.

Major issues: None

Minor issues: None

Nits/editorial comments: 

3.1:

   A set of non-overlapping available frequency ranges MUST be 
   disseminated in order to allow efficient resource management of 
   flexi-grid DWDM links and RSA procedures which are described in 
   Section 4.8 of [RFC7698]. 

Those MUSTs look weird to me. I think instead of "MUST be" you mean
"are", since it doesn't look like an implementation really has a
choice here.

3.2:

   Hence, in order to support all possible applications and 
   implementations the following information should be advertised for
   a flexi-grid DWDM link:
   
Is that "should" in there meant to be normative? That is, do bad
things happen if I don't advertise one of those items? Or do you just
mean "the following information is advertised..."?