Re: [i2rs] WG LC confirmation on draft-ietf-i2rs-yang-l2-network-topology (3/7 to 3/21/2018)

Robert Wilton <rwilton@cisco.com> Wed, 07 March 2018 17:53 UTC

Return-Path: <rwilton@cisco.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 0C901120725; Wed, 7 Mar 2018 09:53:24 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -14.509
X-Spam-Level:
X-Spam-Status: No, score=-14.509 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, HTML_MESSAGE=0.001, RCVD_IN_DNSWL_HI=-5, SPF_PASS=-0.001, T_RP_MATCHES_RCVD=-0.01, URIBL_BLOCKED=0.001, USER_IN_DEF_DKIM_WL=-7.5] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (1024-bit key) header.d=cisco.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 EtOQ-L3HE2sv; Wed, 7 Mar 2018 09:53:22 -0800 (PST)
Received: from aer-iport-4.cisco.com (aer-iport-4.cisco.com [173.38.203.54]) (using TLSv1.2 with cipher DHE-RSA-SEED-SHA (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 38D07120721; Wed, 7 Mar 2018 09:53:21 -0800 (PST)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=cisco.com; i=@cisco.com; l=8073; q=dns/txt; s=iport; t=1520445201; x=1521654801; h=subject:to:references:from:message-id:date:mime-version: in-reply-to; bh=ectGiU/CFDSTn02juSM+42BgEN70RYinHkRTGNNnPi8=; b=Upm9zMI/I4PyIe+0MJzf71L6LsiWsw504R2S6iNeVRYxiQwzgyIHD2Q1 oS7qM3JbOd0miEyP8eDXAQNot7/gLMVxDJLTwR4Xyv7gEKFa0X1xlr4CU vva0kDYmHp+yzVCcX4zm0IPVgR8xraZ+EgYzBT2t3N67BtVoobjrZcV6d c=;
X-IronPort-Anti-Spam-Filtered: true
X-IronPort-Anti-Spam-Result: A0B6AQDvJaBa/xbLJq1dGQEBAQEBAQEBAQEBAQcBAQEBAYJaSYETcCiObI8JgRaPEYU3ggEKGAEKhQ0Cgys4FAECAQEBAQEBAmsnhSQBAQQBAStBGwsOCi4nMAYBDAYCAQGFFxCrQiaETIN0gX8dBYUxhAWCD4MEgy4BAYFZhgUEmmcJkHgHgWeEN4J4hWSBLYodhg2BLjUhgVIzGggbFTqCQ4JjbgEIbkA3jAkBAQE
X-IronPort-AV: E=Sophos;i="5.47,436,1515456000"; d="scan'208,217";a="2436486"
Received: from aer-iport-nat.cisco.com (HELO aer-core-3.cisco.com) ([173.38.203.22]) by aer-iport-4.cisco.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 07 Mar 2018 17:53:19 +0000
Received: from [10.63.23.110] (dhcp-ensft1-uk-vla370-10-63-23-110.cisco.com [10.63.23.110]) by aer-core-3.cisco.com (8.14.5/8.14.5) with ESMTP id w27HrIAj023448; Wed, 7 Mar 2018 17:53:19 GMT
To: Susan Hares <shares@ndzh.com>, i2rs@ietf.org, draft-ietf-i2rs-yang-l2-network-topology@ietf.org
References: <017e01d3b639$50098a70$f01c9f50$@ndzh.com>
From: Robert Wilton <rwilton@cisco.com>
Message-ID: <69d117ad-ca45-0ab4-93b9-ccc6bae2b2b4@cisco.com>
Date: Wed, 07 Mar 2018 17:53:18 +0000
User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:52.0) Gecko/20100101 Thunderbird/52.6.0
MIME-Version: 1.0
In-Reply-To: <017e01d3b639$50098a70$f01c9f50$@ndzh.com>
Content-Type: multipart/alternative; boundary="------------F0AECD928C1CA2775CAAD4A6"
Content-Language: en-US
Archived-At: <https://mailarchive.ietf.org/arch/msg/i2rs/3YPNfoBoXglfu-Jk_VEp2-_1xFM>
Subject: Re: [i2rs] WG LC confirmation on draft-ietf-i2rs-yang-l2-network-topology (3/7 to 3/21/2018)
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: Wed, 07 Mar 2018 17:53:24 -0000

One quick comment (not a proper review):

I note that the extra state module in Appendix A is redefining features 
defined in the main module.  This isn't generically necessarily the 
wrong thing to do, but in this case it looks like they are not required 
in the state module (since they don't appear to be used), and would be 
best removed.

Thanks,
Rob


On 07/03/2018 17:25, Susan Hares wrote:
>
> Greetings I2RS folks:
>
> This is a 2 Week WG LC to confirm the WG LC on 
> draft-ietf-i2rs-yang-l2-network-topology.   During this WG LC, please 
> indicate any issues with the updates to align with Network Management 
> Data Store Module.  These may include:
>
> 1)Yang data model structure,
>
> 2)High level Yang language format
>
> 3)Security consideration section,
>
> 4)Any issues with the notification calls.
>
> If you feel this updated data model is ready for publication, please 
> send indicate that in your response.
>
> Cheerily, Susan Hares
>
>
>
> _______________________________________________
> i2rs mailing list
> i2rs@ietf.org
> https://www.ietf.org/mailman/listinfo/i2rs