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

Andy Bierman <andy@yumaworks.com> Fri, 13 September 2019 05:02 UTC

Return-Path: <andy@yumaworks.com>
X-Original-To: mpls@ietfa.amsl.com
Delivered-To: mpls@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id EE472120088 for <mpls@ietfa.amsl.com>; Thu, 12 Sep 2019 22:02:52 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.898
X-Spam-Level:
X-Spam-Status: No, score=-1.898 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, HTML_MESSAGE=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: ietfa.amsl.com (amavisd-new); dkim=pass (2048-bit key) header.d=yumaworks-com.20150623.gappssmtp.com
Received: from mail.ietf.org ([4.31.198.44]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id 1mIsQKclhZvS for <mpls@ietfa.amsl.com>; Thu, 12 Sep 2019 22:02:50 -0700 (PDT)
Received: from mail-lj1-x22e.google.com (mail-lj1-x22e.google.com [IPv6:2a00:1450:4864:20::22e]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 70419120019 for <mpls@ietf.org>; Thu, 12 Sep 2019 22:02:50 -0700 (PDT)
Received: by mail-lj1-x22e.google.com with SMTP id r14so2716004ljn.3 for <mpls@ietf.org>; Thu, 12 Sep 2019 22:02:50 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=yumaworks-com.20150623.gappssmtp.com; s=20150623; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=HH9UfvziqSvK7rbCVrReFuT3C/PafEM1LEz/tUAV0Us=; b=w91p9LdQwScJVgS2YVSn+O7r7lFNW4R8dYcZFaUXW3vYeZtyM+h6csIuU3t1qI8r4Q 99dqVTChRiGwkhhlt0QD6cWTd36eE/IjIZCPfkrC23lIcNbSzlXw7H+sLgDwjPnzZ0f9 gjPBY/NCtCfKxTth3EEo7fM6CJEUUyefne/TSCjyIbIUezPG57POyBlt9z3I9ZEDxKoo R1w4AZ+4erh6/WWMEV2LqSQoIEesBt0X1r9UhHO+oX/tcfhe3jxzGVqvERxFCyky3NtV sMnpxa5mIDGBxrVcV/kiDOzwLkf2hEQ1PsxmQgrRHf/i/AUrZ+857gEPZfxn5iC8P/Of Xz5w==
X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=HH9UfvziqSvK7rbCVrReFuT3C/PafEM1LEz/tUAV0Us=; b=jicZ3hm04lFJmGhaEkqIGbF8YJNViYu/8dQK//IKS7wrAKEefIlogAtEkPJXNwbMEw 0TqG7BSvnk7rl5REbXx4QWzVZFIEj++rca6o/+BDTD73fPooLAETZB3D+INWO0Vcn8+f 4pOnL/0qcqi2wPE/fzOwo09Vx+2LwucnCuCLrpbMGU7p0xtGpH9rQKY1cpumg1m16OBt /benFuA2rMVR/atU+sgSwbuGo1wnVKB1fQ1G0S2OqrxSAyCllRi3E7g1JE1E7zrJaDGa GffdDdZWQvmJhj3R1T3aBtQAYsMR58laYamuzeG5YEgQWIStp+AaRlcOO4Nt9+sJEfff tr0g==
X-Gm-Message-State: APjAAAWPlRzWwQxYoGaYEm12DB8hVY0pOvYFZo2jIHpuRj0rsrRW+hOu flzvHRnDmqXDwJOQk7FQaOJ4+t18TkOGFxlPEHJzmw==
X-Google-Smtp-Source: APXvYqwbvp0RD7jmQVd9q/e70yqaQFxNUVzM772PbD7XuoZmwl1uhXn7m2ejLA3+dy/lEW6mcPyrtMZRui1fThMXwzA=
X-Received: by 2002:a2e:860c:: with SMTP id a12mr28430513lji.116.1568350968623; Thu, 12 Sep 2019 22:02:48 -0700 (PDT)
MIME-Version: 1.0
References: <156381448119.11357.2555532993158583807@ietfa.amsl.com> <BYAPR19MB341509F6F116B65811BF0DB1FCB00@BYAPR19MB3415.namprd19.prod.outlook.com>
In-Reply-To: <BYAPR19MB341509F6F116B65811BF0DB1FCB00@BYAPR19MB3415.namprd19.prod.outlook.com>
From: Andy Bierman <andy@yumaworks.com>
Date: Thu, 12 Sep 2019 22:02:37 -0700
Message-ID: <CABCOCHRHwOTEy0txGiCz=0k0O+X9pnDHqLVXov82GTwPaYNS-A@mail.gmail.com>
To: Tarek Saad <tsaad.net@gmail.com>
Cc: "yang-doctors@ietf.org" <yang-doctors@ietf.org>, "draft-ietf-mpls-static-yang.all@ietf.org" <draft-ietf-mpls-static-yang.all@ietf.org>, "mpls@ietf.org" <mpls@ietf.org>
Content-Type: multipart/alternative; boundary="0000000000005e48c90592682a54"
Archived-At: <https://mailarchive.ietf.org/arch/msg/mpls/So7siNpLY3-3ho4qxEXb1k_0oQE>
Subject: Re: [mpls] Yangdoctors early review of draft-ietf-mpls-static-yang-09
X-BeenThere: mpls@ietf.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: Multi-Protocol Label Switching WG <mpls.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/mpls>, <mailto:mpls-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/mpls/>
List-Post: <mailto:mpls@ietf.org>
List-Help: <mailto:mpls-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/mpls>, <mailto:mpls-request@ietf.org?subject=subscribe>
X-List-Received-Date: Fri, 13 Sep 2019 05:02:53 -0000

Hi,

OK thanks,

Andy


On Thu, Sep 12, 2019 at 8:27 AM Tarek Saad <tsaad.net@gmail.com> wrote:

> Hi Andy,
>
> Thanks for your review and comments. We have uploaded a new revision that
> addresses your comments.
> https://tools.ietf.org/html/draft-ietf-mpls-static-yang-10
>
> Inline for more details.
>
> On 7/22/19, 12:54 PM, "Andy Bierman via Datatracker" <noreply@ietf.org>
> wrote:
>
>     Reviewer: Andy Bierman
>     Review result: Ready with Nits
>
>     - No pyang warnings or errors
>     - No noticeable typos or omissions
>
>     Nits:
>      - contact info includes WG Chairs; this was removed in the new usage
> guidelines
> [TS]: removed.
>
>     Questions:
>      - 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.?
>
> Regards,
> Tarek
>
>
>