Re: [yang-doctors] Yangdoctors early review of draft-ietf-mpls-base-yang-10

Tarek Saad <> Wed, 23 October 2019 18:46 UTC

Return-Path: <>
Received: from localhost (localhost []) by (Postfix) with ESMTP id 98B5E120D61; Wed, 23 Oct 2019 11:46:29 -0700 (PDT)
X-Virus-Scanned: amavisd-new at
X-Spam-Flag: NO
X-Spam-Score: -1.998
X-Spam-Status: No, score=-1.998 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, FREEMAIL_FROM=0.001, RCVD_IN_DNSWL_NONE=-0.0001, SPF_HELO_NONE=0.001, SPF_PASS=-0.001, URIBL_BLOCKED=0.001] autolearn=ham autolearn_force=no
Authentication-Results: (amavisd-new); dkim=pass (2048-bit key)
Received: from ([]) by localhost ( []) (amavisd-new, port 10024) with ESMTP id 5jgReG5ygeIK; Wed, 23 Oct 2019 11:46:27 -0700 (PDT)
Received: from ( [IPv6:2a00:1450:4864:20::336]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by (Postfix) with ESMTPS id 132EF120D57; Wed, 23 Oct 2019 11:46:24 -0700 (PDT)
Received: by with SMTP id f22so44323wmc.2; Wed, 23 Oct 2019 11:46:23 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed;; s=20161025; h=from:to:cc:subject:thread-topic:thread-index:date:message-id :references:in-reply-to:accept-language:content-language :content-transfer-encoding:mime-version; bh=KHJm6gsfIrGv+oJROU4YwhLVdWNUZNv6QMmt0uSwWUk=; b=VQQMNl/tmNXddEKcSWHiNc4Yqr7OqWufrKzMI3AAzgaKd7Q3T1JnYX5mnZCH69uRQ7 85Xs8orjmWGvjln5y3ThOHvKYmCI4Cgoh1mBbLC+qnFqfTGImgBzjQOR6ZmOL1MjQ1sL 3EkBlNIFGue+9OWQbhrPCCGWtB5LvvU/AyWkGGwa0wY46YulvK2hufoc0N+DC3O89CG/ 1CDJlNW8jlgIrGirgz+p+wJMfMgTe6ilX0zZaDRmI1lOTHdID7ughbBOi4wWHImnp7SQ Bze1Xt2Y8W4sWuDG9tiBEW+jzsTaje+viyepoF4niTi3fgBQP5qHHkoKxy4tAk3w2LnF CnOw==
X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed;; s=20161025; h=x-gm-message-state:from:to:cc:subject:thread-topic:thread-index :date:message-id:references:in-reply-to:accept-language :content-language:content-transfer-encoding:mime-version; bh=KHJm6gsfIrGv+oJROU4YwhLVdWNUZNv6QMmt0uSwWUk=; b=O8i+AqonyPBOcKGmlTlUijwI6WPHd9LUZGJmztkonCMGEIoNqCT9OU3V6B+RQ9D9tg +JqIPTxhViRTBs2AEw9BdAaAjEvydb4QpuMOpiYmK4VrU3PfISHTDFRU9fGj52F4kt0o vKP0lf4e6L4WR2d8ju/Pt0exMrkOdavHXSkpoXOKttcIHDTRIZvHZj/zSzaoPeaXXuil MaZZuo8iiOoBl5uXEOxhbjNHReu2UodAci/CzmvusV9Ar3rPJhrzcPQOduTxPw1o8Zv7 vkWtpg6hT7uH2Y3jvwi173T39r7A9WmAWOg5vot/yuNj2nwMAosR47j3or9M0DHDkUjV yg9A==
X-Gm-Message-State: APjAAAVaceRBxCX7mt6PSNlY9R8KL0GiOsOeq0m+7lJdHttF8bWq8Vh+ mpWlBugLZagfpt/BLfsmmQy4IVhz
X-Google-Smtp-Source: APXvYqxl2o5enN4m5FP4uW5T2WrDJ7Lqga/185OYUm6FLxVcqIjZ86ZO3mUoAt1BkxJqzJd+EIWOxA==
X-Received: by 2002:a1c:544e:: with SMTP id p14mr1191523wmi.17.1571856382497; Wed, 23 Oct 2019 11:46:22 -0700 (PDT)
Received: from ([2603:1036:301:21db::5]) by with ESMTPSA id 65sm21386702wrs.9.2019. (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Wed, 23 Oct 2019 11:46:21 -0700 (PDT)
From: Tarek Saad <>
To: Ebben Aries <>, "" <>
CC: "" <>, "" <>, "" <>
Thread-Topic: Yangdoctors early review of draft-ietf-mpls-base-yang-10
Thread-Index: ATkwNDk0XwiEpmF5LrvZdM9A3YooWzAyQUMztzGp1Iw=
X-MS-Exchange-MessageSentRepresentingType: 1
Date: Wed, 23 Oct 2019 18:46:19 +0000
Message-ID: <>
References: <> <>
In-Reply-To: <>
Accept-Language: en-US
Content-Language: en-US
X-MS-Exchange-Organization-SCL: -1
X-MS-Exchange-Organization-RecordReviewCfmType: 0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: base64
MIME-Version: 1.0
Archived-At: <>
Subject: Re: [yang-doctors] Yangdoctors early review of draft-ietf-mpls-base-yang-10
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: Email list of the yang-doctors directorate <>
List-Unsubscribe: <>, <>
List-Archive: <>
List-Post: <>
List-Help: <>
List-Subscribe: <>, <>
X-List-Received-Date: Wed, 23 Oct 2019 18:46:30 -0000

Hi Ebben,

Can you please advise whether you are OK with revision 11 which addresses your review comments?


On 9/12/19, 9:18 AM, "Tarek Saad" <>; wrote:

    Hi Ebben,
    Thanks for your review and comments. We have uploaded a new revision of the document that addresses the comments.
    Inline for more details.
    On 8/18/19, 4:19 PM, "Ebben Aries via Datatracker" <>; wrote:
        Reviewer: Ebben Aries
        Review result: On the Right Track
        1 module in this draft:
        - ietf-mpls@2019-02-24.yang
        No YANG compiler errors or warnings (pyang 2.0.1, yanglint 1.1.40, confdc 6.6.3)
        Module ietf-mpls@2019-02-24.yang:
        - Remove WG Chairs from contact information per

    [TS]: done.
        - 'ietf-interfaces' import should reference RFC8343 rather
    [TS]: updated.
        - Must clause for start/end-label is incorrect.  Would suggest moving this
          must statement underneath the `leaf end-label` as well:
          leaf end-label {
              type rt-types:mpls-label;
              must '. >= ../start-label' {
                    "The end-label must be greater than or equal " +
                    "to start-label";
              description "Label-block end";
    [TS]: OK, I moved this check to under end-label. I added similar check for start-label too.
        - Use of 'state' container under '/routing/mpls/label-blocks/label-block/state'
          These nodes could sit as r/o nodes by the looks of it directly under the
          label-block list.  In addition, do these nodes need '-count' suffixes?
          Should they rather be of type `yang:counter32`?
    [TS]: removed the 'state' container and updated to directly go under list. Also changed to type to yang:counter32.
        - Is there any intention to define any surrounding features?
    [TS]: updated.
        General comments/minor nits on the draft/modules:
        - Section 1: s/feauture/feature/
        - Section 2.1: s/the the/the/
        - Section 2.1: 'labeled' vs. 'labelled'
        - Section 2.4: s/followinig/following/
        - Module line 354/367: This is for the 'active-route' action statement rather
     [TS]: addressed the typos, thanks.
    Tarek (for authors)