[i2rs] FW: [RTG-DIR] Routing directorate QA review of draft-ietf-i2rs-yang-l2-network-topology

"Zhangxian (Xian)" <zhang.xian@huawei.com> Wed, 18 May 2016 08:51 UTC

From: "Zhangxian (Xian)" <zhang.xian@huawei.com>
To: "draft-ietf-i2rs-yang-l2-network-topology@tools.ietf.org" <draft-ietf-i2rs-yang-l2-network-topology@tools.ietf.org>, "i2rs@ietf.org" <i2rs@ietf.org>
Thread-Topic: [RTG-DIR] Routing directorate QA review of draft-ietf-i2rs-yang-l2-network-topology
Thread-Index: AQHRsAq9RFmKWcS270KqpnV/dHGMVp++Y+cw
Date: Wed, 18 May 2016 08:50:54 +0000
Message-ID: <C636AF2FA540124E9B9ACB5A6BECCE6B7DEC8063@SZXEMA512-MBS.china.huawei.com>
Accept-Language: zh-CN, en-US
Content-Language: en-US
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: base64
MIME-Version: 1.0
Archived-At: <http://mailarchive.ietf.org/arch/msg/i2rs/WBZfGwkURSGAn8YS5n9yGKh7Y6Y>
Cc: Henning Rogge <hrogge@gmail.com>, Susan Hares <shares@ndzh.com>
Subject: [i2rs] FW: [RTG-DIR] Routing directorate QA review of draft-ietf-i2rs-yang-l2-network-topology
Precedence: list

To get the attention of draft authors and the WG to the following review/comments. 

Thank Henning for the constructive comments. 

Cheers,
Xian

-----Original Message-----
From: rtg-dir [mailto:rtg-dir-bounces@ietf.org] On Behalf Of Henning Rogge
Sent: 2016年5月17日 15:07
To: Zhangxian (Xian); jonathan.hardwick@metaswitch.com; jon.hudson@gmail.com; shares@ndzh.com >> Susan Hares
Cc: rtg-dir@ietf.org
Subject: Re: [RTG-DIR] Routing directorate QA review of draft-ietf-i2rs-yang-l2-network-topology

Hi,

I have been asked to provide a review to the following document to the 
routing directorate mailing list.

Please be aware that this is the first time I work with YANG and related 
drafts.


Document: draft-ietf-i2rs-yang-l2-network-topology-02

Reviewer: Henning Rogge
Review Date: Mai 16th, 2016


Intended Status: Standards Track


The data structure suggested by the draft is reasonable and would fit 
most Layer2 network technologies. I have a couple of points on the draft 
document which might be worth looking into:

* The introduction in 
https://tools.ietf.org/html/draft-ietf-i2rs-yang-l2-network-topology-02 
includes a link to "I-D.ietf-netmod-rfc6020bis" that links back to the 
draft document itself. Maybe some links in the document refer to an 
older name of the draft?

* the "termination-point" element only contains the types "ethernet" and 
"legacy" (which does not contain any data like mac-address). Is this 
reasonable or should a few data elements moved from the "ethernet" 
category to the "l2-termination-point-attributes" category?

* there are different types of VLAN tags be used... should there be 
another field ("vlan-type" ?) to announce 802.1ad QinQ usage? I think 
the 802.1ad tag is also sometimes also used to move VLAN over a switch 
that doesn't support it (unknown Ethertypes are usually just ignored), 
which means just knowing the VLAN-id is not enough to reach the endpoint.

* the type of ethernet (100, 1000, 10000) or data-rate could be an 
important attribute for an ethernet termination point, not only for links.


Henning Rogge
-- 
Diplom-Informatiker Henning Rogge , Fraunhofer-Institut für
Kommunikation, Informationsverarbeitung und Ergonomie FKIE
Kommunikationssysteme (KOM)
Fraunhofer Straße 20, 53343 Wachtberg, Germany
Telefon +49 228 9435-961,   Fax +49 228 9435 685
mailto:henning.rogge@fkie.fraunhofer.de http://www.fkie.fraunhofer.de

[i2rs] FW: [RTG-DIR] Routing directorate QA revie… Zhangxian (Xian)
Re: [i2rs] FW: [RTG-DIR] Routing directorate QA r… Susan Hares
Re: [i2rs] [RTG-DIR] Routing directorate QA revie… Dongjie (Jimmy)