Link Management Protocol (LMP) Management Information Base (MIB)
draft-ietf-ccamp-lmp-mib-10
Revision differences
Document history
Date | Rev. | By | Action |
---|---|---|---|
2004-11-05
|
10 | Amy Vezza | State Changes to RFC Ed Queue from Approved-announcement sent by Amy Vezza |
2004-11-04
|
10 | Amy Vezza | IESG state changed to Approved-announcement sent |
2004-11-04
|
10 | Amy Vezza | IESG has approved the document |
2004-11-04
|
10 | Amy Vezza | Closed "Approve" ballot |
2004-11-04
|
10 | Bert Wijnen | State Changes to Approved-announcement to be sent from Approved-announcement to be sent::Point Raised - writeup needed by Bert Wijnen |
2004-11-04
|
10 | Bert Wijnen | Status date has been changed to 2004-11-04 from 2004-11-02 |
2004-11-02
|
10 | Bert Wijnen | Added an RFC-Editor note. Checking with other ADs if it is OK (which I assume)> |
2004-11-02
|
10 | Bert Wijnen | Status date has been changed to 2004-11-02 from 2004-10-20 |
2004-10-29
|
10 | (System) | Removed from agenda for telechat - 2004-10-28 |
2004-10-28
|
10 | Amy Vezza | State Changes to Approved-announcement to be sent::Point Raised - writeup needed from IESG Evaluation by Amy Vezza |
2004-10-28
|
10 | Thomas Narten | [Ballot Position Update] New position, No Objection, has been recorded for Thomas Narten by Thomas Narten |
2004-10-28
|
10 | Bill Fenner | [Ballot Position Update] New position, No Objection, has been recorded for Bill Fenner by Bill Fenner |
2004-10-28
|
10 | Allison Mankin | [Ballot comment] Informative question: does this use the common snmp approach to handling v4/v6 addresses in mibs? The 0/4/16 address length object stands out. |
2004-10-28
|
10 | Allison Mankin | [Ballot Position Update] New position, No Objection, has been recorded for Allison Mankin by Allison Mankin |
2004-10-28
|
10 | Alex Zinin | [Ballot Position Update] New position, No Objection, has been recorded for Alex Zinin by Alex Zinin |
2004-10-28
|
10 | Harald Alvestrand | [Ballot comment] Reviewed by Mary Barnes, Gen-ART Her review: Summary: --------- Draft is ready to publish as Proposed Standard with the correction of the following … [Ballot comment] Reviewed by Mary Barnes, Gen-ART Her review: Summary: --------- Draft is ready to publish as Proposed Standard with the correction of the following editorial nits ( based on the assumption that it's been reviewed/approved by MIB doctors and run through the MIB compilation tools). Nits: ----- - Needs updating to new template reflecting RFC 3668/3667. - Blank lines between authors in the document heading is not at all necessary. - Section 2: Introduction: The 2nd sentence is quite awkward and I would propose rewriting (at a minimum the word "which" should be changed to "whose") from: Along with Generalized MPLS (GMPLS) [RFC3471], the Link Management Protocol [LMP], which primary purpose is to manage traffic engineering (TE) links, is one of the key components of this stan dardization activity." to something like: Generalized MPLS (GMPLS) [RFC3471] and the Link Management Protocol [LMP] are key components of this standardization activity. The primary purpose of LMP is to manage traffic engineering (TE) links." - Section 14.1 Normative References: Remove reference to RFC 2026 per updates to template (curiously, RFC 3668 and 3667 are already listed as references - do they really need to be as I've not seen this done in other drafts?) |
2004-10-28
|
10 | Harald Alvestrand | [Ballot Position Update] New position, No Objection, has been recorded for Harald Alvestrand by Harald Alvestrand |
2004-10-28
|
10 | Jon Peterson | [Ballot Position Update] New position, No Objection, has been recorded for Jon Peterson by Jon Peterson |
2004-10-27
|
10 | Margaret Cullen | [Ballot Position Update] New position, No Objection, has been recorded for Margaret Wasserman by Margaret Wasserman |
2004-10-27
|
10 | David Kessens | [Ballot Position Update] New position, No Objection, has been recorded for David Kessens by David Kessens |
2004-10-25
|
10 | Ted Hardie | [Ballot Position Update] New position, No Objection, has been recorded for Ted Hardie by Ted Hardie |
2004-10-25
|
10 | Steven Bellovin | [Ballot Position Update] New position, No Objection, has been recorded for Steve Bellovin by Steve Bellovin |
2004-10-23
|
10 | Russ Housley | [Ballot Position Update] New position, No Objection, has been recorded for Russ Housley by Russ Housley |
2004-10-22
|
10 | Scott Hollenbeck | [Ballot Position Update] New position, No Objection, has been recorded for Scott Hollenbeck by Scott Hollenbeck |
2004-10-20
|
10 | Bert Wijnen | Placed on agenda for telechat - 2004-10-28 by Bert Wijnen |
2004-10-20
|
10 | Bert Wijnen | State Changes to IESG Evaluation from Waiting for Writeup by Bert Wijnen |
2004-10-20
|
10 | Bert Wijnen | Status date has been changed to 2004-10-20 from 2004-09-21 |
2004-10-20
|
10 | Bert Wijnen | [Ballot Position Update] New position, Yes, has been recorded for Bert Wijnen |
2004-10-20
|
10 | Bert Wijnen | Ballot has been issued by Bert Wijnen |
2004-10-20
|
10 | Bert Wijnen | Created "Approve" ballot |
2004-10-05
|
10 | (System) | State has been changed to Waiting for Writeup from In Last Call by system |
2004-10-04
|
10 | Michelle Cotton | IANA Last Call Comments: Upon approval of this document the IANA will register a new IANA ifType for lmp and a transmission number for LMP-MIB. … IANA Last Call Comments: Upon approval of this document the IANA will register a new IANA ifType for lmp and a transmission number for LMP-MIB. We understand the assignment of the same number for the transmission and ifType assignments is requested. |
2004-09-21
|
10 | Amy Vezza | Last call sent |
2004-09-21
|
10 | Amy Vezza | State Changes to In Last Call from Last Call Requested by Amy Vezza |
2004-09-21
|
10 | Bert Wijnen | Status date has been changed to 2004-09-21 from 2004-08-12 |
2004-09-21
|
10 | Bert Wijnen | State Changes to Last Call Requested from AD Evaluation::AD Followup by Bert Wijnen |
2004-09-21
|
10 | Bert Wijnen | Last Call was requested by Bert Wijnen |
2004-09-21
|
10 | (System) | Ballot writeup text was added |
2004-09-21
|
10 | (System) | Last call text was added |
2004-09-21
|
10 | (System) | Ballot approval text was added |
2004-09-09
|
10 | (System) | Sub state has been changed to AD Follow up from New Id Needed |
2004-09-09
|
10 | (System) | New version available: draft-ietf-ccamp-lmp-mib-10.txt |
2004-08-12
|
10 | Bert Wijnen | State Changes to AD Evaluation::Revised ID Needed from AD Evaluation::AD Followup by Bert Wijnen |
2004-08-12
|
10 | Bert Wijnen | Adrian (in an email of 21 July) suggests a new revision right after San Diego. So waiting for that to come out. |
2004-08-12
|
10 | Bert Wijnen | Status date has been changed to 2004-08-12 from 2004-07-19 |
2004-07-19
|
10 | Bert Wijnen | Checking with ALex if he or I should shepherd this doc thorugh IETF Last Call and IESG review. |
2004-07-19
|
10 | Bert Wijnen | Last AD review (of revision 9) -----Original Message----- From: Wijnen, Bert (Bert) Sent: maandag 19 juli 2004 23:00 To: Martin Dubuc; 'Thomas D. Nadeau'; Jonathan … Last AD review (of revision 9) -----Original Message----- From: Wijnen, Bert (Bert) Sent: maandag 19 juli 2004 23:00 To: Martin Dubuc; 'Thomas D. Nadeau'; Jonathan Lang; Sudheer Dharanikota; Evan McGinnis Cc: Adrian Farrel; Kireeti Kompella (E-mail); Alex Zinin (E-mail) Subject: RE: draft-ietf-ccamp-lmp-mib-09.txt OK, I am going through this doc again. I don't think we need to consider any of the below as blocking. I'd suggest we go ahead with an IETF Last Call and consider the below as the first set of IETF Last Call comments. Alex. this doc is in my name in ID-tracker. If you want I can handle it and do the shepherding. Although it is a CCAMP WG document and so the logical Ad to shepherd this would be you. Possibly better from the IESG review process anyway. Let me know if you want to take over from here. 1. Seems that some changes have caused inconsistencies 6.5. lmpLinkVerificationTable The link verification table is used for configuring the LMP link verification parameters of TE links. This table is an AUGMENT to the lmpTeLinkTable. It is now a "sparse augements", since you are using ifIndex (as indeed is correct). But using the capital AUGMENT here will give readers the wrong impression. I also wonder (having done a more close check again) if the table would not better be named (and prefixed with) lmpTeLinkVerification. That seems even more consistent that it is now. But I won't block on this. 2. Use of address in example (on page 6) not according to RFC3330. In lmpNbrTable: { lmpNbrNodeId = 'c0010101'H, -- 192.1.1.1 Better would be: In lmpNbrTable: { lmpNbrNodeId = 'c0000201'H, -- 192.0.2.1 Also... I understand what you are saying here, but this is an index object and so it WILL have the length of the octet string as a prefix, so it will be prefixed with 04. But probably waht you show is OK. 3. lmpNbrRetransmitDelta OBJECT-TYPE SYNTAX Unsigned32 UNITS "bps" MAX-ACCESS read-create STATUS current DESCRIPTION "This object governs the speed with which the sender increases the retransmission interval as explained in section 10 of the Link Management Protocol specification, which is based on RFC 2914. This value is a power used to express the exponential backoff. The ratio of two succesive retransmission intervals is (1 + Delta)." Mmm.. "This value is a power..." So is UNITS "bps" then correct? 4. I had suggested (in an earlier review) to check for naming consistency throughout the document. I see things like lmpCcXxxx where these are just scalar objects lmpControlChannelTable as a table lmpCcYyyy where these are columns in that table lmpControlChannelPerfTable (a table that augments previous table lmpCcZzzz where these are columns in thius 2nd table Such does not help humans to easily see which objects are in which table or not in a table at all. And it increases the risk of future name clashes. I won't block on it though. 5. lmpTeLinkNbrRemoteNodeId OBJECT-TYPE SYNTAX LmpNodeId MAX-ACCESS read-create STATUS current DESCRIPTION "This is the Node ID of the TE link remote node. This value may be learned during control channel parameter negotiation phase (in the Config message). Node ID address type must be IPv4." Mmmmm the datatype is LmpNodeId. The TC does not say at all that this has an "address type". Maybe the DESCRIPTION clause for the TC needs some expansion/clarification? 6. lmpGlobalLinkVerificationInterval OBJECT-TYPE SYNTAX Unsigned32 UNITS "ms" MAX-ACCESS read-write STATUS current DESCRIPTION "This object indicates how often the link verification procedure is executed. The interval is in milliseconds. So similar to an earlier comment that "nm" would be better spelled out as "nanometers", I think that here the "ms" would be better spelled out to "milliseconds". But again I won't block on it. I had just hoped that I do not have to list every single instance where similar comments apply. 7. lmpNotificationMaxRate is the max number of notifications per second. Mmm... that does not give very fine control does it. Why not max per minute? That would give much better "congestion avoidance" control. I would not be surprised if Transport ADs object to the current control mecahnism. Also a DEFVAL of zero (meaning no limit) seems not very prudent. -----Original Message----- From: Wijnen, Bert (Bert) Sent: maandag 19 juli 2004 16:51 To: 'Adrian Farrel'; Martin Dubuc; 'Thomas D. Nadeau' Cc: zinin@psg.com; 'Kireeti Kompella' Subject: RE: draft-ietf-ccamp-lmp-mib-09.txt Mmm.... can you explain why you did this: E:smimibswork>smi?insmilint -l 6 -s -m -inamelength-32 ./LMP-MIB ./LMP-MIB:140: [5] {sequence-order} warning: SEQUENCE element #3 `lmpNbrRetryLimit' does not match order of columnar objects under `lmpNbrEntry' ./LMP-MIB:676: [5] {sequence-order} warning: SEQUENCE element #51 `lmpCcChannelStatusRspSent' does not match order of columnar objects under `lmpControlChannelPerfEntry' ./LMP-MIB:1569: [5] {sequence-order} warning: SEQUENCE element #38 `lmpTeChannelStatusRspSent' does not match order of columnar objects under `lmpTeLinkPerfEntry' I know it is only a warning message, but it is weird compared to what we normally see in a MIB module. Thanks, Bert |
2004-07-19
|
10 | Bert Wijnen | Status date has been changed to 2004-07-19 from 2004-05-20 |
2004-05-25
|
10 | (System) | Sub state has been changed to AD Follow up from New Id Needed |
2004-05-25
|
09 | (System) | New version available: draft-ietf-ccamp-lmp-mib-09.txt |
2004-05-20
|
10 | Bert Wijnen | State Changes to AD Evaluation::Revised ID Needed from AD Evaluation by Bert Wijnen |
2004-05-20
|
10 | Bert Wijnen | Comments on revision 8 send to author and wg chair (Adrian) |
2004-05-20
|
10 | Bert Wijnen | State Change Notice email list have been change to , ,, from , , |
2004-05-20
|
10 | Bert Wijnen | Status date has been changed to 2004-05-20 from 2003-12-31 |
2004-05-20
|
10 | Bert Wijnen | State Change Notice email list have been change to , , from , , |
2004-05-20
|
10 | Bert Wijnen | State Changes to AD Evaluation from Publication Requested by Bert Wijnen |
2004-04-15
|
10 | Barbara Fuller | State Changes to Publication Requested from AD is watching by Barbara Fuller |
2004-04-15
|
10 | Barbara Fuller | Alex is the Area Advisor for the CCAMP WG. However, Bert is listed as the Shepherding AD for this document. If the document needs to … Alex is the Area Advisor for the CCAMP WG. However, Bert is listed as the Shepherding AD for this document. If the document needs to be reassigned to Alex, then please update the I-D Tracker, or notify the Secretariat and we will do so. |
2004-04-15
|
10 | Barbara Fuller | Intended Status has been changed to Proposed Standard from None |
2004-03-09
|
08 | (System) | New version available: draft-ietf-ccamp-lmp-mib-08.txt |
2004-01-02
|
10 | Bert Wijnen | AD took MIB Doctor role and reviewed a pre-release of revision 8. So we assume those concerns will have been addressed when the final revision … AD took MIB Doctor role and reviewed a pre-release of revision 8. So we assume those concerns will have been addressed when the final revision 8 gets posted to the internet-drafts repository. -----Original Message----- From: Wijnen, Bert (Bert) Sent: woensdag 31 december 2003 19:43 To: Martin Dubuc; Wijnen, Bert (Bert) Cc: Adrian Farrel (E-mail); Kireeti Kompella (E-mail); Alex Zinin (E-mail) Subject: MIB DOctor review of prelimenary/preview draft-ietf-ccamp-lmp-mib-08.txt Martin, My comments on the rev 8 that you did send me today: 1. I see lmpNbrRetransmitDelta OBJECT-TYPE SYNTAX Unsigned32 MAX-ACCESS read-create STATUS current DESCRIPTION "This object governs the speed with which the sender increases the retransmission interval." What does the value represent, i.e. what is "speed? WHat are the UNITS of speed? May want to add a UNITS clause 2. lmpNbrRowStatus need to state which objects/columns MUST have a valid value before row can be made active. 3. I would rename lmpRemoteCcId Unsigned32, lmpRemoteCcAddressType InetAddressType, lmpRemoteCcIpAddr InetAddress, into lmpCcRemoteId Unsigned32, lmpCcRemoteAddressType InetAddressType, lmpCcRemoteIpAddr InetAddress, so that it is easier to see that they are n the CC table, like all the other objects in that table 4. Description clause of lmpRemoteCcIpAddr has: configuration. For point-to-point configuration, the remote control channel address can be of type unknown and this object set the zero length string. Might be easier to read this way: configuration. For point-to-point configuration, the remote control channel address can be of type unknown in which case this object must be the zero length string. 5.lmpCcAuthentication OBJECT-TYPE SYNTAX TruthValue MAX-ACCESS read-create STATUS current DESCRIPTION "This object indicates whether the control channel should use authentication." So what if it is a true value, can someone decide not to use authentication? Or would it be better o state "MUST use authentication" ?? 6. lmpCcOperStatus has in DESCRIPTION clasue: "The actual operational status of this control channel interface." I thought it is not always an "interface"? Should the word "interface" just be removed as with teh AdminStatus DESCRIPTION? 7. lmpCcRowStatus pls describe which objects/columns must have appropriate/valid and consistent values before row can be activated. Could be something aka: "all read-create objects in a row must have valid and consistent values before the row can be activated." if that is indeed the case. Some values of course can be set from the DEFVAL or other defaults as you describe in the specific objects. There are more RowStatus objects in the MIB that have similar cocern I will not repeat 8 Performance table. - Formally, I think every Counter32 object needs to specify which timer functions as the discontinuity timer. I see it is in the DESRIPTION clause of the ...Entry. I can live with that. Let us see if anyone barks. - Formally all descriptors of a counter32 object need to express plurality. Not sure they all do. I can live with it though. 9. lmpTeLinkEntry OBJECT-TYPE SYNTAX LmpTeLinkEntry MAX-ACCESS not-accessible STATUS current DESCRIPTION "An entry in this table exists for each ifEntry with an ifType of teLink(200), i.e. for every TE link. An ifEntry with an ifIndex must exist before the corresponding teLinkEntry is created. If a TE link entry in the ifTable is do you mean teLinkEntry or lmpTeLinkEntry ?? I think I would s/a TE link entry in the ifTable/ an entry in the ifTable with ifType of teLink(200)/ destroyed, then so is the corresponding entry in the teLinkTable. The administrative status value is controlled do you mean teLinkTable or lmpTeLinkTable ?? from the ifEntry. Setting the administrative status to testing prompts LMP to start link verification on the TE link. TE link, or LMP TE link ?? Information about the TE link that is not LMP specific is also contained in teLinkTable [TELINK-MIB]." so that is duplicated info? If so, pls say so and say that agent is required to keep data in sync. INDEX { ifIndex } As you can see I worry a lot here about being very explicit and clear. All these tables and types and names/descriptors that look alike for a new implementor can be very confusing! Pls do check for this confusing-text and/or possible ambuguity for all objects in this table and in the lmpLinkVerificationTable It would be good to aad something to the lmpTeLinkTable DESCRIPTION clause explaining the rleation ships between this table, teLinkTable and ifTable. Even if there is no relation, it is confusing enough to then state that explicitly. 10.Would it be btetr to rename lmpTeLinkNbrNodeId to lmpTeLinkNbrRemoteNodeId ? 11. lmpLinkVerificationInterval add a UNITS clause 12.lmpVerifyAllLinks OBJECT-TYPE SYNTAX INTEGER { verifyAllLinks(1), verifyNewLinks(2) } I think I (personally) would use a TruthValue (personal taste) 13.lmpVerifyTransmissionRate and lmpVerifyWavelength add UNITS clause. Any reasonable DEFVALs possible? Any usefull DEFVALs for other columns in this table? In general, DEFVALs may make it much easier to do rowCreation. 14. No RowStatus for the read-create lmpLinkVerificationTable ?? 15. IN DESCRIPTION clause of lmpDataLinkEntry I see a citation: used to get information in the componentLinkTable [TELINK-MIB]." When the MIB module gets extracted from the RFC, then the citation becomes a dangling ptr. Better to use "RFC xxx" or mention the exact MIB MODULE name directly instead of as a citation. Pls make sure that other citations do not occur in the MIB module. By the way, a citation in a comment line is fine (-- [some-citation]) 16. Your naming convention/consistency for lmpDataLinkTable is MUCH better than for the earlier tables! 17. lmpDataLinkPerfTable What is/are the discontinuityTimer object(s)?? Probably: lmpDataLinkDiscontinuityTime but you do not say so in ENtry DESCRIPTION clause 18. lmpNotificationMaxRate I think I would put all of the comment lines about the notifications into teh DESCRIPTION clause of this object. Personal taste maybe. But realize that some NMS systems use the DESCRIPTION clauses as online help for operators. 19. For lmpTeLinkPropertyMismatch only the first time a mismatch is detected. Otherwise, for a given TE link, this notification can occur no more than once per verification interval." Can you add the objectName for that verification interval. Helps peopel to quickly find it. Same for lmpDataLinkPropertyMismatch and some other notifications 20. For all StorageType objects,. is there not a suitable DEFVAL, probably nonVolatile !!?? 21. WHen I see: DESCRIPTION "Collection of objects needed for LMP interface configuration." ::= { lmpGroups 2 } lmpCcIsInterfaceGroup OBJECT-GROUP OBJECTS { lmpCcIsIf } STATUS current DESCRIPTION "Objects needed to implement control channels that are interfaces." ::= { lmpGroups 3 } I wonder... are those objects "needed"? People can configure manually, no? Or via CLI?? I would maybe word it different (personal taste, so I will not block on this): "Collection of objects that represent then LMP interface configuration." and: "Objects representing control channels that are interfaces." or: "Objects which can be used to configure control channels that are interfaces." or some such. 22. lmpNotificationGroup DESCRIPTION "Set of notifications implemented in this module. None is mandatory." remove the "None is mandatory". That is what you state in the MODULE-COMPLIANCE. Here you only group the set of notifications together, so they can be used in one or more compliance statements where a statement is made about being mandatory, conditional or optional. I'd also change "implemented" into "defined". The MIB Module only defines them, an agent implements them. 23. I see: 1.3.6.1.2.1.10.xx.1.10.1.9 lmpCcAuthentication [LMP-MIB]: columnar-object-type 1.3.6.1.2.1.10.xx.1.10.1.12 lmpCcHelloInterval [LMP-MIB]: columnar-object-type why is there a gap between 9 and 12?? If there is no good reason, then pls don't If there is a reason, pls explain it in comments in the MIB 24. I wonder why lmpVerifyTransportMechanism = 1, -- j016OverheadBytes lmpVerifyAllLinks = verifyAllLinks(1) instead of: lmpVerifyTransportMechanism = j016OverheadBytes(1) lmpVerifyAllLinks = verifyAllLinks(1) is that not more consistent? 25. you have in Sect 8.1 (and also the sect before that a TBD) ifType The value that is allocated for LMP is TBD. This number will be assigned by the IANA. So where is that request to IANA made to assign an ifType for LMP? You better get that added, so they know where to look for it. ANd if it is in this document, then add an IANA Considerations section. Might be good anyway to point out all the TBDs that they need to look at. It will just speed up the process at a later stage. admininstrative - I'd update the dates to 2004 (copyright, REVISION, LAST-UPDATED etc) I did not check all the non-MIB text in detail. FOr example I did not check your examples in detail... did I not do so a few months back? If not, I may want to still do that. Anyway... it is now 7:40pm on Dec 31st for me. So I go and have my wine and fun and won't be back till Jan 2nd. Thanks, Bert |
2004-01-02
|
10 | Bert Wijnen | State Change Notice email list have been change to , , from |
2004-01-02
|
10 | Bert Wijnen | Status date has been changed to 2003-12-31 from 2003-09-01 |
2003-10-02
|
07 | (System) | New version available: draft-ietf-ccamp-lmp-mib-07.txt |
2003-09-01
|
10 | Bert Wijnen | State Changes to AD is watching from Publication Requested by Bert Wijnen |
2003-09-01
|
10 | Bert Wijnen | Initial MIB-doctor/AD review: -----Original Message----- From: Wijnen, Bert (Bert) [mailto:bwijnen@lucent.com] Sent: maandag 1 september 2003 22:43 To: 'Martin Dubuc' Cc: Ccamp-wg (E-mail) Subject: … Initial MIB-doctor/AD review: -----Original Message----- From: Wijnen, Bert (Bert) [mailto:bwijnen@lucent.com] Sent: maandag 1 september 2003 22:43 To: 'Martin Dubuc' Cc: Ccamp-wg (E-mail) Subject: RE: LMP MIB revision 6 Well well... SMICng says: W: f(lmp.mi2), (2251,20) Variable "ifIndex" in notification "lmpDataLinkPropertyMismatch" is an index for a table W: f(lmp.mi2), (2306,20) Variable "ifIndex" in notification "lmpTeLinkDegraded" is an index for a table W: f(lmp.mi2), (2316,20) Variable "ifIndex" in notification "lmpTeLinkNotDegraded" is an index for a table W: f(lmp.mi2), (2329,20) Variable "ifIndex" in notification "lmpDataLinkVerificationFailure" is an index for a table W: f(lmp.mi2), (2641,19) MIN-ACCESS value identical to access specified for "lmpCcOperStatus" W: f(lmp.mi2), (2693,19) MIN-ACCESS value identical to access specified for "lmpTeLinkOperStatus" W: f(lmp.mi2), (2761,19) MIN-ACCESS value identical to access specified for "lmpDataLinkActiveOperStatus" W: f(lmp.mi2), (2768,19) MIN-ACCESS value identical to access specified for "lmpDataLinkPassiveOperStatus" smilint says: .LMP-MIB:2425: [2] {subtype-enumeration-illegal} named number `degraded(4)' illegal in sub-type .LMP-MIB:2439: [2] {subtype-enumeration-illegal} named number `up(1)' illegal in sub-type .LMP-MIB:2439: [2] {subtype-enumeration-illegal} named number `down(2)' illegal in sub-type .LMP-MIB:2439: [2] {subtype-enumeration-illegal} named number `degraded(4)' illegal in sub-type .LMP-MIB:2444: [2] {subtype-enumeration-illegal} named number `up(1)' illegal in sub-type .LMP-MIB:2444: [2] {subtype-enumeration-illegal} named number `down(2)' illegal in sub-type .LMP-MIB:2444: [2] {subtype-enumeration-illegal} named number `degraded(4)' illegal in sub-type .LMP-MIB:2694: [2] {subtype-enumeration-illegal} named number `degraded(4)' illegal in sub-type .LMP-MIB:2763: [2] {subtype-enumeration-illegal} named number `up(1)' illegal in sub-type .LMP-MIB:2763: [2] {subtype-enumeration-illegal} named number `down(2)' illegal in sub-type .LMP-MIB:2763: [2] {subtype-enumeration-illegal} named number `degraded(4)' illegal in sub-type .LMP-MIB:2769: [2] {subtype-enumeration-illegal} named number `up(1)' illegal in sub-type .LMP-MIB:2769: [2] {subtype-enumeration-illegal} named number `down(2)' illegal in sub-type .LMP-MIB:2769: [2] {subtype-enumeration-illegal} named number `degraded(4)' illegal in sub-type .LMP-MIB:138: [5] {inetaddress-inetaddresstype} warning: `InetAddress' object should have an accompanied preceding `InetAdressType' object .LMP-MIB:386: [5] {inetaddress-inetaddresstype} warning: `InetAddress' object should have an accompanied preceding `InetAdressType' object .LMP-MIB:1213: [5] {inetaddress-inetaddresstype} warning: `InetAddress' objectshould have an accompanied preceding `InetAdressType' object Some details: 1. lmpNbrNodeId OBJECT-TYPE SYNTAX InetAddress (SIZE(4)) MAX-ACCESS not-accessible STATUS current DESCRIPTION "This is a unique index for an entry in the LmpNbrTable. This value represents the remote Node ID. The Node ID address type must be IPv4." ::= { lmpNbrEntry 1 } You do know that MIB modules need to be IPv6 friendly, no? Is LMP itself such that it only works with IPv4? I doubt that IESG will approve new protocols that do not support IPv6. 2. For these 2 objects: lmpNbrRetransmitInterval LmpRetransmitInterval, lmpNbrRetryLimit Unsigned32, You may want to add some text as to howyou intend to deal with congestion (RFC 2914). This over UDP if I remember well? I think it would be good to point explicitly to the text in lmp document (sect 10). 3. lmpNbrRowStatus OBJECT-TYPE SYNTAX RowStatus MAX-ACCESS read-create STATUS current DESCRIPTION "This variable is used to create, modify, and/or delete a row in this table. All read-create objects can only be changed when lmpNbrRowStatus is active." That sounds strange. The notInService status was specifically created to allow changes for cases where changes were not allowed while row was active. Here you say it MUST be active in order to make changes? Or did you mean "can not be changed"? This occurs in more (maybe all) RowStatus objects in this doc. 4. You have some objedts that are not part of a table, yet are read-write. For example lmpAdminStatus lmpCcHelloIntervalDefault lmpCcHelloIntervalDefaultMin ... and more ... What is the persistency behaviour of these objects? In other words: is the value preserved over a reboot? 5. Is lmpCcIsIf not redundant? The way I read it, then if the lmpCcUnderlyingIfIndex has a value of zero, then it is NOT an interface, no? 6. lmpRemoteCcAddressType and lmpRemoteCcAddress In the telink MIB you have done things correctly and as presscribe by RFC3291. Here you have not. Why ? Even in this doc you have done it correctly in some places. 7. Mmmmmm??? 52 counters to "measure the performance" of the LMP channels? Sounds overwhelming to me. 8. Formally, every Counterxx object needs to include in its descritpion clause which object indicates a potential discontinuity. I understand that they all are covered by lmpCcCounterDiscontinuityTime in the same row. Maybe write something about that in the ...Entry description clause as well if you do not want to add text to every Counter 9. lmpTeLinkTable DESCRIPTION "This table contains a collection of TE link." Means what?? 10. lmpLinkVerificationInterval What does a value of zero mean? Any comments regarding possible congestiuon if the value is set to a very small number of msecs? 11. lmpLinkVerificationTable Or is it better namep lmpTeLinkLinkVerificationTable and then rename objects as well? If is correct name, Then maybe rename lmpTeLinkBitRate to lmpLinkVerificationTeLinkBitRate lmpTeLinkWavelength to lmpLinkVerificationTeLinkWavelength 12. lmpVerifyTransportMechanism There are a few TBDs in there. Better decide before you submit to AD 13. lmpTeLinkBitRate DESCRIPTION "This is the bit rate at which the Test messages will be transmitted and is expressed in bytes per second." A BIT rate that gest expressed in BYTES per second? Possible of course but strange, no? Why not call it ByteRate? 14. Another 40 or so counters for lmpTeLinkPerfTable ??? 15. lmpDataLinkPropertyMismatch NOTIFICATION-TYPE OBJECTS { ifIndex, lmpDataLinkRemoteIfId } The second object already (implicitly) also contains the value of the first object (it is the index part of the OID). 16. I see quite a few NOTIFICATION-TYPES. You can enable or disable them. But when they are enabled, what kind of rates per second should we fear for in a worst case conidition? Pls think about both an agent (i.e. an LMP node) generating them, but also think about a SNMP management station receiving them from potentially 100s or 1000s of LMP nodes. 17. On page 7 I see: lmpDataLinkAddressType = unnumbered(1), Mm... that is not a valid value for an InetAddressType 18. Security COnsiderations You are not following the guidelines to describe first the SET (read-write, read-create) issues/concerns/vulnerabilities and then the read-only sensistivities. So I would not be surprised to see push back from the Security front. I have done a pretty serious review... but not exhaustive. Thanks, Bert -----Original Message----- From: Wijnen, Bert (Bert) Sent: maandag 1 september 2003 20:14 To: 'Martin Dubuc'; Wijnen, Bert (Bert) Subject: RE: LMP MIB I have it on my todo-list. I am currently checking te-link mib. Thanks, Bert -----Original Message----- From: Martin Dubuc [mailto:m.dubuc@rogers.com] Sent: maandag 1 september 2003 19:55 To: Bert Wijnen Subject: LMP MIB Bert, I know you have been quite busy with all the MPLS drafts. I just wanted to know if you had time review the LMP MIB draft and if not, when you think you'll get around reviewing it. Regards, Martin |
2003-09-01
|
10 | Bert Wijnen | Status date has been changed to 2003-09-01 from |
2003-09-01
|
10 | Bert Wijnen | Draft Added by Bert Wijnen |
2003-05-30
|
06 | (System) | New version available: draft-ietf-ccamp-lmp-mib-06.txt |
2003-04-21
|
05 | (System) | New version available: draft-ietf-ccamp-lmp-mib-05.txt |
2002-11-05
|
04 | (System) | New version available: draft-ietf-ccamp-lmp-mib-04.txt |
2002-07-02
|
03 | (System) | New version available: draft-ietf-ccamp-lmp-mib-03.txt |
2002-05-30
|
02 | (System) | New version available: draft-ietf-ccamp-lmp-mib-02.txt |
2002-03-01
|
01 | (System) | New version available: draft-ietf-ccamp-lmp-mib-01.txt |
2001-10-19
|
00 | (System) | New version available: draft-ietf-ccamp-lmp-mib-00.txt |