[RTG-DIR] RTGDIR Last Call Review of draft-ietf-rtgwg-vrrp-rfc5798bis-13.txt
Donald Eastlake <d3e3e3@gmail.com> Tue, 12 December 2023 23:22 UTC
Return-Path: <d3e3e3@gmail.com>
X-Original-To: rtg-dir@ietfa.amsl.com
Delivered-To: rtg-dir@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 969FEC14CEE3; Tue, 12 Dec 2023 15:22:21 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -6.856
X-Spam-Level:
X-Spam-Status: No, score=-6.856 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_ENVFROM_END_DIGIT=0.25, FREEMAIL_FROM=0.001, RCVD_IN_DNSWL_HI=-5, RCVD_IN_ZEN_BLOCKED_OPENDNS=0.001, SPF_HELO_NONE=0.001, SPF_PASS=-0.001, T_SCC_BODY_TEXT_LINE=-0.01, URIBL_DBL_BLOCKED_OPENDNS=0.001, URIBL_ZEN_BLOCKED_OPENDNS=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 ([50.223.129.194]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id tkWxp38AiA02; Tue, 12 Dec 2023 15:22:17 -0800 (PST)
Received: from mail-ed1-x52b.google.com (mail-ed1-x52b.google.com [IPv6:2a00:1450:4864:20::52b]) (using TLSv1.3 with cipher TLS_AES_128_GCM_SHA256 (128/128 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id A91DBC14F5E9; Tue, 12 Dec 2023 15:22:07 -0800 (PST)
Received: by mail-ed1-x52b.google.com with SMTP id 4fb4d7f45d1cf-54cc60f3613so8048112a12.2; Tue, 12 Dec 2023 15:22:07 -0800 (PST)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1702423326; x=1703028126; darn=ietf.org; h=content-transfer-encoding:cc:to:subject:message-id:date:from :mime-version:from:to:cc:subject:date:message-id:reply-to; bh=tg/FzrM+3DbIe0rMd1IuNu1YZ4HUgmUB45IMgBN11OA=; b=YPvAPzYZ7G49bxNukukyD5eIpZkb6VIsRq/BPq4IHoUnTZsYO78pDLUOVTM4K4Szzm pz65iFih+i9JCuMhb08gh9ND1/zqGJPg2JdDgqQQiJ8GSiNdfjmMxhEtUoK2nGunBmlS agGa9hk+dlsgVeq7/et+tn3v/S807qVRu3LDERO/Ps59mZcduAi2tIE3xc3wpKkn6hFa kjzqltOGXZ9YOl/uWaR9VUmfIad7eh1o9coatr326MHNmL1q2R3GGKaa93KFXrK8z4nP xDAj1V7+B3/oT8eh3Lt6iRRCZ35/DFzj4tIIRKPiW413QxZc4zplXBE6/soaLbuTxlOw hAdw==
X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1702423326; x=1703028126; h=content-transfer-encoding:cc:to:subject:message-id:date:from :mime-version:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=tg/FzrM+3DbIe0rMd1IuNu1YZ4HUgmUB45IMgBN11OA=; b=CfrVZ3dDkKpu1GHPN11eXqmtyTZtSaoe9v2a5L7mxWAjidpDvZwhRtuTbsJG4rOotP a1LHGROSD7QAEqgNonbgBV5h/RQqXdnub7T+npyA4Tx2hymJnQSYgWp0HYJtxDewNxEe pP/AnFAFlukNjjGoRuICqGJ4FFHcxIGHQ7h7NnE7BwbtOXFN4xJhTy2gjCy2046U6LrU k47u8Mnnl0oR0RWITct4ELfopNP9tYEqDutIMI0BLMRKtJ/73eZIHgzYt9FAkLqSj1Wj 8wjhMxVhOlp+2gTzB/dYVxJN4k9JZLS2GOLjxmXj4fSVfytlTnqpMuHPma+dVUXzFNZs vV2g==
X-Gm-Message-State: AOJu0YzAGPlBr/ydHOIcBvUZ0TAD2bfKg3oxMXvT3aDkQ0xqOl3u6CEW v1kabcKEkmKf5nzSbin2ldlQU0lfArWxJsi26g57DkvjWS0=
X-Google-Smtp-Source: AGHT+IGRqbqbRUFGG4TedjmNQdpFaKcSnPge29SJTDhvwjh8jSONnCP/eWUX8TGNJB60d03gni+2tphtYxMo/EdQvQg=
X-Received: by 2002:aa7:d884:0:b0:54c:79ec:561c with SMTP id u4-20020aa7d884000000b0054c79ec561cmr3517281edq.73.1702423325514; Tue, 12 Dec 2023 15:22:05 -0800 (PST)
MIME-Version: 1.0
From: Donald Eastlake <d3e3e3@gmail.com>
Date: Tue, 12 Dec 2023 18:21:53 -0500
Message-ID: <CAF4+nEFFgKYA1CCCsNyJ5rYjQVuTc=Cz7nBFcQQjFLNteHTWcg@mail.gmail.com>
To: "<rtg-ads@ietf.org> (rtg-ads@ietf.org)" <rtg-ads@ietf.org>
Cc: rtg-dir@ietf.org, draft-ietf-rtgwg-vrrp-rfc5798bis.all@ietf.org, RTGWG <rtgwg@ietf.org>, Last Call <last-call@ietf.org>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
Archived-At: <https://mailarchive.ietf.org/arch/msg/rtg-dir/UuobBuZQxRAzb25IxLWwphofpvY>
Subject: [RTG-DIR] RTGDIR Last Call Review of draft-ietf-rtgwg-vrrp-rfc5798bis-13.txt
X-BeenThere: rtg-dir@ietf.org
X-Mailman-Version: 2.1.39
Precedence: list
List-Id: Routing Area Directorate <rtg-dir.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/rtg-dir>, <mailto:rtg-dir-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/rtg-dir/>
List-Post: <mailto:rtg-dir@ietf.org>
List-Help: <mailto:rtg-dir-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/rtg-dir>, <mailto:rtg-dir-request@ietf.org?subject=subscribe>
X-List-Received-Date: Tue, 12 Dec 2023 23:22:21 -0000
Hello, I have been selected as the Routing Directorate reviewer for draft-ietf-rtgwg-vrrp- rfc5798bis-13.txt. The Routing Directorate seeks to review all routing or routing-related drafts as they pass through IETF last call and IESG review, and sometimes on special request. The purpose of the review is to provide assistance to the Routing ADs. For more information about the Routing Directorate, please see https://wiki.ietf.org/group/rtg/RtgDir Although these comments are primarily for the use of the Routing ADs, it would be helpful if you could consider them along with any other IETF Last Call comments that you receive and strive to resolve them through discussion or by updating the draft. Document: draft-ietf-rtgwg-vrrp-rfc5798bis-13.txt Reviewer: Donald Eastlake 3rd Review Date: 12 December 2023 IETF LC End Date: 10 December 2023 Intended Status: Standards Track Summary: I have some minor concerns about this document that I think should be resolved before publication. Comments: --------- This is an updated replacement for RFC 5798 on VRRP v2 and v3 making the changes listed in Section 1.1. As would be expected in an update of a long standing and widely deployed protocol, I found no technical issues. Major Issues: ------------- No major issues found. Minor Issues: ------------- Abstract/Introduction: Since this document obsoletes RFC 5798, it seems to me RFC 5798 should be *the* reference in the text at the beginning of the Abstract and the Introduction. I understand why RFC 5798 referred to RFC 3768 and to the "Virtual Router Redundancy Protocol for IPv6" draft, but those were subsumed and replaced by RFC 5798. I think there need be no reference to RFC 3768 or that VRRP IPv6 draft in this document except, perhaps, as a historic mention in the Acknowledgements section. It looks like the beginning Abstract/Introduction text of RFC 3768 was simply copied into this document and then updated a bit at the beginning of the Abstract but not updated at the beginning of the Introduction. I believe this will be confusing for some readers and should be fixed. Section 7.3 Virtual Router MAC Address: There should be an informational reference to draft-ietf-intarea-rfc7042bis-11 (currently in the RFC Editor's queue in the EDIT state), probably right after "IEEE 802 MAC Address" in the first sentence. Section 8.2.3 Router Advertisements: I think the "must" in the first paragraph and the "should not" in the second paragraph should be MUST and SHOULD NOT respectively. Section 11 IANA Considerations: This needs to direct IANA to update references to RFC 5798. Suggest adding wording like: "IANA is requested to update all IANA Registry references to [RFC5798] to be references to [this document]." (Alternatively, instead of “all IANA Registries” it could list the protocol number, 48-bit MAC address block, IPv4 multicast address local network control block, and IPv6 link-local scope multicast addresses registries.) Nits: ----- Abstract: The second paragraph of the abstract has no technical significance. I don't think it should be in the Abstract or in the first part of the Introduction (between the 1. and 1.1 subject lines). However, it makes a lot of sense in Section 1.1 so I think it should be moved there. Section 1.1, Point 2: I believe it is good practice to include the Errata fixed by a revision in the Informational References. As an example, RFC 7176 fixes Errata 2869 in RFC 6326 which it obsoletes and thus the Informational References for RFC 7176 include the following: [Err2869] RFC Errata, Errata ID 2869, RFC 6326, <http://www.rfc-editor.org>. Section 5.1: The Figure should have a Figure number and caption. Section 5.2.5 IPvX Addr Count: Says the minimum value of the count is 1 but does not say what to do if it is zero. Suggest saying here that the message is ignored. (Admittedly, this is covered many pages later in Section 7.1.) Section 5.2.8 Checksum: I guess the final paragraph overrides the 2nd paragraph, but I think it would be better to restructure the 2nd, 3rd, and 4th paragraphs into two paragraphs, one each for IPv4 and IPv6. Section 6.1 Parameters Per Virtual Router: Although the effect of the other parameters is generally spelled out, it doesn't explain "Skew_Time". Suggested adding some more explanation and/or a reference to Section 8.3.2. Section 8.2.2 ND Neighbor Solicitation: The last paragraph of this Section is the only place "DAD" is used so I would just delete the acronym and spell it out on second use like it is on first use. Section 8.3.1 Potential Forwarding Loop: There is a word missing in the final one-sentence paragraph. Suggest "…Routers to these forwarding…" -> "…Routers avoid to these forwarding…". Section 11 IANA Considerations: The reference to [RFC7042] should be replaced by a reference to the rfc7042bis draft. Thanks, Donald =============================== Donald E. Eastlake 3rd +1-508-333-2270 (cell) 2386 Panoramic Circle, Apopka, FL 32703 USA d3e3e3@gmail.com
- Re: [RTG-DIR] RTGDIR Last Call Review of draft-ie… Donald Eastlake
- [RTG-DIR] RTGDIR Last Call Review of draft-ietf-r… Donald Eastlake
- Re: [RTG-DIR] RTGDIR Last Call Review of draft-ie… Acee Lindem
- Re: [RTG-DIR] RTGDIR Last Call Review of draft-ie… Acee Lindem
- Re: [RTG-DIR] RTGDIR Last Call Review of draft-ie… Acee Lindem
- Re: [RTG-DIR] RTGDIR Last Call Review of draft-ie… Donald Eastlake
- Re: [RTG-DIR] RTGDIR Last Call Review of draft-ie… James Guichard
- Re: [RTG-DIR] RTGDIR Last Call Review of draft-ie… Acee Lindem
- Re: [RTG-DIR] RTGDIR Last Call Review of draft-ie… Yingzhen Qu