[yang-doctors] Yangdoctors last call review of draft-ietf-ospf-yang-09
Ladislav Lhotka <lhotka@nic.cz> Wed, 06 December 2017 11:26 UTC
Return-Path: <lhotka@nic.cz>
X-Original-To: yang-doctors@ietf.org
Delivered-To: yang-doctors@ietfa.amsl.com
Received: from ietfa.amsl.com (localhost [IPv6:::1]) by ietfa.amsl.com (Postfix) with ESMTP id A3B90126B7E; Wed, 6 Dec 2017 03:26:47 -0800 (PST)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
From: Ladislav Lhotka <lhotka@nic.cz>
To: yang-doctors@ietf.org
Cc: draft-ietf-ospf-yang.all@ietf.org, ietf@ietf.org, ospf@ietf.org
X-Test-IDTracker: no
X-IETF-IDTracker: 6.67.0
Auto-Submitted: auto-generated
Precedence: bulk
Message-ID: <151255960762.30655.17225294251460480729@ietfa.amsl.com>
Date: Wed, 06 Dec 2017 03:26:47 -0800
Archived-At: <https://mailarchive.ietf.org/arch/msg/yang-doctors/yJdw4fdz1OnZLJazV7tyRLI8vP8>
Subject: [yang-doctors] Yangdoctors last call review of draft-ietf-ospf-yang-09
X-BeenThere: yang-doctors@ietf.org
X-Mailman-Version: 2.1.22
List-Id: Email list of the yang-doctors directorate <yang-doctors.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/yang-doctors>, <mailto:yang-doctors-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/yang-doctors/>
List-Post: <mailto:yang-doctors@ietf.org>
List-Help: <mailto:yang-doctors-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/yang-doctors>, <mailto:yang-doctors-request@ietf.org?subject=subscribe>
X-List-Received-Date: Wed, 06 Dec 2017 11:26:48 -0000
Reviewer: Ladislav Lhotka Review result: Ready with Issues The data model defined in this document is a massive piece of work: it consists of 11 YANG modules and defines around 1200 schema nodes. The "ietf-ospf@2017-10-30" module is compatible with the NMDA architecture. **** Comments 1. Unless there is a really compelling reason not to do so, the "ietf-ospf" should declare YANG version 1.1. For one, "ietf-routing" that is being augmented by "ietf-ospf" already declares this version. Some of my suggestions below also assume version 1.1. 2. The "ietf-ospf" can work only with the new NMDA-compatible revisions of some modules, such as "ietf-interfaces" and "ietf-routing". I understand it is not desirable to import such modules by revision, but at least it should be mentioned in a description attached to every such import. 3. Maybe the draft could mention that implementations should supply a default routing domain as a system-controlled resource. 4. In "when" expressions, the module uses literal strings for identities. This is known to be problematic, the XPath functions derived-from() or derived-from-or-self() should be used instead. 5. Some enumerations, such as "packet-type" and "if-state-type" define enum identifiers with uppercase letters and/or underscores, for example "Database-Description" or "LONG_WAIT". RFC6087bis recommends that only lowercase letters, numbers and dashes. I think this convention should be observed despite the fact that the current names are traditionally used in OSPF specs. The "ietf-routing" module also defines "router-id" even though the documents use "Router ID". 6. The types of LSA headers are modelled as integers. While OSPF gurus probably know these numbers by heart, it is not very reader-frienly. So at least some references to documents defining these numbers should be provided, but my suggestion is to consider implementing them with identities. It seems it might also be useful to define some "abstract" identities for these types. For example, if "opaque-lsa" is defined, then the definition of container "opaque" could simply use when "derived-from(../../header/type, 'ospf:opaque-lsa')"; instead of when "../../header/type = 9 or " + "../../header/type = 10 or " + "../../header/type = 11"; 7. The title of sec. 2.9 should be "OSPF Notifications" rather than "OSPF notification".
- [yang-doctors] Yangdoctors last call review of dr… Ladislav Lhotka
- Re: [yang-doctors] Yangdoctors last call review o… Martin Bjorklund
- Re: [yang-doctors] Yangdoctors last call review o… Ladislav Lhotka
- Re: [yang-doctors] Yangdoctors last call review o… Acee Lindem (acee)
- Re: [yang-doctors] Yangdoctors last call review o… Acee Lindem (acee)
- Re: [yang-doctors] Yangdoctors last call review o… Ladislav Lhotka
- Re: [yang-doctors] Yangdoctors last call review o… Martin Bjorklund
- Re: [yang-doctors] Yangdoctors last call review o… Ladislav Lhotka
- Re: [yang-doctors] Yangdoctors last call review o… Ladislav Lhotka
- Re: [yang-doctors] Yangdoctors last call review o… Martin Bjorklund
- Re: [yang-doctors] Yangdoctors last call review o… Ladislav Lhotka
- Re: [yang-doctors] Yangdoctors last call review o… Martin Bjorklund
- Re: [yang-doctors] Yangdoctors last call review o… Martin Bjorklund
- Re: [yang-doctors] Yangdoctors last call review o… Ladislav Lhotka
- Re: [yang-doctors] Yangdoctors last call review o… Martin Bjorklund
- Re: [yang-doctors] Yangdoctors last call review o… Ladislav Lhotka
- Re: [yang-doctors] Yangdoctors last call review o… Martin Bjorklund
- Re: [yang-doctors] Yangdoctors last call review o… Ladislav Lhotka
- Re: [yang-doctors] Yangdoctors last call review o… Andy Bierman
- Re: [yang-doctors] Yangdoctors last call review o… Ladislav Lhotka
- Re: [yang-doctors] Yangdoctors last call review o… Ladislav Lhotka
- Re: [yang-doctors] Yangdoctors last call review o… Martin Bjorklund
- Re: [yang-doctors] Yangdoctors last call review o… Juergen Schoenwaelder
- Re: [yang-doctors] Yangdoctors last call review o… Ladislav Lhotka
- Re: [yang-doctors] Yangdoctors last call review o… Juergen Schoenwaelder
- Re: [yang-doctors] Yangdoctors last call review o… Ladislav Lhotka
- Re: [yang-doctors] Yangdoctors last call review o… Andy Bierman
- Re: [yang-doctors] Yangdoctors last call review o… Ladislav Lhotka
- Re: [yang-doctors] Yangdoctors last call review o… Andy Bierman
- Re: [yang-doctors] Yangdoctors last call review o… Acee Lindem (acee)
- Re: [yang-doctors] Yangdoctors last call review o… Acee Lindem (acee)
- Re: [yang-doctors] Yangdoctors last call review o… Acee Lindem (acee)
- Re: [yang-doctors] Yangdoctors last call review o… Martin Bjorklund
- Re: [yang-doctors] Yangdoctors last call review o… Acee Lindem (acee)
- Re: [yang-doctors] Yangdoctors last call review o… Acee Lindem (acee)
- Re: [yang-doctors] Yangdoctors last call review o… Acee Lindem (acee)
- Re: [yang-doctors] Yangdoctors last call review o… Acee Lindem (acee)
- Re: [yang-doctors] Yangdoctors last call review o… Acee Lindem (acee)
- Re: [yang-doctors] Yangdoctors last call review o… Martin Bjorklund
- Re: [yang-doctors] Yangdoctors last call review o… Acee Lindem (acee)
- Re: [yang-doctors] [Lsr] Yangdoctors last call re… Rob Shakir
- Re: [yang-doctors] [Lsr] Yangdoctors last call re… Acee Lindem (acee)
- Re: [yang-doctors] Yangdoctors last call review o… Christian Hopps