[CCAMP] WG Last Call Comments on "GMPLS Enhancement for Signal and Network Element Compatibility of Wavelength Switched Optical Networks"

Acee Lindem <acee.lindem@ericsson.com> Tue, 29 October 2013 22:16 UTC

Return-Path: <acee.lindem@ericsson.com>
X-Original-To: ccamp@ietfa.amsl.com
Delivered-To: ccamp@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id ED62611E80F7 for <ccamp@ietfa.amsl.com>; Tue, 29 Oct 2013 15:16:33 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.661
X-Spam-Level:
X-Spam-Status: No, score=-2.661 tagged_above=-999 required=5 tests=[AWL=-0.063, BAYES_00=-2.599, HTML_MESSAGE=0.001]
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 oSWq-iHXCZ1R for <ccamp@ietfa.amsl.com>; Tue, 29 Oct 2013 15:16:26 -0700 (PDT)
Received: from usevmg21.ericsson.net (usevmg21.ericsson.net [198.24.6.65]) by ietfa.amsl.com (Postfix) with ESMTP id 230DD11E8214 for <ccamp@ietf.org>; Tue, 29 Oct 2013 15:16:15 -0700 (PDT)
X-AuditID: c6180641-b7fe28e000000d82-9a-527033ae583f
Received: from EUSAAHC003.ericsson.se (Unknown_Domain [147.117.188.81]) by usevmg21.ericsson.net (Symantec Mail Security) with SMTP id A7.73.03458.EA330725; Tue, 29 Oct 2013 23:16:14 +0100 (CET)
Received: from EUSAAMB101.ericsson.se ([147.117.188.118]) by EUSAAHC003.ericsson.se ([147.117.188.81]) with mapi id 14.02.0328.009; Tue, 29 Oct 2013 18:16:14 -0400
From: Acee Lindem <acee.lindem@ericsson.com>
To: CCAMP <ccamp@ietf.org>
Thread-Topic: WG Last Call Comments on "GMPLS Enhancement for Signal and Network Element Compatibility of Wavelength Switched Optical Networks"
Thread-Index: AQHO1PR6OxsJkAR0EUmBv2bkfRkIWg==
Date: Tue, 29 Oct 2013 22:16:13 +0000
Message-ID: <94A203EA12AECE4BA92D42DBFFE0AE47030B0E4A@eusaamb101.ericsson.se>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
x-originating-ip: [147.117.188.134]
Content-Type: multipart/alternative; boundary="_000_94A203EA12AECE4BA92D42DBFFE0AE47030B0E4Aeusaamb101erics_"
MIME-Version: 1.0
X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFjrGLMWRmVeSWpSXmKPExsUyuXRPoO4644Igg941RhZP5txgcWD0WLLk J1MAYxSXTUpqTmZZapG+XQJXRv+GNywFqwIrlu3fzNLA2OPcxcjJISFgIvFn3ipWCFtM4sK9 9WxdjFwcQgJHGCUONB5nhXCWM0osWQZRxSagI/H80T9mEFtEQEri5r5b7CBFwgK9jBK3T18E c0QEJjFK/Fm7lAmiSk/i7fH3YN0sAqoSx5bcZwGxeQV8JTqPvWIDsRmBdn8/tQasnllAXOLW k/lMEDcJSCzZc54ZwhaVePn4H9StyhJLnuxngajPlzh08DYjxExBiZMzn7BMYBSahWTULCRl s5CUQcR1JBbs/sQGYWtLLFv4mhnGPnPgMVSvtUTvgQ+MyGoWMHKsYuQoLU4ty003MtzECIyL YxJsjjsYF3yyPMQozcGiJM775a1zkJBAemJJanZqakFqUXxRaU5q8SFGJg5OqQbGaP81/6ft ltyxueg872mlJDt3Uc3fgVMmNlu8XbHLxePvzFB574giR/1Zx5fIH7jL/tWBc0/Fi9tfzyQt jvaeKLdtk2v8J1c9iU3NypusBcU5Mt4ILul3FWt8yvY9Y0u/TeEauf3Z3OWWss8rWa/cP3bl y6L7zHmf2vd9XzTllNzB0rZJT061KrEUZyQaajEXFScCAAnkeE1ZAgAA
Subject: [CCAMP] WG Last Call Comments on "GMPLS Enhancement for Signal and Network Element Compatibility of Wavelength Switched Optical Networks"
X-BeenThere: ccamp@ietf.org
X-Mailman-Version: 2.1.12
Precedence: list
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: Tue, 29 Oct 2013 22:16:34 -0000

I have the following comments on the subject draft:

               1. State the action to take if the new TLV and sub-TLVs or their attendant encodings are malformed. You should log the problem and ignore the entire LSA, subsuming TLV, or just the sub-TLV in GMPLS path computations.
               2. Section 2 - Your definition of "At most once" is semantically wrong. "At most once" means the TLV or sub-TLV can be include one time or not at all. It has nothing to with whether or not it should be specified. I hope we are not going to attempt to change the English language with this draft.
               3. Section 3 - Figure 1 should not span multiple pages and the scale is off by one - it should be shifted right 1 column.
               4. Section 6 - Explicitly state which are IANA registries are being extended. Since you are adding a new TLV, you will also need a new registry for the sub-TLVs. Seehttp://www.iana.org/assignments/ospf-traffic-eng-tlvs/ospf-traffic-eng-tlvs.xhtml#top-level for examples.


Editorial Comments:

I would suggest the following corrections:


125c125
<    to allow both multiple WSON signal types and common hybrid electro
---
>    to support both multiple WSON signal types and common hybrid electro
197c197
<    node. It is constructed of a set of sub-TLVs. There are no ordering
---
>    node. It is comprised of a set of sub-TLVs. There are no ordering
203c203
<    encodings of these properties are defined in [WSON-Encode].
---
>    encodings for these properties are defined in [WSON-Encode].
253,254c253
<    router, as described in [RFC3630] and [RFC5250]. Resource Block
<    Information
---
>    router, as described in [RFC3630] and [RFC5250].
279,280c278,279
<    The detail encodings of these sub-TLVs are found in [WSON-Encode] as
<    indicated in the table below.
---
>    The detailed encodings of these sub-TLVs are found in [WSON-Encode]
>    as indicated in the table below.
293c292
<    relation to the switching device. In particular it indicates the
---
>    relation to the switching device. In particular, it indicates the
302,303c301,302
<    reach or leave all the resources. Resource Block Wavelength
<    Constraints sub-TLV describe these properties.
---
>    reach or leave all the resources. The Resource Block Wavelength
>    Constraints sub-TLV describes these properties.
316c315
<    case then wavelength availability on these shared fibers is needed
---
>    case, then wavelength availability on these shared fibers is needed
353c352
<    Bandwidth TLV are defined (TBA by IANA):
---
>    Bandwidth sub-TLVs are defined (TBA by IANA):
402c401
<    produce LSAs that exceed the IP Maximum Transmission Unit (MTU). In
---
>    produce LSAs that exceeds the IP Maximum Transmission Unit (MTU). In
417,422c416,421
<    is received for a system path cannot make use of the other four sub-
<    TLVs since it does not know the nature of the resources, e.g., are
<    the resources wavelength converters, regenerators, or something
<    else. Once this sub-TLV is received path computation can proceed
<    with whatever of the additional types of sub-TLVs it may have
<    received (there use is dependent upon the system type). If path
---
>    is received for a system, path compuation cannot make use of the
>    other four sub-TLVs since it does not know the nature of the
>    resources, e.g., are the resources wavelength converters,
>    regenerators, or something else. Once this sub-TLV is received,
>    path computation can proceed with whatever sub-TLVs it may have
>    received (their use is dependent upon the system type). If path
433c432
<    these sub-TLVs then there is the possibility of either (a) path
---
>    these sub-TLVs, then there is the possibility of either (a) path

Thanks,
Acee