Last Call Review of draft-perrault-behave-natv2-mib-03
review-perrault-behave-natv2-mib-03-opsdir-lc-jiang-2015-05-04-00

Request Review of draft-perrault-behave-natv2-mib
Requested rev. no specific revision (document currently at 05)
Type Last Call Review
Team Ops Directorate (opsdir)
Deadline 2015-04-29
Requested 2015-04-05
Authors Simon Perreault, Tina Tsou, Senthil Sivakumar, Tom Taylor
Draft last updated 2015-05-04
Completed reviews Genart Last Call review of -03 by Suresh Krishnan (diff)
Secdir Last Call review of -03 by Takeshi Takahashi (diff)
Opsdir Last Call review of -03 by Sheng Jiang (diff)
Assignment Reviewer Sheng Jiang
State Completed
Review review-perrault-behave-natv2-mib-03-opsdir-lc-jiang-2015-05-04
Reviewed rev. 03 (document currently at 05)
Review result Has Issues
Review completed: 2015-05-04

Review
review-perrault-behave-natv2-mib-03-opsdir-lc-jiang-2015-05-04

Hi,

I have reviewed this document as part of the Operational directorate's ongoing effort to review all IETF documents being processed by the IESG. These comments were written with the intent of improving the operational aspects of the IETF drafts. Comments that are not addressed in last call may be included in AD reviews during the IESG review. Document editors and WG chairs should treat these comments just like any other last call comments.

This Standards Track document defines a new version of NATV2-MIB for monitoring the NAT devices. It is intended to replace module NAT-MIB (RFC 4008). However, deprecating all objects in NAT-MIB (v1) is by a companion document draft-perrault-behave-deprecate-nat-mib-v1. This document is well written. While I am not a MIB expert, my review does not include the detailed design of MIB objects. This document does require another review by MIB expertise or MIB doctor.

I have some minor comments as follow:

1)	On page 31, as it defined 
natv2SubscriberThresholdPortMapEntriesHigh OBJECT-TYPE
       SYNTAX Unsigned32
       MAX-ACCESS read-write
       STATUS current
       DESCRIPTION
           "Notification threshold for total number of port mappings
            active for this subscriber. Whenever
            natv2SubscriberPortMapEntries is updated, if it equals or
            exceeds natv2SubscriberThresholdPortMapEntriesHigh, the
            notification
            natv2NotificationSubscriberPortMappingEntriesHigh is
            triggered, unless the notification is disabled by setting
            the threshold to 0. Reporting is subject to the minimum
            inter-notification interval given by
            natv2SubscriberNotificationInterval. If multiple
            notifications are triggered during one interval, the agent
            MUST report only the one containing the highest value of
            natv2SubscriberPortMapEntries and discard the others."
       DEFVAL
            { 0 }
       ::= { natv2SubscriberEntry 16 }

It needs a sentence to describe that the threshold is default to set 0.

2)	On page 55, it is missing the definition of the natv2PoolEntry 13. The continuous numbers jumped from 12 to 14.

3)	On page 57, why not to define the natv2PoolRange objects into the natv2PoolTable? These objects should be part of the natv2Pool objects. 

Best regards,

Sheng