[CCAMP] AD review of draft-ietf-ccamp-wson-signaling

"Adrian Farrel" <adrian@olddog.co.uk> Wed, 04 March 2015 14:49 UTC

Return-Path: <adrian@olddog.co.uk>
X-Original-To: ccamp@ietfa.amsl.com
Delivered-To: ccamp@ietfa.amsl.com
Received: from localhost (ietfa.amsl.com [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 37BC91A00A2 for <ccamp@ietfa.amsl.com>; Wed, 4 Mar 2015 06:49:19 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -102.6
X-Spam-Level:
X-Spam-Status: No, score=-102.6 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, 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 wMchptR5590L for <ccamp@ietfa.amsl.com>; Wed, 4 Mar 2015 06:49:18 -0800 (PST)
Received: from asmtp5.iomartmail.com (asmtp5.iomartmail.com [62.128.201.176]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 469E71A007A for <ccamp@ietf.org>; Wed, 4 Mar 2015 06:49:16 -0800 (PST)
Received: from asmtp5.iomartmail.com (localhost.localdomain [127.0.0.1]) by asmtp5.iomartmail.com (8.13.8/8.13.8) with ESMTP id t24En2Uu029460; Wed, 4 Mar 2015 14:49:02 GMT
Received: from 950129200 (089144210187.atnat0019.highway.a1.net [89.144.210.187]) (authenticated bits=0) by asmtp5.iomartmail.com (8.13.8/8.13.8) with ESMTP id t24En0Yd029419 (version=TLSv1/SSLv3 cipher=AES256-SHA bits=256 verify=NO); Wed, 4 Mar 2015 14:49:01 GMT
From: Adrian Farrel <adrian@olddog.co.uk>
To: gregb@grotto-networking.com, xsg@nict.go.jp, 'Leeyoung' <leeyoung@huawei.com>, giomarti@cisco.com, harai@nict.go.jp
Date: Wed, 04 Mar 2015 14:49:02 -0000
Message-ID: <016901d0568a$5c91d0c0$15b57240$@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: AdBWijwn4CG/2CFPRJaoMn5GzUFRUA==
Content-Language: en-gb
X-TM-AS-MML: disable
X-TM-AS-Product-Ver: IMSS-7.1.0.1576-7.5.0.1018-21374.000
X-TM-AS-Result: No--8.819-10.0-31-10
X-imss-scan-details: No--8.819-10.0-31-10
X-TMASE-MatchedRID: xVsbTpU5Eh8mMPeO88gK4Jmug812qIbzAHN/Ezu9EnEkt9BigJAcVqAc vIcH5JEBQbOFXEEclEFz+cTBskrhCXHVdX7/QM0wF6z9HGHKwNsrHkgIan9a0aj5v7I4/SgY4OE EoEwjVDm/383xLccqQrADdpJedi9GmFozpQIyI+s1yhbbA7We0x9fNWA7SFWqfeB8ZkBTx8uZtw pGbMkRhROE79OcqkWsIOssMF33rUyPU5OGrpmnoPI2836jbv0ElPV6Vaqi4bDFHcwKL6UE/0Hic /bqcDgc8Mx8NkhQ3uj9KP9acIz8s01+zyfzlN7y/sToY2qzpx7dB/CxWTRRu25FeHtsUoHukNmj 524nc7KAdYlaCc1EsH8ihk5QyANzYelbf9XUh2EfwV6sBPR0lg==
Archived-At: <http://mailarchive.ietf.org/arch/msg/ccamp/ZGh_-dBOmAuvNjm1uGYMAG0u3XY>
Cc: 'CCAMP' <ccamp@ietf.org>, akatlas@gmail.com
Subject: [CCAMP] AD review of draft-ietf-ccamp-wson-signaling
X-BeenThere: ccamp@ietf.org
X-Mailman-Version: 2.1.15
Precedence: list
Reply-To: adrian@olddog.co.uk
List-Id: Discussion list for the CCAMP working group <ccamp.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/ccamp>, <mailto:ccamp-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/ccamp/>
List-Post: <mailto:ccamp@ietf.org>
List-Help: <mailto:ccamp-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/ccamp>, <mailto:ccamp-request@ietf.org?subject=subscribe>
X-List-Received-Date: Wed, 04 Mar 2015 14:49:19 -0000

[Using explicit addresses because the tools aliases are currently unreliable]

Authors,

I've done my usual AD review of your document (the nth in the sequence 
of n+1 WSON drafts!). I have a few comments, and normally I would ask
you for a new revision at once, but because of dates and things I am
going to start the IETF last call and enter these as last call comments.

The last call will end on 3/18 and you can post a new revision then (by
sending it to me to ask the Secretary to post it) or wait to post it
yourselves when the gates reopen on 3/23.

Thanks for the work,
Adrian

===

You need to split the Authors' Addresses section and move out all those
not on the current front page to be present in a Contributors section.

---

You seem to have the name of RFC 6205 in the references section a bit
wrong!

---
                                              
Do you really mean that you are updating 6205? That is the document
that defines lambda labels.

It has no mention of G.709 despite what you say in the Abstract. This
seems a little odd.                          

What is more, I don't see any discussion of updates to the label format
or usage in the document although Section 4 does say

   [RFC6205] defines the label format as applicable to LSC capable
   device. This document extends [RFC6205] as make its label format
   applicable also to WSON-LSC capable devices.

I don't think that applying the label to WSON without changing its
meaning or the way it is used is really an "update".

---

Section 2 has FOADM, but it is not used in the document.

You need to hyphenate "Cross Connect" unless you are really angry :-)

---

Section 3 gives a caption to a list

    List 1. WSON Signal Characteristics

I've not seen that before in an I-D, and since you don't refer to the 
list by name, I suggest you just delete this.

---

I wish you hadn't suggested explicit values for the two new Error Codes
in 4.2.2 and 6. But we'll see what IANA says during IETF last call.