[mpls] Review of draft-ietf-mpls-tp-linear-protection-mib-11

Brian Carpenter <brian.e.carpenter@gmail.com> Sun, 15 January 2017 22:04 UTC

Return-Path: <brian.e.carpenter@gmail.com>
X-Original-To: mpls@ietf.org
Delivered-To: mpls@ietfa.amsl.com
Received: from ietfa.amsl.com (localhost [IPv6:::1]) by ietfa.amsl.com (Postfix) with ESMTP id 59F3E12986B; Sun, 15 Jan 2017 14:04:07 -0800 (PST)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
From: Brian Carpenter <brian.e.carpenter@gmail.com>
To: gen-art@ietf.org
X-Test-IDTracker: no
X-IETF-IDTracker: 6.40.3
Auto-Submitted: auto-generated
Precedence: bulk
Message-ID: <148451784736.3226.15889507299394185360.idtracker@ietfa.amsl.com>
Date: Sun, 15 Jan 2017 14:04:07 -0800
Archived-At: <https://mailarchive.ietf.org/arch/msg/mpls/oV6nejaxlMCWMkFRMIK6cEru0LM>
Cc: mpls@ietf.org, draft-ietf-mpls-tp-linear-protection-mib.all@ietf.org, brian.e.carpenter@gmail.com
Subject: [mpls] Review of draft-ietf-mpls-tp-linear-protection-mib-11
X-BeenThere: mpls@ietf.org
X-Mailman-Version: 2.1.17
List-Id: Multi-Protocol Label Switching WG <mpls.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/mpls>, <mailto:mpls-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/mpls/>
List-Post: <mailto:mpls@ietf.org>
List-Help: <mailto:mpls-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/mpls>, <mailto:mpls-request@ietf.org?subject=subscribe>
X-List-Received-Date: Sun, 15 Jan 2017 22:04:07 -0000

Reviewer: Brian Carpenter
Review result: Ready with Issues

Gen-ART Last Call review of
draft-ietf-mpls-tp-linear-protection-mib-11

I am the assigned Gen-ART reviewer for this draft. The General Area
Review Team (Gen-ART) reviews all IETF documents being processed
by the IESG for the IETF Chair.  Please treat these comments just
like any other last call comments.

For more information, please see the FAQ at
<http://wiki.tools.ietf.org/area/gen/trac/wiki/GenArtfaq>.

Document: draft-ietf-mpls-tp-linear-protection-mib-11.txt
Reviewer: Brian Carpenter
Review Date: 2017-01-16
IETF LC End Date: 2017-01-26
IESG Telechat date:  

Summary: Ready with minor issues
--------

Comment:
--------

I have not reviewed most details of the MIB module itself. As usual,
I trust the MIB Doctors.

"We know of a handful of implementations (or intent to implement)."
Good. It would have been nice to see an Implementation Status section
under RFC 6982.

Minor issues:
-------------

   At the time of writing, Simple Network Management Protocol (SNMP)
SET
   is no longer recommended as a way to configure MPLS networks as
was
   described in RFC 3812 [RFC3812].

RFC3812 is explicit that it should be used for configuration:

   This MIB module should be used in conjunction with the
   companion document [RFC3813] for MPLS based traffic engineering
   configuration and management.

RFC3812 has not been formally updated or obsoleted. Therefore, it
seems
to me that the present draft should formally update RFC3812 in this
respect.

Does the same issue apply to RFC3813, whose Abstract also states that
it is used to configure an LSR?