[pim] Paul Wouters' Discuss on draft-ietf-pim-igmp-mld-proxy-yang-08: (with DISCUSS and COMMENT)

Paul Wouters via Datatracker <noreply@ietf.org> Tue, 29 November 2022 15:59 UTC

MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
From: Paul Wouters via Datatracker <noreply@ietf.org>
To: The IESG <iesg@ietf.org>
Cc: draft-ietf-pim-igmp-mld-proxy-yang@ietf.org, pim-chairs@ietf.org, pim@ietf.org, stig@venaas.com, aretana.ietf@gmail.com, stig@venaas.com
Auto-Submitted: auto-generated
Precedence: bulk
Reply-To: Paul Wouters <paul.wouters@aiven.io>
Message-ID: <166973755175.53412.13169591106491337030@ietfa.amsl.com>
Date: Tue, 29 Nov 2022 07:59:11 -0800
Archived-At: <https://mailarchive.ietf.org/arch/msg/pim/zTzk327nEnqC3jDWws-Au1ieTlg>
Subject: [pim] Paul Wouters' Discuss on draft-ietf-pim-igmp-mld-proxy-yang-08: (with DISCUSS and COMMENT)

Paul Wouters has entered the following ballot position for
draft-ietf-pim-igmp-mld-proxy-yang-08: Discuss

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-igmp-mld-proxy-yang/



----------------------------------------------------------------------
DISCUSS:
----------------------------------------------------------------------

Should these not be a rw value?

              +--ro filter-mode      enumeration

            +--ro group* [group-address]
                +--ro group-address

                +--ro source* [source-address]

The example in the appendix shows these values being set (so rw):

        "group-address": "233.252.0.23",
                        "filter-mode": "include",


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

NITS:
    The mld-version represents version of MLD protocol, and default value is 2.

The "2" here is a link to Section 2, which I assume is not intended to be a link.

    The mld-version represents version of MLD protocol

Missing "the" before version.

[pim] Paul Wouters' Discuss on draft-ietf-pim-igm… Paul Wouters via Datatracker
Re: [pim] Paul Wouters' Discuss on draft-ietf-pim… Hongji Zhao