[CCAMP] Robert Wilton's No Objection on draft-ietf-ccamp-layer0-types-08: (with COMMENT)
Robert Wilton via Datatracker <noreply@ietf.org> Tue, 01 December 2020 16:32 UTC
Return-Path: <noreply@ietf.org>
X-Original-To: ccamp@ietf.org
Delivered-To: ccamp@ietfa.amsl.com
Received: from ietfa.amsl.com (localhost [IPv6:::1]) by ietfa.amsl.com (Postfix) with ESMTP id 06E693A15F7; Tue, 1 Dec 2020 08:32:39 -0800 (PST)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
From: Robert Wilton via Datatracker <noreply@ietf.org>
To: The IESG <iesg@ietf.org>
Cc: draft-ietf-ccamp-layer0-types@ietf.org, ccamp-chairs@ietf.org, ccamp@ietf.org, Daniele Ceccarelli <daniele.ceccarelli@ericsson.com>, daniele.ceccarelli@ericsson.com
X-Test-IDTracker: no
X-IETF-IDTracker: 7.23.0
Auto-Submitted: auto-generated
Precedence: bulk
Reply-To: Robert Wilton <rwilton@cisco.com>
Message-ID: <160684035949.15525.449537069074560370@ietfa.amsl.com>
Date: Tue, 01 Dec 2020 08:32:39 -0800
Archived-At: <https://mailarchive.ietf.org/arch/msg/ccamp/EaueCtWSbdlW_-14SB-cc_oOYaQ>
Subject: [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
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: Tue, 01 Dec 2020 16:32:48 -0000
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
- [CCAMP] Robert Wilton's No Objection on draft-iet… Robert Wilton via Datatracker
- Re: [CCAMP] Robert Wilton's No Objection on draft… Zhenghaomian