[Tsv-art] Tsvart last call review of draft-ietf-softwire-iftunnel-04
David Black via Datatracker <noreply@ietf.org> Tue, 07 May 2019 22:45 UTC
Return-Path: <noreply@ietf.org>
X-Original-To: tsv-art@ietf.org
Delivered-To: tsv-art@ietfa.amsl.com
Received: from ietfa.amsl.com (localhost [IPv6:::1]) by ietfa.amsl.com (Postfix) with ESMTP id 89B4C12006E; Tue, 7 May 2019 15:45:51 -0700 (PDT)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
From: David Black via Datatracker <noreply@ietf.org>
To: tsv-art@ietf.org
Cc: softwires@ietf.org, ietf@ietf.org, draft-ietf-softwire-iftunnel.all@ietf.org
X-Test-IDTracker: no
X-IETF-IDTracker: 6.96.0
Auto-Submitted: auto-generated
Precedence: bulk
Reply-To: David Black <david.black@dell.com>
Message-ID: <155726915148.24435.7582686501694078061@ietfa.amsl.com>
Date: Tue, 07 May 2019 15:45:51 -0700
Archived-At: <https://mailarchive.ietf.org/arch/msg/tsv-art/k-ZnpNlI_qJnve-lWCyZPYo2ouI>
Subject: [Tsv-art] Tsvart last call review of draft-ietf-softwire-iftunnel-04
X-BeenThere: tsv-art@ietf.org
X-Mailman-Version: 2.1.29
List-Id: Transport Area Review Team <tsv-art.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/tsv-art>, <mailto:tsv-art-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/tsv-art/>
List-Post: <mailto:tsv-art@ietf.org>
List-Help: <mailto:tsv-art-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/tsv-art>, <mailto:tsv-art-request@ietf.org?subject=subscribe>
X-List-Received-Date: Tue, 07 May 2019 22:45:52 -0000
Reviewer: David Black Review result: Not Ready This document has been reviewed as part of the transport area review team's ongoing effort to review key IETF documents. These comments were written primarily for the transport area directors, but are copied to the document's authors and WG to allow them to address any issues raised and also to the IETF discussion list for information. When done at the time of IETF Last Call, the authors should consider this review as part of the last-call comments they receive. Please always CC tsv-art@ietf.org if you reply to or forward this review. This draft defines a YANG module for tunnel types based on the MIB-2 tunnel type registry maintained by IANA. My fundamental concern with this draft is that the MIB-2 tunnel type registry is seriously incomplete and out of date, as there are a large number of tunnel types that aren't included in that registry, e.g., IPsec tunnel-mode AMT tunneling. In its current form, that registry does not appear to be a good starting point for specifying YANG management of tunnels. A limited justification that I could envision for defining this YANG module would be to use it for mechanical translations to YANG of existing MIBs that use MIB-2 tunnel types - if that's the justification, then it would need to be clearly stated in an applicability statement within this draft, and the discussion of extension of this YANG module would need to be aligned with that limited applicability. The proverbial "right thing to do" would be to update both the MIB-2 tunnel type registry and this draft with all of the currently known tunnel types. The references section of draft-ietf-tsvwg-rfc6040update-shim (https://datatracker.ietf.org/doc/draft-ietf-tsvwg-rfc6040update-shim/) may help in identifying tunnel protocols that should be included. A minor concern involves the use of RFC 8085 as the reference for UDP tunnels; while that's certainly better than the existing use of RFC 4087, due to the extensive design guidance in RFC 8085, designers of UDP-encapsulated tunnel protocols ought to be encouraged to register their protocols as separate tunnel types (e.g., so the network operator has some idea of what the UDP tunnel is actually being used for). This draft ought to encourage tunnel protocol designers to register their own tunnel types in preference to reuse of the UDP tunnel type, including placing text in the IANA tunnel type registry and this YANG module to encourage that course of action.
- [Tsv-art] Tsvart last call review of draft-ietf-s… David Black via Datatracker
- Re: [Tsv-art] Tsvart last call review of draft-ie… Rajiv Asati (rajiva)
- Re: [Tsv-art] Tsvart last call review of draft-ie… Black, David
- Re: [Tsv-art] Tsvart last call review of draft-ie… mohamed.boucadair
- Re: [Tsv-art] Tsvart last call review of draft-ie… Black, David
- Re: [Tsv-art] Tsvart last call review of draft-ie… mohamed.boucadair
- Re: [Tsv-art] Tsvart last call review of draft-ie… tom petch
- Re: [Tsv-art] Tsvart last call review of draft-ie… mohamed.boucadair
- Re: [Tsv-art] Tsvart last call review of draft-ie… Magnus Westerlund
- Re: [Tsv-art] Tsvart last call review of draft-ie… mohamed.boucadair
- Re: [Tsv-art] Tsvart last call review of draft-ie… Eric Vyncke (evyncke)
- Re: [Tsv-art] Tsvart last call review of draft-ie… Suresh Krishnan
- Re: [Tsv-art] Tsvart last call review of draft-ie… Eric Vyncke (evyncke)
- Re: [Tsv-art] Tsvart last call review of draft-ie… mohamed.boucadair
- Re: [Tsv-art] Tsvart last call review of draft-ie… Eric Vyncke (evyncke)
- Re: [Tsv-art] Tsvart last call review of draft-ie… Mirja Kuehlewind
- Re: [Tsv-art] Tsvart last call review of draft-ie… Rajiv Asati (rajiva)
- Re: [Tsv-art] Tsvart last call review of draft-ie… tom petch
- Re: [Tsv-art] Tsvart last call review of draft-ie… tom petch
- Re: [Tsv-art] Tsvart last call review of draft-ie… mohamed.boucadair