Re: [yang-doctors] Yangdoctors early review of draft-ietf-mpls-static-yang-09

Tarek Saad <> Thu, 12 September 2019 15:27 UTC

Return-Path: <>
Received: from localhost (localhost []) by (Postfix) with ESMTP id BC4EB12010D; Thu, 12 Sep 2019 08:27:42 -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 1iFwdA6-38x2; Thu, 12 Sep 2019 08:27:41 -0700 (PDT)
Received: from ( [IPv6:2607:f8b0:4864:20::d35]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by (Postfix) with ESMTPS id 54006120111; Thu, 12 Sep 2019 08:27:38 -0700 (PDT)
Received: by with SMTP id f4so54967153ion.2; Thu, 12 Sep 2019 08:27:38 -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=VwmYvBuBcSoHVSbp3gI7DLKpLK1Jsfk76IOi6La5wVU=; b=eT5Bh7jX19llVK5wZk1EFioWYNt9/WNgRFTOYG9zx+zLlfEndFWRBGvQssocrXVY2+ r6HP0H7awCiGgTyxu8XngQj9YoJ/HTSY83Xbrpbwvi5fSF+pUCXSoMGiKi2zDKwX9Io6 IE2cDlPmxjOwjKthbgu2LeMHPy9dQPXMCHKNPy87lzGv4bUS5KfWxyQSeCzKntY+4vod HvtLa+EvbfbQBT9P7xeatezI3KZfY2EpBXQMZJBG8w4jx95Oiasr1l/VqMG7vRwKJTJO 3UBcuA5qTsIy3xsr+Duc7xaD2XAKcpeAylqcqg/R1y+mY9XtDw/G6ytAcxO51+lpI6ef UNOw==
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=VwmYvBuBcSoHVSbp3gI7DLKpLK1Jsfk76IOi6La5wVU=; b=qxPHLiMlLVJxaqxbvUkh3bEbe73bKx3351cGzHyL7ycHPm2Dl5QxfGVGFdBwOmpwvg Jx89ssKs9skREikQC3IIdaQCLicVwqSdcNRuVsumpMMi3glfkTf6hd6yKASguOb76yg3 s1XNr4JCxF4VvHdaHZv3tzd5WDQEqx8Uuy82Pm5JGy0EL+/zNtvDeoCHSeYocbH6xHkR sBAQyz30qgARrSc9ozcv9DfRSOWJHjYbarq9qrVQkF9QrdsWD8hlFpnoKiX6v1tOg2F3 Rc4jM3lkI2De6rHrZ/ZY9ZCF8U56fqxZrfXnG6sOgcU4UfB6vCP3Me3DxioZ70834b+T N7eA==
X-Gm-Message-State: APjAAAV7Abo59HlCFZxGMeFrYwfVmE7IUqvjg03CfCVLH8+qU6Clld5u dYoHu4NCXKzFeslA/fK5ZoA+kbtn
X-Google-Smtp-Source: APXvYqxDX9tG3JPkAGPGsskuq83rTD2XW3miypAXK6vsvk7QcTW1tyC26r8npTUT09AtVJN+9uPsLA==
X-Received: by 2002:a02:3e8a:: with SMTP id s132mr36866165jas.66.1568302057498; Thu, 12 Sep 2019 08:27:37 -0700 (PDT)
Received: from ([2603:1036:307:293b::5]) by with ESMTPSA id m67sm36531617iof.21.2019. (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Thu, 12 Sep 2019 08:27:37 -0700 (PDT)
From: Tarek Saad <>
To: Andy Bierman <>, "" <>
CC: "" <>, "" <>
Thread-Topic: Yangdoctors early review of draft-ietf-mpls-static-yang-09
Thread-Index: ATc1NTUxGHi6xqa0qYEQyyZ6K5i2C7xoksqT
X-MS-Exchange-MessageSentRepresentingType: 1
Date: Thu, 12 Sep 2019 15:27:36 +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-static-yang-09
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: Thu, 12 Sep 2019 15:27:43 -0000

Hi Andy,

Thanks for your review and comments. We have uploaded a new revision that addresses your comments.

Inline for more details.

´╗┐On 7/22/19, 12:54 PM, "Andy Bierman via Datatracker" <>; wrote:

    Reviewer: Andy Bierman
    Review result: Ready with Nits
    - No pyang warnings or errors
    - No noticeable typos or omissions
     - contact info includes WG Chairs; this was removed in the new usage guidelines
[TS]: removed.

     - The ietf-mpls-static-extended module is used instead of a YANG feature.
       Is the generic name 'extended' appropriate? What will the next module used to
       avoid a YANG feature be called? extended2?
[TS]: there are multiple models (at IETF) that have adopted similar approach of grouping non-base data in a separate module named as 'xxx-extended'. One argument was to enforce the base model be supported without any feature check - and hence makes it simpler to understand. Not sure I have an answer for names of future extended modules, but maybe extended-<feature1>, extended-<feature2>, etc.?