[Pce] 'MIB Doctor' review of draft-ietf-pce-pcep-mib-06
Juergen Schoenwaelder <j.schoenwaelder@jacobs-university.de> Sun, 19 January 2014 18:22 UTC
Return-Path: <j.schoenwaelder@jacobs-university.de>
X-Original-To: pce@ietfa.amsl.com
Delivered-To: pce@ietfa.amsl.com
Received: from localhost (ietfa.amsl.com [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id C7BA11ADF70; Sun, 19 Jan 2014 10:22:51 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.788
X-Spam-Level:
X-Spam-Status: No, score=-2.788 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, HELO_EQ_DE=0.35, RCVD_IN_DNSWL_LOW=-0.7, RP_MATCHES_RCVD=-0.538] autolearn=ham
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 gmW-wcM-B5XY; Sun, 19 Jan 2014 10:22:49 -0800 (PST)
Received: from hermes.jacobs-university.de (hermes.jacobs-university.de [212.201.44.23]) by ietfa.amsl.com (Postfix) with ESMTP id 713A81ADF38; Sun, 19 Jan 2014 10:22:48 -0800 (PST)
Received: from localhost (demetrius2.jacobs-university.de [212.201.44.47]) by hermes.jacobs-university.de (Postfix) with ESMTP id 0264520056; Sun, 19 Jan 2014 19:22:35 +0100 (CET)
X-Virus-Scanned: amavisd-new at jacobs-university.de
Received: from hermes.jacobs-university.de ([212.201.44.23]) by localhost (demetrius2.jacobs-university.de [212.201.44.32]) (amavisd-new, port 10024) with ESMTP id JE1m7bvH9XB2; Sun, 19 Jan 2014 19:22:34 +0100 (CET)
Received: from elstar.local (elstar.jacobs.jacobs-university.de [10.50.231.133]) by hermes.jacobs-university.de (Postfix) with ESMTP id 95ACA2003F; Sun, 19 Jan 2014 19:22:34 +0100 (CET)
Received: by elstar.local (Postfix, from userid 501) id 77FDB2ABDB8C; Sun, 19 Jan 2014 19:22:31 +0100 (CET)
Date: Sun, 19 Jan 2014 19:22:31 +0100
From: Juergen Schoenwaelder <j.schoenwaelder@jacobs-university.de>
To: draft-ietf-pce-pcep-mib.all@tools.ietf.org, pce@ietf.org
Message-ID: <20140119182231.GB42438@elstar.local>
Mail-Followup-To: draft-ietf-pce-pcep-mib.all@tools.ietf.org, pce@ietf.org, mib-doctors@ietf.org
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Disposition: inline
User-Agent: Mutt/1.5.21 (2010-09-15)
X-Mailman-Approved-At: Mon, 20 Jan 2014 01:43:44 -0800
Cc: mib-doctors@ietf.org
Subject: [Pce] 'MIB Doctor' review of draft-ietf-pce-pcep-mib-06
X-BeenThere: pce@ietf.org
X-Mailman-Version: 2.1.15
Precedence: list
Reply-To: Juergen Schoenwaelder <j.schoenwaelder@jacobs-university.de>
List-Id: Path Computation Element <pce.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/pce>, <mailto:pce-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/pce/>
List-Post: <mailto:pce@ietf.org>
List-Help: <mailto:pce-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/pce>, <mailto:pce-request@ietf.org?subject=subscribe>
X-List-Received-Date: Sun, 19 Jan 2014 18:22:52 -0000
Hi, please find below my review of draft-ietf-pce-pcep-mib-06. Overall, the document is in a pretty good shape. It is nice to review MIB modules that are well put together. The MIB module compiles fine with smiling 0.4.8 and produces no warnings. Below is a list of comments in document order of appearance. Some may require an action other maybe not. I have marked the ones that I find more important with (*). a) I guess you should spell out PCE In the title. b) The abstract says the I-D defines an "experimental portion" of the MIB while the document is marked as standards-track. This does not seem to be consistent. c) In section 2, ATD should STD. d) The LAST-UPDATED and REVISION clauses as well as the Copyright year in the DESCRIPTION clause of the MODULE-IDENTITY all date back to 2013 even though the I-D was last updated in 2014. You may want to move to a date in 2014. e) I suggest to replace pcePcepMIBObjects with pcePcepObjects. This aligns more with the recommendation given in appendix D of RFC 4181. f) I think the description of pcePcepEntityTable should use plural, that is, "... about PCEP Entities". Or will this table always have only a single row? g) If you want to describe the enums of an INTEGER, please do so in the DESCRIPTION clause and not in comments (since comments may not be preserved by tools processing MIB modules). h) pcePcepEntityAddr seems to serve two purposes, depending on whether the peer is a pce and a pcc. The pcePcepPeerRole object makes me believe an entity can also be both at the same time. If so, then how does one interpret pcePcepEntityAddr? (*) i) pcePcepEntityConnectMaxRetry has a UNITS clause saying it is in seconds. This might be a cut and paste error? j) pcePcepPeerAddr has a length restriction which essentially also restricts pcePcepPeerAddrType. pcePcepPeerAddrType kind of has a restriction inline but not explicit via subtyping. It might be more inline with the INET-ADDRESS-MIB to not restrict this. If you want to restrict things, you may want to do that in a compliance statement. (*) Despite future proofing considerations, the restrictions make it impossible to have PCE entities with non-global that is non-unique addresses (because you may not be able to disambiguate them). k) pcePcepPeerDiscontinuityTime refers to RFC 1907. I think the current version it should refer to is RFC 3418. In any case, if you think explicit text concerning sysUpTime is needed, I think this warrants a normative reference added to RFC 3418 and a statement in section 5.5. (I am not sure this is really needed since you kind of import knowledge about sysUpTime via the TimeStamp TC you are using. j) The *AvgRspTime, *LWMRspTime, *HWMRspTime objects have a resolution of seconds. This means that times < 500ms will be represented as 0 seconds and thus indistinguishable from no message at all. I do not know whether this is an issue or not. Is it possible to have sub-second response times? (*) k) pcePcepSessState typo: entiries -> entries l) pcePcepSessPeerDeadTimer typo: for for -> for -- Juergen Schoenwaelder Jacobs University Bremen gGmbH Phone: +49 421 200 3587 Campus Ring 1, 28759 Bremen, Germany Fax: +49 421 200 3103 <http://www.jacobs-university.de/>
- Re: [Pce] 'MIB Doctor' review of draft-ietf-pce-p… Daniel King
- [Pce] 'MIB Doctor' review of draft-ietf-pce-pcep-… Juergen Schoenwaelder