Opsdir last call partial review of draft-ietf-ospf-encapsulation-cap-06

Tim Wicinski <tjw.ietf@gmail.com> Tue, 29 August 2017 12:38 UTC

Return-Path: <tjw.ietf@gmail.com>
X-Original-To: ietf@ietf.org
Delivered-To: ietf@ietfa.amsl.com
Received: from ietfa.amsl.com (localhost [IPv6:::1]) by ietfa.amsl.com (Postfix) with ESMTP id AF48D132A8F; Tue, 29 Aug 2017 05:38:04 -0700 (PDT)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
From: Tim Wicinski <tjw.ietf@gmail.com>
To: ops-dir@ietf.org
Cc: ospf@ietf.org, draft-ietf-ospf-encapsulation-cap.all@ietf.org, ietf@ietf.org
Subject: Opsdir last call partial review of draft-ietf-ospf-encapsulation-cap-06
X-Test-IDTracker: no
X-IETF-IDTracker: 6.59.0
Auto-Submitted: auto-generated
Precedence: bulk
Message-ID: <150401028465.13280.5869374542696138659@ietfa.amsl.com>
Date: Tue, 29 Aug 2017 05:38:04 -0700
Archived-At: <https://mailarchive.ietf.org/arch/msg/ietf/XGTQ6RN4EcWyZWdxzxPlhZBMt0s>
X-BeenThere: ietf@ietf.org
X-Mailman-Version: 2.1.22
List-Id: IETF-Discussion <ietf.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/ietf>, <mailto:ietf-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/ietf/>
List-Post: <mailto:ietf@ietf.org>
List-Help: <mailto:ietf-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/ietf>, <mailto:ietf-request@ietf.org?subject=subscribe>
X-List-Received-Date: Tue, 29 Aug 2017 12:38:05 -0000

Review is partially done. Another review request has been registered for
completing it.

Reviewer: Tim Wicinski
Review result: Has Issues

Summary: Almost Ready

The content appears to be fine, but there are some outdated (the biggest one is
5226 replaced by 8126), but its the IANA section which appears the most
confusing.

7.1 OSPF Router Information (RI) Registry -  appears fine

7.2 OSPF Tunnel Encapsulation Attribute Sub-TLV Registry

This one defines the values being defined/allocated from "This Document" but in
Section 5, each Sub-TLV is defined in other documents, so it's totally
confusing.