Adam Roach's No Objection on draft-ietf-bfd-multipoint-18: (with COMMENT)

Adam Roach <adam@nostrum.com> Tue, 03 July 2018 02:50 UTC

MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 8bit
From: Adam Roach <adam@nostrum.com>
To: The IESG <iesg@ietf.org>
Cc: draft-ietf-bfd-multipoint@ietf.org, Reshad Rahman <rrahman@cisco.com>, bfd-chairs@ietf.org, rrahman@cisco.com, rtg-bfd@ietf.org
Subject: Adam Roach's No Objection on draft-ietf-bfd-multipoint-18: (with COMMENT)
Auto-Submitted: auto-generated
Precedence: bulk
Message-ID: <153058620656.16408.3046099961115857075.idtracker@ietfa.amsl.com>
Date: Mon, 02 Jul 2018 19:50:06 -0700
Archived-At: <https://mailarchive.ietf.org/arch/msg/rtg-bfd/ts4VxnOUmDXI6CK3pOOd_b9B9eM>

Adam Roach has entered the following ballot position for
draft-ietf-bfd-multipoint-18: 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-bfd-multipoint/



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


The text in §5.1 says that MultipointHead sessions send packets with the M bit
set. It probably bears mention that this is an explicit update to the RFC 5880
requirement that "It MUST be zero on both transmit and receipt."

---------------------------------------------------------------------------

Nit (from id-nits):

  -- The draft header indicates that this document updates RFC5880, but the
     abstract doesn't seem to mention this, which it should.

---------------------------------------------------------------------------

§4:

>  If no
>  BFD Control packets are received by a tail for a detection time, the
>  tail declares the path to having failed.

Nit: "...to have failed."

---------------------------------------------------------------------------

§5.6:

>  A session of type MultipointHead is created for each multipoint path
>  over which the head wishes to run BFD.  This session runs in the
>  Active role , per section 6.1 [RFC5880].  Except when

Nit: extra space before comma

Adam Roach's No Objection on draft-ietf-bfd-multi… Adam Roach
Re: Adam Roach's No Objection on draft-ietf-bfd-m… Greg Mirsky