Re: [Lsr] Mirja Kühlewind's No Objection on draft-ietf-ospf-yang-26: (with COMMENT)

Mirja Kuehlewind <ietf@kuehlewind.net> Tue, 20 August 2019 15:51 UTC

Return-Path: <ietf@kuehlewind.net>
X-Original-To: lsr@ietfa.amsl.com
Delivered-To: lsr@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 4D408120987; Tue, 20 Aug 2019 08:51:59 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.897
X-Spam-Level:
X-Spam-Status: No, score=-1.897 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, SPF_HELO_NONE=0.001, SPF_NONE=0.001, URIBL_BLOCKED=0.001] autolearn=ham autolearn_force=no
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 7nXqDltpLqGQ; Tue, 20 Aug 2019 08:51:57 -0700 (PDT)
Received: from wp513.webpack.hosteurope.de (wp513.webpack.hosteurope.de [IPv6:2a01:488:42:1000:50ed:8223::]) (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 3209C12096D; Tue, 20 Aug 2019 08:51:57 -0700 (PDT)
Received: from 200116b82c91c600bdef7f40a09bd364.dip.versatel-1u1.de ([2001:16b8:2c91:c600:bdef:7f40:a09b:d364]); authenticated by wp513.webpack.hosteurope.de running ExIM with esmtpsa (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) id 1i06QL-0005sY-Nn; Tue, 20 Aug 2019 17:51:53 +0200
Content-Type: text/plain; charset=utf-8
Mime-Version: 1.0 (Mac OS X Mail 12.4 \(3445.104.11\))
From: Mirja Kuehlewind <ietf@kuehlewind.net>
In-Reply-To: <932F3922-B99A-4AC1-AEA7-105688ABBD9C@cisco.com>
Date: Tue, 20 Aug 2019 17:51:53 +0200
Cc: The IESG <iesg@ietf.org>, "draft-ietf-ospf-yang@ietf.org" <draft-ietf-ospf-yang@ietf.org>, Stephane Litkowski <stephane.litkowski@orange.com>, "aretana.ietf@gmail.com" <aretana.ietf@gmail.com>, "lsr-chairs@ietf.org" <lsr-chairs@ietf.org>, "lsr@ietf.org" <lsr@ietf.org>
Content-Transfer-Encoding: quoted-printable
Message-Id: <6FAECED8-5AE0-459E-B4F5-1CBB45D58D01@kuehlewind.net>
References: <156623193076.19944.688705526703341576.idtracker@ietfa.amsl.com> <932F3922-B99A-4AC1-AEA7-105688ABBD9C@cisco.com>
To: "Acee Lindem (acee)" <acee@cisco.com>
X-Mailer: Apple Mail (2.3445.104.11)
X-bounce-key: webpack.hosteurope.de;ietf@kuehlewind.net;1566316317;4278b190;
X-HE-SMSGID: 1i06QL-0005sY-Nn
Archived-At: <https://mailarchive.ietf.org/arch/msg/lsr/RNUGsjkRJnDG5ylFsVGiSeUn0tg>
Subject: Re: [Lsr] =?utf-8?q?Mirja_K=C3=BChlewind=27s_No_Objection_on_draft-i?= =?utf-8?q?etf-ospf-yang-26=3A_=28with_COMMENT=29?=
X-BeenThere: lsr@ietf.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: Link State Routing Working Group <lsr.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/lsr>, <mailto:lsr-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/lsr/>
List-Post: <mailto:lsr@ietf.org>
List-Help: <mailto:lsr-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/lsr>, <mailto:lsr-request@ietf.org?subject=subscribe>
X-List-Received-Date: Tue, 20 Aug 2019 15:51:59 -0000

Hi Acee,

Thanks for these changes/additions.

One comment below.

> On 20. Aug 2019, at 17:05, Acee Lindem (acee) <acee@cisco.com>; wrote:
> 
> Hi Mirja, 
> 
> On 8/19/19, 12:25 PM, "Mirja Kühlewind via Datatracker" <noreply@ietf.org>; wrote:
> 
>    Mirja Kühlewind has entered the following ballot position for
>    draft-ietf-ospf-yang-26: No Objection
> 
>    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/iesg/statement/discuss-criteria.html
>    for more information about IESG DISCUSS and COMMENT positions.
> 
> 
>    The document, along with other ballot positions, can be found here:
>    https://datatracker.ietf.org/doc/draft-ietf-ospf-yang/
> 
> 
> 
>    ----------------------------------------------------------------------
>    COMMENT:
>    ----------------------------------------------------------------------
> 
>    Just two quick questions about references:
> 
>    - Is there no reference for mtu-ignore (see section 2.4)? If not, can you
>    further describe what exactly would be disabled?
> 
> I added "specified in section 10.6 of [RFC2328]." to the sentence. 
> 
> 
>    - Also is there no reference for OSPF Non-Stop Routing (NSR) (see section
>    2.4)...?
> 
> While many vendors have implemented this feature, there is no standard as the goal is to restart without other OSPF routers being impacted. We did add the additional text to describe the feature and differences with OSPF Graceful Restart. 
> 
>    And one more comment:
> 
>    In the interface-common-config part (p76 and p77) you provide example or
>    default values for various intervals and delays. Where does those values come
>    from? Would it be possible to provide a reference/RFC that specifies actual
>    default values? Especially when you specify something normatively ("The value
>    MUST be greater than 'hello-interval'.") it would be good to provide a
>    reference!  Do any specification maybe also specify min and max value? If so,
>    you should mention them here as well! If not would it make sense to recommend
>    min and max values? If possible I would strongly support to describe min and
>    max values as well!
> 
> The whole question of defaults generated quite a lot of discussion as there isn't really any one-size-fits all default and the sample values are very conservative. This is how we ended up with sample values in the text rather than YANG defaults. Since the protocol specification doesn’t specify min and max values, we were hesitant to specify them in the YANG model more than 20 years later. Also, many vendors have supported sub-second hellos. This wasn't a good idea and it has been supplanted by BFD. I will add RFC 2328 Appendix C as a reference for the sample values. 

Good to hear that this was discussed! I understand the problem and a YANG is not the right place to “change” the spec. Adding a reference would be good!

One more question regarding the sentence with normative language above: Is that already specified in some other spec and it would make sense to not use normative language here but provide the reference? 

Mirja



> 
> Thanks,
> Acee
> 
> 
> 
> 
> 
>