Re: WGLC for draft-ietf-bfd-unaffiliated-echo (ending 7 April, 2023)

Jeffrey Haas <jhaas@pfrc.org> Thu, 06 April 2023 14:22 UTC

Return-Path: <jhaas@pfrc.org>
X-Original-To: rtg-bfd@ietfa.amsl.com
Delivered-To: rtg-bfd@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id C05F4C151541 for <rtg-bfd@ietfa.amsl.com>; Thu, 6 Apr 2023 07:22:01 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.899
X-Spam-Level:
X-Spam-Status: No, score=-1.899 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, RCVD_IN_DNSWL_BLOCKED=0.001, RCVD_IN_ZEN_BLOCKED_OPENDNS=0.001, SPF_PASS=-0.001] autolearn=ham autolearn_force=no
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 CbEae4QmSV9p for <rtg-bfd@ietfa.amsl.com>; Thu, 6 Apr 2023 07:21:59 -0700 (PDT)
Received: from slice.pfrc.org (slice.pfrc.org [67.207.130.108]) by ietfa.amsl.com (Postfix) with ESMTP id 52675C16952D for <rtg-bfd@ietf.org>; Thu, 6 Apr 2023 07:21:39 -0700 (PDT)
Received: from smtpclient.apple (104-10-90-238.lightspeed.livnmi.sbcglobal.net [104.10.90.238]) by slice.pfrc.org (Postfix) with ESMTPSA id B63891E037; Thu, 6 Apr 2023 10:21:38 -0400 (EDT)
Content-Type: text/plain; charset="us-ascii"
Mime-Version: 1.0 (Mac OS X Mail 16.0 \(3696.120.41.1.1\))
Subject: Re: WGLC for draft-ietf-bfd-unaffiliated-echo (ending 7 April, 2023)
From: Jeffrey Haas <jhaas@pfrc.org>
In-Reply-To: <00b301d966d7$d1094df0$731be9d0$@tsinghua.org.cn>
Date: Thu, 06 Apr 2023 10:21:38 -0400
Cc: rtg-bfd@ietf.org
Content-Transfer-Encoding: quoted-printable
Message-Id: <BD15D446-FBD4-4EB4-AAB2-F99D3282D55C@pfrc.org>
References: <20230321160207.GA7334@pfrc.org> <00b301d966d7$d1094df0$731be9d0$@tsinghua.org.cn>
To: Aijun Wang <wangaijun@tsinghua.org.cn>
X-Mailer: Apple Mail (2.3696.120.41.1.1)
Archived-At: <https://mailarchive.ietf.org/arch/msg/rtg-bfd/bWSAD5AujfQnRc9gEqZqbo5h6sM>
X-BeenThere: rtg-bfd@ietf.org
X-Mailman-Version: 2.1.39
Precedence: list
List-Id: "RTG Area: Bidirectional Forwarding Detection DT" <rtg-bfd.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/rtg-bfd>, <mailto:rtg-bfd-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/rtg-bfd/>
List-Post: <mailto:rtg-bfd@ietf.org>
List-Help: <mailto:rtg-bfd-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/rtg-bfd>, <mailto:rtg-bfd-request@ietf.org?subject=subscribe>
X-List-Received-Date: Thu, 06 Apr 2023 14:22:01 -0000

Aijun,


> On Apr 4, 2023, at 5:28 AM, Aijun Wang <wangaijun@tsinghua.org.cn> wrote:
> From the description of this document, the state machine of local device is
> conformed that described in RFC5880, the main standard parts of this
> document are the contents of related fields within the BFD ECHO Packet. If
> so, I suggested to point out these fields and its value in more explicit
> manner, to facilitate the implementation interoperability.

Perversely, the fact that this mechanism has an implementation "talking to itself" means the interoperability considerations are not a primary issue.

Providing additional detail to help illustrate the mechanism would be in-scope and perhaps helpful.  Did you have any explicit recommendations for the text?

> Should the section 2(update to RFC5880) be moved afterwards the section
> 3(Unaffiliated BFD Echo Procedures)? 
> And I am worrying that is it easy for the reader/implementer to keep up with
> the updated contents in current manner, because they must compare the two
> documents simultaneously? 

I agree that this would be a helpful change.  It would move the procedure ahead of the changes that impact the BFD normative text.

> 
> Is there any other better style to point out the update to RFC5880?

Unfortunately, this is a common problem for internet-drafts that impact protocol state machinery.  We have either the option of trying to issue a "patch" on the draft, as we're doing here, or do a -bis of the base RFC to more cleanly integrate the changes.

Since I think this feature is best documented as an optional extension at this time, the "patch" format is our best option.

-- Jeff