[pim] Éric Vyncke's No Objection on draft-ietf-pim-assert-packing-10: (with COMMENT)

Éric Vyncke via Datatracker <noreply@ietf.org> Thu, 16 March 2023 13:20 UTC

Return-Path: <noreply@ietf.org>
X-Original-To: pim@ietf.org
Delivered-To: pim@ietfa.amsl.com
Received: from ietfa.amsl.com (localhost [IPv6:::1]) by ietfa.amsl.com (Postfix) with ESMTP id 20DFBC15256E; Thu, 16 Mar 2023 06:20:56 -0700 (PDT)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 8bit
From: Éric Vyncke via Datatracker <noreply@ietf.org>
To: The IESG <iesg@ietf.org>
Cc: draft-ietf-pim-assert-packing@ietf.org, pim-chairs@ietf.org, pim@ietf.org, stig@venaas.com, aretana.ietf@gmail.com, stig@venaas.com
X-Test-IDTracker: no
X-IETF-IDTracker: 9.14.0
Auto-Submitted: auto-generated
Precedence: bulk
Reply-To: Éric Vyncke <evyncke@cisco.com>
Message-ID: <167897285612.3230.189721272161382244@ietfa.amsl.com>
Date: Thu, 16 Mar 2023 06:20:56 -0700
Archived-At: <https://mailarchive.ietf.org/arch/msg/pim/KlNylMoFTJE2I6OAVtnRSWkKgFE>
Subject: [pim] Éric Vyncke's No Objection on draft-ietf-pim-assert-packing-10: (with COMMENT)
X-BeenThere: pim@ietf.org
X-Mailman-Version: 2.1.39
List-Id: Protocol Independent Multicast <pim.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/pim>, <mailto:pim-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/pim/>
List-Post: <mailto:pim@ietf.org>
List-Help: <mailto:pim-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/pim>, <mailto:pim-request@ietf.org?subject=subscribe>
X-List-Received-Date: Thu, 16 Mar 2023 13:20:56 -0000

Éric Vyncke has entered the following ballot position for
draft-ietf-pim-assert-packing-10: 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/about/groups/iesg/statements/handling-ballot-positions/ 
for more information about how to handle DISCUSS and COMMENT positions.


The document, along with other ballot positions, can be found here:
https://datatracker.ietf.org/doc/draft-ietf-pim-assert-packing/



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


# Éric Vyncke, INT AD, comments for draft-ietf-pim-assert-packing-10
CC @evyncke

Thank you for the work put into this document.

Please find below some non-blocking COMMENT points (but replies would be
appreciated even if only for my own education), and one nit.

Special thanks to Stig Venaas for the shepherd's detailed write-up including
the WG consensus **but** it lacks the justification of the intended status.

I hope that this review helps to improve the document,

Regards,

-éric

## COMMENTS

### Section 1

`there is typically more than one upstream router` unsure whether it is really
the common case... Suggest to s/there is typically/Sometimes there is/

### Section 2

Should there be references for assertions such as `something not possible
equally with an L3 subnet based ring`?

### Section 4.2

*Strongly* suggest to add a reference to section 4.9.1 of RFC 7761 for the
`encoded-* format` (as my heart missed a beat when seeing a 32-bit field).

## NITS

### Units

s/100msec/100 msec/ and other similar issues.

## Notes

This review is in the ["IETF Comments" Markdown format][ICMF], You can use the
[`ietf-comments` tool][ICT] to automatically convert this review into
individual GitHub issues.

[ICMF]: https://github.com/mnot/ietf-comments/blob/main/format.md
[ICT]: https://github.com/mnot/ietf-comments