[RTG-DIR] RtgDir review: draft-ietf-ccamp-gmpls-ted-mib-14.txt
Leeyoung <leeyoung@huawei.com> Thu, 18 October 2012 17:21 UTC
Return-Path: <leeyoung@huawei.com>
X-Original-To: rtg-dir@ietfa.amsl.com
Delivered-To: rtg-dir@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 8E51321F877A; Thu, 18 Oct 2012 10:21:58 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -6.599
X-Spam-Level:
X-Spam-Status: No, score=-6.599 tagged_above=-999 required=5 tests=[AWL=0.000, BAYES_00=-2.599, RCVD_IN_DNSWL_MED=-4]
Received: from mail.ietf.org ([64.170.98.30]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id F8UWz6s4CCoK; Thu, 18 Oct 2012 10:21:57 -0700 (PDT)
Received: from lhrrgout.huawei.com (lhrrgout.huawei.com [194.213.3.17]) by ietfa.amsl.com (Postfix) with ESMTP id 2898621F876A; Thu, 18 Oct 2012 10:21:53 -0700 (PDT)
Received: from 172.18.7.190 (EHLO lhreml203-edg.china.huawei.com) ([172.18.7.190]) by lhrrg02-dlp.huawei.com (MOS 4.3.5-GA FastPath queued) with ESMTP id AKS69709; Thu, 18 Oct 2012 17:21:53 +0000 (GMT)
Received: from LHREML403-HUB.china.huawei.com (10.201.5.217) by lhreml203-edg.huawei.com (172.18.7.221) with Microsoft SMTP Server (TLS) id 14.1.323.3; Thu, 18 Oct 2012 18:20:59 +0100
Received: from DFWEML407-HUB.china.huawei.com (10.193.5.132) by lhreml403-hub.china.huawei.com (10.201.5.217) with Microsoft SMTP Server (TLS) id 14.1.323.3; Thu, 18 Oct 2012 18:21:52 +0100
Received: from dfweml511-mbs.china.huawei.com ([169.254.15.28]) by dfweml407-hub.china.huawei.com ([10.193.5.132]) with mapi id 14.01.0323.003; Thu, 18 Oct 2012 10:21:49 -0700
From: Leeyoung <leeyoung@huawei.com>
To: "rtg-ads@tools.ietf.org" <rtg-ads@tools.ietf.org>
Thread-Topic: RtgDir review: draft-ietf-ccamp-gmpls-ted-mib-14.txt
Thread-Index: AQHNrVUOsw0jlNebbkWWRrXkkxw8+g==
Date: Thu, 18 Oct 2012 17:21:49 +0000
Message-ID: <7AEB3D6833318045B4AE71C2C87E8E1729089755@dfweml511-mbs.china.huawei.com>
References: <507F1CC1.7070301@riw.us>
In-Reply-To: <507F1CC1.7070301@riw.us>
Accept-Language: en-US, zh-CN
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
x-originating-ip: [10.192.11.137]
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: base64
MIME-Version: 1.0
X-CFilter-Loop: Reflected
Cc: "rtg-dir@ietf.org" <rtg-dir@ietf.org>, CCAMP <ccamp@ietf.org>, "draft-ietf-ccamp-gmpls-ted-mib.all@tools.ietf.org" <draft-ietf-ccamp-gmpls-ted-mib.all@tools.ietf.org>
Subject: [RTG-DIR] RtgDir review: draft-ietf-ccamp-gmpls-ted-mib-14.txt
X-BeenThere: rtg-dir@ietf.org
X-Mailman-Version: 2.1.12
Precedence: list
List-Id: Routing Area Directorate <rtg-dir.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/rtg-dir>, <mailto:rtg-dir-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/rtg-dir>
List-Post: <mailto:rtg-dir@ietf.org>
List-Help: <mailto:rtg-dir-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/rtg-dir>, <mailto:rtg-dir-request@ietf.org?subject=subscribe>
X-List-Received-Date: Thu, 18 Oct 2012 17:22:02 -0000
Hello, I have been selected as the Routing Directorate reviewer for this draft. The Routing Directorate seeks to review all routing or routing-related drafts as they pass through IETF last call and IESG review. The purpose of the review is to provide assistance to the Routing ADs. For more information about the Routing Directorate, please see http://www.ietf.org/iesg/directorate/routing.html Although these comments are primarily for the use of the Routing ADs, it would be helpful if you could consider them along with any other IETF Last Call comments that you receive, and strive to resolve them through discussion or by updating the draft. Document: draft-ietf-ccamp-gmpls-ted-mib-14.txt Reviewer: Young Lee Review Date: 17 October 2012 IETF LC End Date: 18 October 2012 Intended Status: Standard track Summary: I have no major concerns about this document that I think should be resolved before publication. Comments: This document is clearly written, but hard to read due to tight indentations and no spacing between major sections and paragraphs throughout the whole document. Major Issues: No major issues found. Minor Issues: Abstract Indentations need to be fixed Section 2 Introduction S/ On the other side, MPLS/GMPLS based traffic engineering has so far extended OSPF/ISIS routing protocol with TE functionality [RFC4202], [RFC3630], [RFC5329], [RFC5307], [RFC5305]. / On the other side, MPLS/GMPLS based traffic engineering has so far extended OSPF/ISIS routing protocol with TE functionality per [RFC4202], [RFC3630], [RFC5329], [RFC5307] and [RFC5305]. Section 2 S/ To manage such MPLS-TE/GMPLS networks effectively, routing information associated with MPLS/GMPLS TE parameters (TED) is preferred for the network management, however, there is no clear definition of MPLS/GMPLS TE information in existing MIBs related to OSPF(v2 and v3)/ISIS. / To manage such MPLS-TE/GMPLS networks effectively, routing information associated with MPLS/GMPLS TE parameters (TED) is preferred for the network management; however, there is no clear definition of MPLS/GMPLS TE information in existing MIBs related to OSPF(v2 and v3)/ISIS. Section 3.3 – Please use consistent cases for all acronyms. GMPLS: Generalized Multi-Protocol Label Switching ISIS: Intermediate System to Intermediate System LSA: Link state advertisement Link State Advertisement LSP: Label Switching Path LSR: Label Switching Router MIB: Management Information Base OSPF: Open Shortest Path First PSC: Packet switch capable -> Packet Switch Capable SRLG: Shared risk link group -> Shared Risk Link Group TE: Traffic Engineering TED: Traffic Engineering Database TDM: Time Division Multiplexing Section 5.3 – Style of consistency with Section 5.2 S / Also, this is utilized independently because one or more local interface IP address sub TLVs may exist in the same Link-TLV. / This is independently defined, because the Interface IP Address sub-TLV may appear more than once within the same Link-TLV. Section 5.4 and 5.5 --- same comment as above Section 7 - Not clear what these MIB Definitions are all about. If these need to be supported, there should statements that these are additional MIBS to be added to existing MIB, etc. Style of writing in the following sentence S / These are the tables and objects and their sensitivity/vulnerability: tedTable, tedLocalIfAddrTable, tedRemoteIfAddrTable, tedSwCapTable and tedSrlgTable contain topology information for the MPLS/GMPLS network. / The list of tables and objects that may be vulnerable or sensitive: tedTable, tedLocalIfAddrTable, tedRemoteIfAddrTable, tedSwCapTable and tedSrlgTable. They contain topology information for the MPLS/GMPLS network. Section 9.1 --- Not clear what is being asked to the IANA. Would it be more specific than “transmission XXX”? Need come clarification on this. The IANA is requested to assign {transmission XXX } to the TED-MIB module specified in this document. Nits: Please correct the following issues generated by the Nits tool: == The document seems to lack a disclaimer for pre-RFC5378 work, but was first submitted before 10 November 2008. Should you add the disclaimer? (See the Legal Provisions document at http://trustee.ietf.org/license-info for more information.). -- The document date (October 4, 2012) is 14 days in the past. Is this intentional? Checking references for intended status: Proposed Standard ---------------------------------------------------------------------------- (See RFCs 3967 and 4897 for information about using normative references to lower-maturity documents in RFCs) == Unused Reference: 'RFC2328' is defined on line 1552, but no explicit reference was found in the text == Unused Reference: 'RFC4001' is defined on line 1566, but no explicit reference was found in the text == Unused Reference: 'RFC4801' is defined on line 1579, but no explicit reference was found in the text == Unused Reference: 'RFC5340' is defined on line 1587, but no explicit reference was found in the text == Unused Reference: 'RFC6340' is defined on line 1593, but no explicit reference was found in the text == Unused Reference: 'ISO10589' is defined on line 1596, but no explicit reference was found in the text == Unused Reference: 'RFC4220' is defined on line 1625, but no explicit reference was found in the text -- Possible downref: Non-RFC (?) normative reference: ref. 'ISO10589' Young
- [RTG-DIR] RtgDir review: draft-ietf-karp-ospf-ana… Russ White
- [RTG-DIR] RtgDir review: draft-ietf-ccamp-gmpls-t… Leeyoung
- Re: [RTG-DIR] RtgDir review: draft-ietf-ccamp-gmp… Masanori Miyazawa
- Re: [RTG-DIR] RtgDir review: draft-ietf-ccamp-gmp… Leeyoung
- Re: [RTG-DIR] RtgDir review: draft-ietf-ccamp-gmp… Masanori Miyazawa