[Pce] Roman Danyliw's No Objection on draft-ietf-pce-stateful-pce-auto-bandwidth-11: (with COMMENT)

Roman Danyliw via Datatracker <noreply@ietf.org> Wed, 18 September 2019 17:38 UTC

Return-Path: <noreply@ietf.org>
X-Original-To: pce@ietf.org
Delivered-To: pce@ietfa.amsl.com
Received: from ietfa.amsl.com (localhost [IPv6:::1]) by ietfa.amsl.com (Postfix) with ESMTP id 7D167120AD4; Wed, 18 Sep 2019 10:38:56 -0700 (PDT)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 8bit
From: Roman Danyliw via Datatracker <noreply@ietf.org>
To: "The IESG" <iesg@ietf.org>
Cc: draft-ietf-pce-stateful-pce-auto-bandwidth@ietf.org, Adrian Farrel <adrian@olddog.co.uk>, pce-chairs@ietf.org, adrian@olddog.co.uk, pce@ietf.org
X-Test-IDTracker: no
X-IETF-IDTracker: 6.101.0
Auto-Submitted: auto-generated
Precedence: bulk
Reply-To: Roman Danyliw <rdd@cert.org>
Message-ID: <156882833650.4553.15366878761681665492.idtracker@ietfa.amsl.com>
Date: Wed, 18 Sep 2019 10:38:56 -0700
Archived-At: <https://mailarchive.ietf.org/arch/msg/pce/Bb1YVGezbgclb18xUTS7p6Kpzt0>
Subject: [Pce] Roman Danyliw's No Objection on draft-ietf-pce-stateful-pce-auto-bandwidth-11: (with COMMENT)
X-BeenThere: pce@ietf.org
X-Mailman-Version: 2.1.29
List-Id: Path Computation Element <pce.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/pce>, <mailto:pce-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/pce/>
List-Post: <mailto:pce@ietf.org>
List-Help: <mailto:pce-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/pce>, <mailto:pce-request@ietf.org?subject=subscribe>
X-List-Received-Date: Wed, 18 Sep 2019 17:38:57 -0000

Roman Danyliw has entered the following ballot position for
draft-ietf-pce-stateful-pce-auto-bandwidth-11: 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-pce-stateful-pce-auto-bandwidth/



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

** I support Ben Kaduk’s DISCUSS position on the need for clarity in the
definition of Overflow-Count and Overflow-Threshold

** Section 5.2.x.  Error handling:
-- A number of the sub-TLVs define ranges smaller than would be possible given
the number of bit (e.g., 1..604800 in a 32-bit field; 1.100), how would an
error be signaled for values used outside that range?

-- For values that are [IEEE.754.1985]), how should negative value be processed?

** Nits:
-- Section 1.  Typo. s/a Active stateful/an Active stateful/

-- Section 5.2.2.2.  For consistency with the other sections s/1 to 604800/1 to
604800 (7 days)/

-- Section 6.6.  Typo. s/signalling/signaling/