[Roll] Benoit Claise's No Objection on draft-ietf-roll-mpl-parameter-configuration-06: (with COMMENT)

"Benoit Claise" <bclaise@cisco.com> Wed, 08 July 2015 13:16 UTC

Return-Path: <bclaise@cisco.com>
X-Original-To: roll@ietfa.amsl.com
Delivered-To: roll@ietfa.amsl.com
Received: from localhost (ietfa.amsl.com [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 8F32C1B3528; Wed, 8 Jul 2015 06:16:49 -0700 (PDT)
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] 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 0OBXcoQYJTI8; Wed, 8 Jul 2015 06:16:48 -0700 (PDT)
Received: from ietfa.amsl.com (localhost [IPv6:::1]) by ietfa.amsl.com (Postfix) with ESMTP id 123C61A92B2; Wed, 8 Jul 2015 06:16:48 -0700 (PDT)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
From: "Benoit Claise" <bclaise@cisco.com>
To: "The IESG" <iesg@ietf.org>
X-Test-IDTracker: no
X-IETF-IDTracker: 6.0.4.p1
Auto-Submitted: auto-generated
Precedence: bulk
Message-ID: <20150708131648.13510.47968.idtracker@ietfa.amsl.com>
Date: Wed, 08 Jul 2015 06:16:48 -0700
Archived-At: <http://mailarchive.ietf.org/arch/msg/roll/FfWynAqTRgHxowQfojgYJl5KJzk>
Cc: menachemdodge1@gmail.com, roll-chairs@ietf.org, draft-ietf-roll-mpl-parameter-configuration@ietf.org, roll@ietf.org, maria.ines.robles@ericsson.com, draft-ietf-roll-mpl-parameter-configuration.shepherd@ietf.org, draft-ietf-roll-mpl-parameter-configuration.ad@ietf.org
Subject: [Roll] Benoit Claise's No Objection on draft-ietf-roll-mpl-parameter-configuration-06: (with COMMENT)
X-BeenThere: roll@ietf.org
X-Mailman-Version: 2.1.15
Reply-To: Routing Over Low power and Lossy networks <roll@ietf.org>
List-Id: Routing Over Low power and Lossy networks <roll.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/roll>, <mailto:roll-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/roll/>
List-Post: <mailto:roll@ietf.org>
List-Help: <mailto:roll-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/roll>, <mailto:roll-request@ietf.org?subject=subscribe>
X-List-Received-Date: Wed, 08 Jul 2015 13:16:49 -0000

Benoit Claise has entered the following ballot position for
draft-ietf-roll-mpl-parameter-configuration-06: 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-roll-mpl-parameter-configuration/



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

>From the OPS-DIR review done by Menachem:

The document is quite readable.

Section 2.1 could be improved if for each parameter defined a reference
would be given to indicate where the parameter is defined. I found most
of them in: draft-ietf-roll-trickle-mcast-12 but for example, I'm not
sure what the Proactive_Forwarding flag is used for.

Section 4 discusses Security Considerations but I think that some phrases
in the paragraph are too general.
Example:  "other methods for security should be applied" does not
indicate what these methods are or where to look for them.
Another Example: "To protect attacks from outside of
the network, unneccessary DHCPv6 packets should be filtered on the
border router between the ROLL network and the Internet"  - what is meant
by "unnecessary DHCPv6 packets"?


NITS 
====

Section 2.3 First Paragraph Second Sentence - Remove 'is'

OLD           ==> Note that there may be cases in which a node may fail
is to join a domain
SUGGEST ==> Note that there may be cases in which a node may fail to join
a domain

Section 2.3 Last Paragraph is not clear.

Perhaps the last sentence should read "In this case" (instead of "In the
case").


Section 2.6 - First Sentence - Not Clear

Is the update rate not to be more than twice the Information Refresh
Time? Not clear to me how many updates are allowed in an Information
Refresh Time.

Should the word 'of' be replaced by 'the'.

"more often than twice the Information Refresh Time".

Suggest rephrasing this paragraph.