Re: [mpls] [yang-doctors] Yangdoctors last call review of draft-ietf-mpls-ldp-yang-06

Martin Bjorklund <> Tue, 15 October 2019 12:26 UTC

Return-Path: <>
Received: from localhost (localhost []) by (Postfix) with ESMTP id 3ED2A12010D; Tue, 15 Oct 2019 05:26:07 -0700 (PDT)
X-Virus-Scanned: amavisd-new at
X-Spam-Flag: NO
X-Spam-Score: -1.9
X-Spam-Status: No, score=-1.9 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, SPF_HELO_NONE=0.001, SPF_PASS=-0.001] autolearn=ham autolearn_force=no
Received: from ([]) by localhost ( []) (amavisd-new, port 10024) with ESMTP id 2RDBYG3nm6OL; Tue, 15 Oct 2019 05:26:05 -0700 (PDT)
Received: from ( []) by (Postfix) with ESMTP id 129CD12008A; Tue, 15 Oct 2019 05:26:02 -0700 (PDT)
Received: from localhost (unknown []) by (Postfix) with ESMTPSA id 659EC1AE02BD; Tue, 15 Oct 2019 14:26:00 +0200 (CEST)
Date: Tue, 15 Oct 2019 14:25:32 +0200 (CEST)
Message-Id: <>
From: Martin Bjorklund <>
In-Reply-To: <>
References: <>
X-Mailer: Mew version 6.8 on Emacs 25.2
Mime-Version: 1.0
Content-Type: Text/Plain; charset=us-ascii
Content-Transfer-Encoding: 7bit
Archived-At: <>
Subject: Re: [mpls] [yang-doctors] Yangdoctors last call review of draft-ietf-mpls-ldp-yang-06
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: Multi-Protocol Label Switching WG <>
List-Unsubscribe: <>, <>
List-Archive: <>
List-Post: <>
List-Help: <>
List-Subscribe: <>, <>
X-List-Received-Date: Tue, 15 Oct 2019 12:26:07 -0000


Some comments inline.

Jan Lindblad via Datatracker <>; wrote:

> #4) Weakly defined types for neighbor-list-ref, prefix-list-ref, peer-list-ref
> These types are not leafrefs, but strings without any YANG substatements to
> define the format. The only thing the description does is to claim that the
> entities they refer to are outside the scope of this document. For an
> operator/programmer encountering this type, that isn't very helpful, and is not
> going to be interoperable.
>   typedef neighbor-list-ref {
>     type string;
>     description
>       "A type for a reference to a neighbor address list.
>        The string value is the name identifier for uniquely
>        identifying the referenced address list, which contains a list
>        of addresses that a routing policy can applied. The definition
>        of such an address list is outside the scope of this
>        document.";
>   }
> I'm not sure if this is fixable by sharpening the YANG module, but maybe more
> could be done to guide a confused reader. What would the user do to find out
> the format of this type and valid values? Add to the description.

If the target of such a reference is outside the scope of this
document, it is reasonable to also leave this leaf as outside the
scope of this document.  When a new module defines such a list, it can
easily augment this module with a proper leafref.   I.e., I suggest
that you can remove this leaf and perhaps explain in text how a future
module can add the reference.

> #6) MD5 key
> There is a leaf md5-key of type string. Is this leaf sensitive from a security
> point of view? A plaintext string would not be ideal if that is the case.
> Choose a crypto type instead.

Or perhaps mark the node with nacm:default-deny-all.