Re: [pim] Robert Wilton's No Objection on draft-ietf-pim-igmp-mld-extension-06: (with COMMENT)

Alvaro Retana <aretana.ietf@gmail.com> Tue, 01 February 2022 19:41 UTC

Return-Path: <aretana.ietf@gmail.com>
X-Original-To: pim@ietfa.amsl.com
Delivered-To: pim@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 858FF3A08C6; Tue, 1 Feb 2022 11:41:29 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.098
X-Spam-Level:
X-Spam-Status: No, score=-2.098 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, FREEMAIL_FROM=0.001, SPF_HELO_NONE=0.001, SPF_PASS=-0.001, UNPARSEABLE_RELAY=0.001] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (2048-bit key) header.d=gmail.com
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 P3topOGpVU7T; Tue, 1 Feb 2022 11:41:28 -0800 (PST)
Received: from mail-ej1-x636.google.com (mail-ej1-x636.google.com [IPv6:2a00:1450:4864:20::636]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 241403A08C0; Tue, 1 Feb 2022 11:41:28 -0800 (PST)
Received: by mail-ej1-x636.google.com with SMTP id d10so57483551eje.10; Tue, 01 Feb 2022 11:41:28 -0800 (PST)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20210112; h=from:in-reply-to:references:mime-version:date:message-id:subject:to :cc:content-transfer-encoding; bh=WuLd/x5Rjb6K1SLNljH6Pxy4vDGaS0Yl7wUUbyRucHk=; b=HRDcJMD7dEF6hyz22Ib2yjDqMSjQKu92RAAoKMfS4Ux7gkBueeA6nihp7w/Ot7tZuS 1yJ7CqTHzckX6F1oHHW8aTjYJ6hoVIOgOB0rySjbmJYI/0dsVKvk70HxwsSR9vlixHq9 9c9o5piWJNtw23Tg+/OHQAultHD1ZGhlGpyUN8MZlW2WKcpbMiRR90zzl/WyW7t2MUBx JUMukMZv4/QNk89z2YxFzbOYYHe3WZJFYBHLk5z+pl5oD9sPXbe+/O7mKi+oJ46CrYcg jfk1J10QbQ/FsSAnib7zjZ8oU5CpsYBOdQZZ1oBhlg23DgwmE1QvbZYH0e9CyEKr9sWb y6wg==
X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:from:in-reply-to:references:mime-version:date :message-id:subject:to:cc:content-transfer-encoding; bh=WuLd/x5Rjb6K1SLNljH6Pxy4vDGaS0Yl7wUUbyRucHk=; b=m6LOeVxQEl+2mfqUBr6A4UFry4RJ3ne6M2tlfQcwUVnfkJIqlcuFt+o3iQs1nWqsOj P/ZgaeD2E95QDokBryr2J271mZS35tMI64NGoLm5943/H9w7p3AcoRZJAoE7tMir2WEf bkL6k+Q+DQtrvw8ea3Pg7cY0/avvizze8W668Uyh3RGmxXTGxvQxHOuo1f6EZ4zQFWp1 Cq3VrH8WkpynCLaIGPDZd+ROTQyLDSLvl6lwJIn3pvyH9Ybd8fVSNlNDsOthhCndTHYA vlkj5fMn6f+qhqWV8O1HbSYDYCDTgZneOU0oMmPI20vVprxfjk4dCu57N8LUqAbptCoB 6EOA==
X-Gm-Message-State: AOAM531yNqu5YhG12nDrF3BtYs6TPRvZyGc/bC7FZZ1p62irADHeJVBQ egB9ZebPjUblU5QzjjgGVSP8vmzHLOcATEmrU/o=
X-Google-Smtp-Source: ABdhPJwK5XHN1A7B6NnPgtr+buY15PFxNsf96khJJKy99NtWaXwqwskFMPouqC7xFAiTPNciRMDKuHAcZH4qFYtdzEs=
X-Received: by 2002:a17:906:dc50:: with SMTP id yz16mr118176ejb.633.1643744485940; Tue, 01 Feb 2022 11:41:25 -0800 (PST)
Received: from 1058052472880 named unknown by gmailapi.google.com with HTTPREST; Tue, 1 Feb 2022 11:41:25 -0800
From: Alvaro Retana <aretana.ietf@gmail.com>
In-Reply-To: <164364087774.14877.17656092608377325488@ietfa.amsl.com>
References: <164364087774.14877.17656092608377325488@ietfa.amsl.com>
MIME-Version: 1.0
Date: Tue, 01 Feb 2022 11:41:25 -0800
Message-ID: <CAMMESsy9RQHNwE3B_AUt=nAbgxd+QkxxZaySyXN-wQqzWaQAVg@mail.gmail.com>
To: Robert Wilton <rwilton@cisco.com>
Cc: pim-chairs@ietf.org, mmcbride7@gmail.com, The IESG <iesg@ietf.org>, draft-ietf-pim-igmp-mld-extension@ietf.org, pim@ietf.org
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
Archived-At: <https://mailarchive.ietf.org/arch/msg/pim/VudLRGvHRMX-SKakjCxoE6XlRwI>
Subject: Re: [pim] Robert Wilton's No Objection on draft-ietf-pim-igmp-mld-extension-06: (with COMMENT)
X-BeenThere: pim@ietf.org
X-Mailman-Version: 2.1.29
Precedence: list
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: Tue, 01 Feb 2022 19:41:30 -0000

On January 31, 2022 at 9:54:39 AM, Robert Wilton wrote:


Rob:

Hi!

Thanks for the review!


...
> Thanks for this document. I found the use of the term "extension" to perhaps
> not be as clear as it could be. In most places the reference is to Extension
> in the singular, and the attributes carried within the extension as TLVs.
> However, section 5 talks of "as new extensions are defined". I presume that
> what is meant here are "new extension TLVs" and not a different new
> Extension to replace the Extension defined in this document.
>
> Related to the ambiguity above, although the document makes it clear how an
> implementation is expected to behave if it doesn't understand the
> "extension" format, it doesn't define how an implementation should behave if
> it receives an extension TLV that it doesn't understand. I presume, for
> backwards compatibility, that the default behavior is that receivers MUST
> ignore extension TLVs that they don't understand and I think that needs to
> be specified here.

Right §4 says this: "Unsupported types MUST be ignored."

But that only supports your point -- the names are inconsistent and
can be confusing.  I'll make sure we make one more clarifying pass
before publication.

Thanks!

Alvaro.