Re: [Last-Call] Secdir last call review of draft-ietf-lsr-ospf-bfd-strict-mode-07

Wes Hardaker <wes@hardakers.net> Mon, 26 September 2022 18:15 UTC

Return-Path: <wes@hardakers.net>
X-Original-To: last-call@ietfa.amsl.com
Delivered-To: last-call@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id D8EEEC14CE25; Mon, 26 Sep 2022 11:15:55 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -6.907
X-Spam-Level:
X-Spam-Status: No, score=-6.907 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, 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
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 jktU4LL-qeyn; Mon, 26 Sep 2022 11:15:52 -0700 (PDT)
Received: from mail.hardakers.net (mail.hardakers.net [168.150.192.181]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 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 884D8C14CE23; Mon, 26 Sep 2022 11:15:51 -0700 (PDT)
Received: from localhost (unknown [10.0.0.9]) by mail.hardakers.net (Postfix) with ESMTPA id 8008720AE1; Mon, 26 Sep 2022 11:15:50 -0700 (PDT)
From: Wes Hardaker <wes@hardakers.net>
To: Ketan Talaulikar <ketant.ietf@gmail.com>
Cc: Wes Hardaker <wjhns1@hardakers.net>, secdir@ietf.org, draft-ietf-lsr-ospf-bfd-strict-mode.all@ietf.org, last-call@ietf.org, raw@ietf.org
References: <yblzgewx0dg.fsf@wx.hardakers.net> <CAH6gdPwvr8uSdOXT5SuqWXH5JjT4a3jqS0WZLoLgKCk=MZv7DA@mail.gmail.com>
Date: Mon, 26 Sep 2022 11:15:50 -0700
In-Reply-To: <CAH6gdPwvr8uSdOXT5SuqWXH5JjT4a3jqS0WZLoLgKCk=MZv7DA@mail.gmail.com> (Ketan Talaulikar's message of "Tue, 20 Sep 2022 19:50:08 +0530")
Message-ID: <yblczbic889.fsf@wd.hardakers.net>
User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/27.2 (gnu/linux)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: quoted-printable
Archived-At: <https://mailarchive.ietf.org/arch/msg/last-call/49UrxLb5wb8Nm5SpmbpRubNBZt0>
Subject: Re: [Last-Call] Secdir last call review of draft-ietf-lsr-ospf-bfd-strict-mode-07
X-BeenThere: last-call@ietf.org
X-Mailman-Version: 2.1.39
Precedence: list
List-Id: IETF Last Calls <last-call.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/last-call>, <mailto:last-call-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/last-call/>
List-Post: <mailto:last-call@ietf.org>
List-Help: <mailto:last-call-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/last-call>, <mailto:last-call-request@ietf.org?subject=subscribe>
X-List-Received-Date: Mon, 26 Sep 2022 18:15:55 -0000

Ketan Talaulikar <ketant.ietf@gmail.com> writes:

>     - In multiple places it talks about "strict-mode is enabled on the
>       link" or similar.  It is unclear from the context where this
>       enabling is happening, and I'd be tempted to add a bit more
>       operational context such as "strict-mode is enabled by the
>       operator..." or similar.
> 
> KT> It is understood that it is the operator that is enabling BFD or BFD strict
> mode. We can clarify this in a couple of places like the introduction and the
> procedures sections. I don't think it would help much to insert "operator" at
> every sentence in the document which discusses enablement. I hope that would
> address your comment.

Yes, if it's well mentioned somewhere then it doesn't need to be
everywhere, I agree.

>     - In the state discussions the phrase "or higher" is used to indicate
>       multiple states.  The original OSPF RFC generally uses different
>       terminology: "or greater".  It might be wise to switch to the
>       original terminology instead.
> 
> KT> RFC2328 uses the word "higher" in describing the Init state in section 10.1
> but then uses "greater" in the other state descriptions. We will keep the word
> "higher" in the update Init state, but perhaps change to "greater" in another
> place where we reference 2-way. in the document.

I'll leave it to you to decide where the right place to make terminology
swaps are of course.

-- 
Wes Hardaker                                     
My Pictures:       http://photos.capturedonearth.com/