Re: [i2rs] Ignas Bagdonas' Discuss on draft-ietf-i2rs-yang-dc-fabric-network-topology-08: (with DISCUSS and COMMENT)

"Susan Hares" <shares@ndzh.com> Tue, 03 April 2018 20:54 UTC

Return-Path: <shares@ndzh.com>
X-Original-To: i2rs@ietfa.amsl.com
Delivered-To: i2rs@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 069A812D873; Tue, 3 Apr 2018 13:54:34 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: 0.946
X-Spam-Level:
X-Spam-Status: No, score=0.946 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DOS_OUTLOOK_TO_MX=2.845, URIBL_BLOCKED=0.001] autolearn=no autolearn_force=no
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 nEsPkUkxL7aQ; Tue, 3 Apr 2018 13:54:32 -0700 (PDT)
Received: from hickoryhill-consulting.com (50-245-122-97-static.hfc.comcastbusiness.net [50.245.122.97]) (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 3D65712706D; Tue, 3 Apr 2018 13:54:32 -0700 (PDT)
X-Default-Received-SPF: pass (skip=loggedin (res=PASS)) x-ip-name=166.170.24.89;
From: Susan Hares <shares@ndzh.com>
To: 'Martin Bjorklund' <mbj@tail-f.com>
Cc: ibagdona@gmail.com, iesg@ietf.org, i2rs@ietf.org, draft-ietf-i2rs-yang-dc-fabric-network-topology@ietf.org, i2rs-chairs@ietf.org
References: <152275562986.13946.4129194230664503798.idtracker@ietfa.amsl.com> <006401d3cb53$f17c36d0$d474a470$@ndzh.com> <20180403.222933.530851196105443807.mbj@tail-f.com>
In-Reply-To: <20180403.222933.530851196105443807.mbj@tail-f.com>
Date: Tue, 03 Apr 2018 16:54:15 -0400
Message-ID: <018101d3cb8d$edef8140$c9ce83c0$@ndzh.com>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
X-Mailer: Microsoft Outlook 14.0
Thread-Index: AQHrfpTnkeJxdov8ELnNq0f+qOuaSgCz2sVuAZ2IZAWjrZNJ0A==
Content-Language: en-us
X-Authenticated-User: skh@ndzh.com
Archived-At: <https://mailarchive.ietf.org/arch/msg/i2rs/dwYBiJtMy2I04AVc5l9FJFK5scg>
Subject: Re: [i2rs] Ignas Bagdonas' Discuss on draft-ietf-i2rs-yang-dc-fabric-network-topology-08: (with DISCUSS and COMMENT)
X-BeenThere: i2rs@ietf.org
X-Mailman-Version: 2.1.22
Precedence: list
List-Id: "Interface to The Internet Routing System \(IRS\)" <i2rs.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/i2rs>, <mailto:i2rs-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/i2rs/>
List-Post: <mailto:i2rs@ietf.org>
List-Help: <mailto:i2rs-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/i2rs>, <mailto:i2rs-request@ietf.org?subject=subscribe>
X-List-Received-Date: Tue, 03 Apr 2018 20:54:34 -0000

Martin:

Thank you for the comments on the Yang doctors.  The discussion reference
was in the introductory material and not in the descriptions in the YANG
text.  Do you also want additional comments in the introductory section?

Sue Hares 

-----Original Message-----
From: Martin Bjorklund [mailto:mbj@tail-f.com] 
Sent: Tuesday, April 3, 2018 4:30 PM
To: shares@ndzh.com
Cc: ibagdona@gmail.com; iesg@ietf.org; i2rs@ietf.org;
draft-ietf-i2rs-yang-dc-fabric-network-topology@ietf.org;
i2rs-chairs@ietf.org
Subject: Re: [i2rs] Ignas Bagdonas' Discuss on
draft-ietf-i2rs-yang-dc-fabric-network-topology-08: (with DISCUSS and
COMMENT)

Hi,

Just a quick comment on the YANG doctor's review.

"Susan Hares" <shares@ndzh.com> wrote:
> Earlier feedback (rtg-dir, ops-dir, yang-doctors) on YANG suggested 
> taking out the lengthy descriptions regarding logic and history.

It is very common that the YANG doctor review ask for *more* details in the
descriptions.  In general, we want the module to have as much explanatory
text as possible.  So was the case for the YD review for this document as
well; the YD wrote "The descriptions in all YANG Modules are very
short/terse."  That was for the -02 version, and even the -00 version did
not contain lengthy descriptions AFAICT.


/martin