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

Tarek Saad <> Thu, 12 September 2019 13:18 UTC

Return-Path: <>
Received: from localhost (localhost []) by (Postfix) with ESMTP id 4F8F0120041; Thu, 12 Sep 2019 06:18:08 -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 b5UEE0P6gWIu; Thu, 12 Sep 2019 06:18:05 -0700 (PDT)
Received: from ( [IPv6:2607:f8b0:4864:20::d34]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by (Postfix) with ESMTPS id 3634C12002F; Thu, 12 Sep 2019 06:18:02 -0700 (PDT)
Received: by with SMTP id b136so54432806iof.3; Thu, 12 Sep 2019 06:18:02 -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=N2XUqEdbN7bvsk+zQxf7o/Q8SqAMYjlq+AkDB8sbIH0=; b=a4m0pAtXAIo2EcNXpt+oliAi5Cm8Zk995Sx+zgMjdXPKMh/eUKZ+qwL7k3pRlvlfZw 5gKsHUgGiFCWXa8iE0VYag/f0IKoUweVcog/BPnyDRzHp93LSub6N06HgzWra/RA7nTo lPKm9hc0U3O6K1EDKqAiYfrRn4evzsBsmRSkBG/i/gkAXP4QMZbFhCmcpsc/7E21HArH kXLfdPph3GWfP6PN0BYIoXrfxrOxKyL8fmnf9oZpNnhPJZfyTP30W+JurCreVYl+dnSg 5UiDPkDdvHprydcHy0cvwgKPvt9wpVZ++qtQb3O7vOocyvlvPWa+u+Xa9XtxjCdIOWca a6Lw==
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=N2XUqEdbN7bvsk+zQxf7o/Q8SqAMYjlq+AkDB8sbIH0=; b=aUwEc2fhjF0t3tPJvhtpviwy5rNf/Jw4GMiydDuAY8yrekkKSOpv3+OfmJQmJ/GG0F VlrhRWbN2cgsaNzNCQ8flg4hAje1+RQ3/yIAPGfVfdVm8oqbX7jHKiVgJpuXrFJaZhkk m3xWrHSYcMD1+jUqvXeHZ0a8Zc0wPTb9lf1Fahh0IZ1/0BKr+PsCp+/luO/VAFwdOjrF DnBSaFuNWq6+SUtNyJcQOK/yBfTNt0dyPNJtPsmY6H+HI2jGA+v7fQS1VjNtxZ4j/quE EwtP0/DbI1jLwfoBXOfKDnWVENDh06nlN8sVK9pz+jowbke11fysc4mUa7OA09rOCi/b C2ZA==
X-Gm-Message-State: APjAAAWad48X/JclvP9nWwD9rROqrPkmbuGyYz2H3YdOG5BmWqrPqgpq adclsaOQ6mhlBN569LEPRfEQroBf
X-Google-Smtp-Source: APXvYqw0kmMFsoH5/vmsuvwOs5Zoun4V5EtllqG1eLtD1dTgbJAl0JkKSSXAp95LB2aWv/2Rv9ljxw==
X-Received: by 2002:a02:ab90:: with SMTP id t16mr45644698jan.110.1568294281306; Thu, 12 Sep 2019 06:18:01 -0700 (PDT)
Received: from ([2603:1036:307:293b::5]) by with ESMTPSA id w22sm6917083ior.79.2019. (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Thu, 12 Sep 2019 06:18:00 -0700 (PDT)
From: Tarek Saad <>
To: Ebben Aries <>, "" <>
CC: "" <>, "" <>, "" <>
Subject: Re: Yangdoctors early review of draft-ietf-mpls-base-yang-10
Thread-Topic: Yangdoctors early review of draft-ietf-mpls-base-yang-10
Thread-Index: ATkwNDk0XwiEpmF5LrvZdM9A3YooW7hycIxb
X-MS-Exchange-MessageSentRepresentingType: 1
Date: Thu, 12 Sep 2019 13:17:59 +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: <>
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: IETF-Discussion <>
List-Unsubscribe: <>, <>
List-Archive: <>
List-Post: <>
List-Help: <>
List-Subscribe: <>, <>
X-List-Received-Date: Thu, 12 Sep 2019 13:18:08 -0000

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)