Re: [CCAMP] Robert Wilton's No Objection on draft-ietf-ccamp-layer0-types-08: (with COMMENT)

Zhenghaomian <zhenghaomian@huawei.com> Wed, 16 December 2020 07:25 UTC

Return-Path: <zhenghaomian@huawei.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 4DE5F3A1162; Tue, 15 Dec 2020 23:25:36 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.9
X-Spam-Level:
X-Spam-Status: No, score=-1.9 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, RCVD_IN_MSPIKE_H2=-0.001, SPF_HELO_NONE=0.001, SPF_PASS=-0.001, URIBL_BLOCKED=0.001] 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 wxgo85ZUfGIH; Tue, 15 Dec 2020 23:25:34 -0800 (PST)
Received: from frasgout.his.huawei.com (frasgout.his.huawei.com [185.176.79.56]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 130073A112F; Tue, 15 Dec 2020 23:25:31 -0800 (PST)
Received: from fraeml715-chm.china.huawei.com (unknown [172.18.147.201]) by frasgout.his.huawei.com (SkyGuard) with ESMTP id 4Cwmlc4WP7z67Qhn; Wed, 16 Dec 2020 15:21:48 +0800 (CST)
Received: from fraeml715-chm.china.huawei.com (10.206.15.34) by fraeml715-chm.china.huawei.com (10.206.15.34) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2106.2; Wed, 16 Dec 2020 08:25:29 +0100
Received: from DGGEML403-HUB.china.huawei.com (10.3.17.33) by fraeml715-chm.china.huawei.com (10.206.15.34) with Microsoft SMTP Server (version=TLS1_0, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA) id 15.1.2106.2 via Frontend Transport; Wed, 16 Dec 2020 08:25:28 +0100
Received: from DGGEML511-MBX.china.huawei.com ([169.254.1.73]) by DGGEML403-HUB.china.huawei.com ([fe80::74d9:c659:fbec:21fa%31]) with mapi id 14.03.0487.000; Wed, 16 Dec 2020 15:25:26 +0800
From: Zhenghaomian <zhenghaomian@huawei.com>
To: Robert Wilton <rwilton@cisco.com>, The IESG <iesg@ietf.org>
CC: "draft-ietf-ccamp-layer0-types@ietf.org" <draft-ietf-ccamp-layer0-types@ietf.org>, "ccamp-chairs@ietf.org" <ccamp-chairs@ietf.org>, "ccamp@ietf.org" <ccamp@ietf.org>, "Daniele Ceccarelli" <daniele.ceccarelli@ericsson.com>
Thread-Topic: Robert Wilton's No Objection on draft-ietf-ccamp-layer0-types-08: (with COMMENT)
Thread-Index: AdbTfHm3AM1X4D8BTYOUdviymg+cRQ==
Date: Wed, 16 Dec 2020 07:25:25 +0000
Message-ID: <E0C26CAA2504C84093A49B2CAC3261A43FA3DF7D@dggeml511-mbx.china.huawei.com>
Accept-Language: zh-CN, en-US
Content-Language: zh-CN
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
x-originating-ip: [10.24.176.98]
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: base64
MIME-Version: 1.0
X-CFilter-Loop: Reflected
Archived-At: <https://mailarchive.ietf.org/arch/msg/ccamp/0FNY0vz9h-26LDt2wSj5bL1UE6g>
Subject: Re: [CCAMP] Robert Wilton's No Objection on draft-ietf-ccamp-layer0-types-08: (with COMMENT)
X-BeenThere: ccamp@ietf.org
X-Mailman-Version: 2.1.29
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: <https://mailarchive.ietf.org/arch/browse/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, 16 Dec 2020 07:25:37 -0000

Hi, Rob, 

Thank you for the review and comments, again. As the comments are all about the YANG model changes, please refer to https://github.com/ietf-ccamp-wg/draft-ietf-ccamp-layer0-types/pull/15 for our revision in the module, and let us know if you have other considerations.

Best wishes,
Haomian (on behalf of all authors & contributors)

-----邮件原件-----
发件人: Robert Wilton via Datatracker [mailto:noreply@ietf.org] 
发送时间: 2020年12月2日 0:33
收件人: The IESG <iesg@ietf.org>
抄送: draft-ietf-ccamp-layer0-types@ietf.org; ccamp-chairs@ietf.org; ccamp@ietf.org; Daniele Ceccarelli <daniele.ceccarelli@ericsson.com>om>; daniele.ceccarelli@ericsson.com
主题: Robert Wilton's No Objection on draft-ietf-ccamp-layer0-types-08: (with COMMENT)

Robert Wilton has entered the following ballot position for
draft-ietf-ccamp-layer0-types-08: No Objection

When responding, please keep the subject line intact and reply to all email addresses included in the To and CC lines. (Feel free to cut this introductory paragraph, however.)


Please refer to https://www.ietf.org/iesg/statement/discuss-criteria.html
for more information about IESG DISCUSS and COMMENT positions.


The document, along with other ballot positions, can be found here:
https://datatracker.ietf.org/doc/draft-ietf-ccamp-layer0-types/



----------------------------------------------------------------------
COMMENT:
----------------------------------------------------------------------

Hi,

Thank you for your effort on this document.  I appreciate that I have reviewed this module previously.  I have a few minor (mostly editorial) comments on the latest revision of the YANG module:

A few minor comments on the latest version of the YANG module:

Some of the descriptions have been wrapped at less than 69 characters. 
Corrected examples are below.

1)
  grouping wson-label-start-end {
    description
      "The WSON label-start or label-end used to specify WSON label
       range.";
 

2)
          case single {
            leaf dwdm-n {
              type l0-types:dwdm-n;
              description
                "The given value 'N' is used to determine the nominal
                 central frequency.";
            }
          }

3)
    leaf priority {
      type uint8;
      description
        "Priority in Interface Switching Capability Descriptor
         (ISCD).";

4)
  grouping flexi-grid-label-start-end {
    description
      "The Flexi-grid label-start or label-end used to specify
       Flexi-grid label range.";

5)
      case super {
        list subcarrier-flexi-n {
          key flexi-n;
          uses flexi-grid-frequency-slot;
          description
            "List of subcarrier channels for flexi-grid super
             channel.";
        }
      }

6)
      leaf slot-width-granularity {
        type identityref {
          base flexi-slot-width-granularity;
        }
        default flexi-swg-12p5ghz;
        description
          "Minimum space between slot widths. Default is 12.500 GHz";

The following leaf had an extra blank line before the description:
PROPOSED:

    leaf flexi-n {
      type l0-types:flexi-n;
      description
        "The given value 'N' is used to determine the nominal central
         frequency.";
    }

Please change the description to "Flexi-grid-specific label range related information", i.e.,

CURRENT:
  grouping flexi-grid-label-range-info {
    description
      "Info of Flexi-grid-specific label range";

PROPOSED:
  grouping flexi-grid-label-range-info {
    description
      "Flexi-grid-specific label range related information";
    uses l0-label-range-info;

Nit, some of the descriptions have an extra trailing space after the full stop, plesae search/replace '. "'


For min-slot-width-factor, I would suggest removing the last sentence, and only specifying the constraint on the max-slot-width-factor, and also define a must statement as well.

CURRENT:
      leaf min-slot-width-factor {
        type uint16 {
          range "1..max";
        }
        default 1;
        description
          "Slot width range: two multipliers of the slot width ,
           granularity, each indicating the minimal and maximal slot
           width supported by a port, respectively.

           Minimum slot width is calculated by:
             Minimum slot width (GHz) =
               min-slot-width-factor * slot-width-granularity.
           Minimum slot width should be smaller than or equal to
           Maximum slot width. ";

PROPOSED:
      leaf min-slot-width-factor {
        type uint16 {
          range "1..max";
        }
        default 1;
        description
          "Slot width range: two multipliers of the slot width ,
           granularity, each indicating the minimal and maximal slot
           width supported by a port, respectively.

           Minimum slot width is calculated by:
             Minimum slot width (GHz) =
               min-slot-width-factor * slot-width-granularity.";
        reference
          "RFC8363: GMPLS OSPF-TE Extensions in Support of Flexi-Grid
           Dense Wavelength Division Multiplexing (DWDM) Networks";
      }

I suggest tweaking the description and adding a must constraint to the max-slot-width-factor.  My assumption here is that a client could set the min-slot-width-factor to 2, and doesn't need to also specify max-slot-width-factor unless they want it to be greater than 2.  Coversely, they can specify both if they wish.

CURRENT:
      leaf max-slot-width-factor {
        type uint16 {
          range "1..max";
        }
        description
          "Slot width range: two multipliers of the slot width ,
           granularity, each indicating the minimal and maximal slot
           width supported by a port, respectively.

           Maximum slot width is calculated by:
             Maximum slot width (GHz) =
               max-slot-width-factor * slot-width-granularity
           Maximum slot width should be bigger than or equal to
           Minimum slot width. ";
        reference
          "RFC8363: GMPLS OSPF-TE Extensions in Support of Flexi-Grid
           Dense Wavelength Division Multiplexing (DWDM) Networks";
      }

PROPOSED:

      leaf max-slot-width-factor {
        type uint16 {
          range "1..max";
        }
        must '. >= min-slot-width-factor' {
          error-message
            "Maximum slot width must be greater than or equal to
             minimum slot width.";
        }
        description
          "Slot width range: two multipliers of the slot width ,
           granularity, each indicating the minimal and maximal slot
           width supported by a port, respectively.

           Maximum slot width is calculated by:
             Maximum slot width (GHz) =
               max-slot-width-factor * slot-width-granularity

           If specified, maximum slot width must be greater than or
           equal to minimum slot width.";
        reference
          "RFC8363: GMPLS OSPF-TE Extensions in Support of Flexi-Grid
           Dense Wavelength Division Multiplexing (DWDM) Networks";
      }

Regards,
Rob